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.

Sign up for GitHub

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

Jump to bottom

collision between groovy LICENSE openjdk LICENSE #19872

Closed
pmahoney-raise opened this issue Oct 25, 2016 · 7 comments
Closed

collision between groovy LICENSE openjdk LICENSE #19872

pmahoney-raise opened this issue Oct 25, 2016 · 7 comments
Labels
2.status: work-in-progress 3.skill: good-first-bug 9.needs: clean-up 9.needs: package (update)

Comments

@pmahoney-raise
Copy link

Issue description

There is a conflict installing groovy and openjdk in the same nix profile. The LICENSE file exists at the toplevel; it should probably be located elsewhere for both packages.

Steps to reproduce

$ nix-env -iA nixpkgs.groovy nixpkgs.openjdk
...
collision between ‘/nix/store/mjd2gsnx14q6n05f8fwjk0ry6kms0m40-groovy-2.4.6/LICENSE’ and ‘/nix/store/g26vd7kl13vsfpghs9pks4avrwl8drng-openjdk-7u60b30/LICENSE’; use ‘nix-env --set-flag priority NUMBER PKGNAME’ to change the priority of one of the conflicting packages

Technical details

  • System: OS X 10.10.5 Yosemite
  • Nix version: (run nix-env --version) nix-env (Nix) 1.11.4
  • Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion) 17.03pre94294.a057dfc
@Mic92
Copy link
Member

Mic92 commented Oct 25, 2016

I think you could make a pull request, where you delete LICENSE in both package, within postInstall.

@NeQuissimus
Copy link
Member

May be related to what I am seeing with bind in #19761

@sternenseemann
Copy link
Member

Wouldn't it be more sensible to move it to, say, ${out}/share/<pkgname>/LICENSE in postInstall?

@vcunat
Copy link
Member

vcunat commented Oct 29, 2016

Yes, this stuff should not be installed in top level.

@sternenseemann
Copy link
Member

sternenseemann commented Oct 29, 2016 •
edited

Okay, I will fix that. (Can someone assign me?)

@NeQuissimus
Copy link
Member

I think only people with commit access can be assigned. But it's yours :)

@sternenseemann sternenseemann mentioned this issue Oct 30, 2016
Merged
7 tasks
sternenseemann added a commit to sternenseemann/nixpkgs that referenced this issue Oct 31, 2016
@sternenseemann
groovy: no easily conflicting files in top-level (NixOS#19872)
ce90d09
NeQuissimus added a commit that referenced this issue Nov 1, 2016
@NeQuissimus
Merge pull request #19984 from sternenseemann/license-clash
79424a0 
groovy: no easily conflicting files in top-level (#19872)
@calvertvl
Copy link
Contributor

@pSub: It looks like this can be closed, unless you want to cherry pick to 16.09

@pSub pSub closed this as completed Mar 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
2.status: work-in-progress 3.skill: good-first-bug 9.needs: clean-up 9.needs: package (update)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@pSub @Mic92 @NeQuissimus @vcunat @sternenseemann @calvertvl @pmahoney-raise