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

add --licensedir to allow installing license files elsewhere (#5281) #5323

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@juhp
Copy link
Collaborator

juhp commented May 14, 2018

  • needed for modern RPM-based Linux distros like Fedora and SUSE
  • defaults to docdir
  • Patches conform to the coding conventions.
  • Any changes that could be relevant to users have been recorded in the changelog.
  • The documentation has been updated, if necessary.

I tested with a dummy package (bootstrapped with stack to build Cabal with ghc-8.4.2)

@hvr

This comment has been minimized.

Copy link
Member

hvr commented May 14, 2018

This still will break my upcoming tooling if it's merged without addressing my voiced concerns

add --licensedir to allow installing license files elsewhere (#5281)
- needed for modern RPM-based Linux distros like Fedora and SUSE
- defaults to docdir

@juhp juhp force-pushed the juhp-licensedir branch from fc5bc48 to 445fcf8 May 14, 2018

@juhp

This comment has been minimized.

Copy link
Collaborator Author

juhp commented May 14, 2018

One think I didn't understand is why copyPackage doesn't install docfiles (other than license)?
Or is that the expected behaviour?

@gbaz

This comment has been minimized.

Copy link
Collaborator

gbaz commented May 14, 2018

An easy workaround could be to always install license files where they currently go, but optionally also either copy or symlink them elsewhere?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.