Skip to content

Document development setup #33

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gyst
Copy link

@gyst gyst commented May 1, 2025

I had to perform a number of undocumented tweaks to get a working Euphorie development environment. This documents that setup.

euphorie.deployment-overrides

[versions]
Euphorie =
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The provided pin in base.cfg is not available on the configured pypi servers.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you are doing something wrong:

See:

  1. Euphorie = 18.0.0
  2. https://pypi.org/project/Euphorie/18.0.0/

saconfig-uri = sqlite:///app/var/quoira.sql

[instance]
# remove the zcml declaration if you configure a main-package
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

base.cfg defines Euphorie as the default main-package.

main-package = Euphorie

But it then proceeds to load the main package zcml

${settings:main-package}

which errors out because Euphorie has no top level zcml

@pilz
Copy link
Member

pilz commented May 1, 2025

@ale-rt Could you take a look if the approach is good or if we should do something different?

@pilz pilz requested a review from ale-rt May 1, 2025 10:35
Copy link
Member

@ale-rt ale-rt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not understand the purpose of this PR.
What should people do with the provided config/custom.cfg?

@gyst
Copy link
Author

gyst commented May 6, 2025

The purpose of this PR is to ship with a working configuration people can start with. As distinct to having some incomplete pointers in the README. People can cp config/custom.cfg . && make and they'll be up and running.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants