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

Add flag to cabal to specify requiredInstall packages in modular solver. #4209

Closed
robinp opened this Issue Jan 6, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@robinp
Copy link

robinp commented Jan 6, 2017

In the modular solver at

P.requireInstalled (`elem` [ mkPackageName "base"
, the list of wired-in packages is hardcoded.

When one wants to rebuild one of these packages (see mail thread about rebuilding base), the given package needs to be commented out, and cabal rebuilt locally.

It would be nice if the list of packages came from a flag (defaulting to the current set?), which users could override. Or, if it's needed for some reason, keep the list hardcoded but let some packages be removed with a flag.

grayjay added a commit to grayjay/cabal that referenced this issue Jan 12, 2017

Add flag to allow the dependency solver to install any package (issue h…
…askell#4209).

--install-base-libraries makes cabal ignore the list of packages that cannot be
installed or upgraded.
@grayjay

This comment has been minimized.

Copy link
Collaborator

grayjay commented Jan 12, 2017

@robinp I rebased my branch from #3987 and made a PR. The flag always ignores the whole list of non-installable libraries, because cabal already has "installed" constraints. For example, cabal new-build --constraint="base installed" --install-base-libraries would allow cabal to install template-haskell but not base.

EDIT: The PR is #4227.

grayjay added a commit that referenced this issue Jan 12, 2017

Merge pull request #4227 from grayjay/install-base
Add flag to allow the dependency solver to install any package (issue #4209).
@grayjay

This comment has been minimized.

Copy link
Collaborator

grayjay commented Jan 12, 2017

Fixed by #4227.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment