Skip to content
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

lace public repo #124

Closed
duncdrum opened this issue Oct 3, 2020 · 4 comments
Closed

lace public repo #124

duncdrum opened this issue Oct 3, 2020 · 4 comments

Comments

@duncdrum
Copy link

duncdrum commented Oct 3, 2020

http://heml.ddns.net:8080/exist/apps/public-repo/ seems either down or not publicly accessible, since installing the app makes this the default app repo for packagemanager this is a bit unfortunate.

the switch should be documented so users are aware what's happening, and in a future update maybe we can get this to work more elegantly so that users still get their regular app updates from exist-db's default public repo.

see eXist-db/existdb-packageservice#16

duncdrum added a commit to duncdrum/Lace2 that referenced this issue Oct 3, 2020
@brobertson
Copy link
Owner

Thanks. I don't consider this feature as useful as did in the past, especially since one makes the eXist repo unuseable. So my current testing code does not use a repo, and the plan is for users just to download from a website. The downside of this is that the package dependency of images and texts can't be automatically resolved, but it's not too much to ask that people install two packages, and I've found that in general package management has fallen on my shoulders for our project, not on individual users.

@duncdrum
Copy link
Author

duncdrum commented Oct 3, 2020

@brobertson i think the feature could be very useful. Not just for this app. So in light of this, I would slightly modify the replace statement in your preinstall.xq to insert And see that we can fix the problem upstream so that specifying multiple repos, will actually show packages from both.

@brobertson
Copy link
Owner

Nice idea. Thanks.

@duncdrum
Copy link
Author

duncdrum commented Oct 5, 2020

@brobertson the PR is here eXist-db/existdb-packageservice#17 I ll incorporate the necessary updates into my fork.

duncdrum added a commit to duncdrum/Lace2 that referenced this issue Oct 5, 2020
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

No branches or pull requests

2 participants