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

kent: init at 394 #81550

Merged
merged 1 commit into from Oct 2, 2020
Merged

Conversation

scalavision
Copy link
Contributor

@scalavision scalavision commented Mar 2, 2020

Motivation for this change

Kent utilities are a very versatile and fast suite of bioinformatic tools that has been available for a long time. It has some very nice tools to optimize for visualization of big data.

Things done

I have put the mit license, however some of these tools are not allowed to be used outside non-profit organizations. My understanding of nixpkgs is that it is completely non-profit.

For lib, inc and utils the MIT wording has been used:

The licenses where there are exceptions from the mit license says this:

License hereby is granted for personal, academic and non-profit purposes

Should I choose something else than mit?

It is possible to build the utility suite without these libraries, but I am also working on adding ensembl-vep, where these libraries are required. Since there also are commercial binaries in nixpkgs, I don't think this should be a problem, but I am not a lawyer ..

I really hope these tools can be added to nixpkgs.

  • [ x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • [x ] NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • [ x] Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • [x ] Ensured that relevant documentation is up to date
  • [x ] Fits CONTRIBUTING.md.

@puzzlewolf
Copy link
Contributor

Hey, the discussion here: #31210 should help with the licensing issue. Also this situation seems to be comparable: #77519.

It seems to me that, unfortunately, the restriction made by
License hereby is granted for personal, academic and non-profit purposes actually makes those parts unfree. However, I'd really like someone more knowledgeable to weigh in here.

@scalavision
Copy link
Contributor Author

Thank for the pointers @puzzlewolf !!

I'll look into them during the weekend. I think unfree could be an option as well :-)

@GregorySchwartz
Copy link
Contributor

This code works for me in Arch linux and Ubuntu.

@ryantm
Copy link
Member

ryantm commented Sep 24, 2020

Result of nixpkgs-review pr 81550 1

1 package built:
- kent

@scalavision scalavision force-pushed the add-kent-utils-to-bioinf branch 2 times, most recently from bee7a47 to 9b3c167 Compare October 1, 2020 17:39
@scalavision
Copy link
Contributor Author

not able to see what it failed on, was it a timeout?

@ryantm
Copy link
Member

ryantm commented Oct 1, 2020

@GrahamcOfBorg eval

@scalavision
Copy link
Contributor Author

@ryantm is everything ok with this PR? I see there is one changes requested, but I am not able to find out what it is. I guess it is the change of license, and that this disappeared when I force pushed. I messed up a bit with the history, so had to fix it.

@ryantm
Copy link
Member

ryantm commented Oct 2, 2020

@scalavision I probably just didn't re-approve it. Checking now!

@ryantm ryantm merged commit 6aadae6 into NixOS:master Oct 2, 2020
@GregorySchwartz
Copy link
Contributor

kent is missing an entry in all packages so I get an error with pkgs.kent as the attribute is not found.

@ryantm ryantm mentioned this pull request Nov 6, 2020
10 tasks
@ryantm
Copy link
Member

ryantm commented Nov 6, 2020

@GregorySchwartz Oops. Fixed in #103036

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

Successfully merging this pull request may close these issues.

None yet

4 participants