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

package manager struggling with dependencies #9

Open
duncdrum opened this issue Jul 7, 2018 · 3 comments
Open

package manager struggling with dependencies #9

duncdrum opened this issue Jul 7, 2018 · 3 comments
Labels

Comments

@duncdrum
Copy link
Contributor

duncdrum commented Jul 7, 2018

what is the problem?

When users have installed an application called app that depends on the app-lib (think tei-publisher, or anything depending on shared-resources) updates can break installation in the following way:

  • app 1.0 depends on app-lib 1.0
  • app 1.1 depends on app-lib 1.1

Packemanager shows two new updates: One to app, one to app-lib.
If users attempt to update app first, the process fails silently, since the requested app 1.1 dependency on app-lib 1.1 is at this point not fulfilled. app can no longer be reached from dashboard.

what did I expect

When first installing apps, dependencies are automatically resolved and installed. I d expect the same behavior for updates.

The old package manager displays installation errors in a pop-up warning, the new one should do something similar. Silent failures are never a good thing.

to reproduce

  • make sure tei-publisher 3.0.x, and tei-publisher-lib 2.2.0 are installed.
  • open package manger to see two available updates
  • update tei-publisher first (and lib second or not at all)
  • check the logs, and try to access tei-publisher either from dashboard, launcher.
    • Alternatively, refresh browser and look for tei-publisher under installed apps.
@duncdrum
Copy link
Contributor Author

duncdrum commented Apr 3, 2019

@eXist-db/exist-webcomponents any comments? see also #10 in both instances the fact that intallation errors are no longer displayed creates problems for users

@JoernT
Copy link
Member

JoernT commented Apr 11, 2019

this ticket does not belong here but to exist-db/packageservice. This is the app querying the repo. The code has been factored out as it is used by launcher and packagemanager.

@duncdrum regarding errors - yes, that's true and due to the fact that the server is returning 200 also for errors while client expects an error response for its handler to fire. Thus each ajax request needs to check whether response didn't contain an error. In some place this has been done but not in all.

@duncdrum
Copy link
Contributor Author

feel free to move the ticket. Or split, transient dependency handling is one problem, the app reporting errors (or not) is another.

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

No branches or pull requests

2 participants