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

When PackageManager is loaded, could LoadPackage give useful tips on failure? #69

Open
wilfwilson opened this issue May 20, 2021 · 2 comments
Labels
discussion Not exactly an issue to be resolved, but a discussion about the future of the package

Comments

@wilfwilson
Copy link
Member

This is just a vague idea. Suppose that LoadPackage("new"); fails. If appropriate, LoadPackage could mention things such as...

  • If PackageManager knows about the package new, but no version of new seems to be installed, it could offer to install new.
  • If new is present but not compiled, it could offer to compile the package.
  • If it fails because of a package that it depends on, it could offer to install/update/compile that dependency.

I'm not sure about the feasibility of any of this! Just thought I'd record my ideas here.

@mtorpey mtorpey added feature request New feature discussion Not exactly an issue to be resolved, but a discussion about the future of the package and removed feature request New feature labels May 26, 2021
@mtorpey
Copy link
Collaborator

mtorpey commented May 26, 2021

Some good ideas here! We've talked about something like InstallAndLoadPackage that would combine the two somehow, but maybe the idea of having LoadPackage give hints to PackageManager is a good one.

A minimal idea would be a message that says "Could not load package. Consider using InstallPackage to install or update it.

I guess this is something that needs to be done in GAP itself? Or perhaps we could have this package override LoadPackage in some way.

@slel
Copy link
Contributor

slel commented Jun 29, 2022

Another idea:

  • When LoadPackage("new"); requires access to the web,
    first check for network connectivity. If it turns out there is
    no internet access, fail with a clear message stating that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Not exactly an issue to be resolved, but a discussion about the future of the package
Projects
None yet
Development

No branches or pull requests

3 participants