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

pythonPackages.lit: init at 0.5 #23643

Closed
wants to merge 1 commit into from
Closed

Conversation

dtzWill
Copy link
Member

@dtzWill dtzWill commented Mar 8, 2017

Motivation for this change

'lit' is handy for running test-suites of various LLVM projects.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@mention-bot
Copy link

@dtzWill, thanks for your PR! By analyzing the history of the files in this pull request, we identified @FRidh to be a potential reviewer.

@@ -8242,6 +8242,25 @@ in {
};
};

lit = buildPythonApplication rec {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

applications do not belong here

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, my mistake. Will fix shortly.

disabled = isPy3k;

src = pkgs.fetchurl {
url = "https://files.pythonhosted.org/packages/5b/a0/dbed2c8dfb220eb9a5a893257223cd0ff791c0fbc34ce2f1a957fa4b6c6f/lit-0.5.0.tar.gz";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use fetchPypi

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure how to make fetchPypi work here--all the variants AFAICT give a 404 :/. I'm pushing what I have to demonstrate. Not sure why this package would be different than others?

@FRidh
Copy link
Member

FRidh commented Mar 15, 2017

I've pushed c766960.

@FRidh FRidh closed this Mar 15, 2017
@dtzWill
Copy link
Member Author

dtzWill commented Mar 15, 2017

@FRidh great, thank you!

It's curious that it no longer 404's!

.. after some investigation it appears this is because the version needed to be "0.5.0" not simply "0.5". D'oh.

Thanks for taking care of this!

@dtzWill dtzWill deleted the feature/lit branch March 15, 2017 12:27
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

3 participants