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

cliapp: 1.20140719 -> 1.20150305 #10717

Merged
merged 1 commit into from
Oct 30, 2015
Merged

cliapp: 1.20140719 -> 1.20150305 #10717

merged 1 commit into from
Oct 30, 2015

Conversation

richardipsum
Copy link
Contributor

This updates cliapp to the latest stable release,
and also switches to fetching the source from git, instead of from a tarball.

This has been tested by running nix-shell on a python derivation
that includes cliapp as a build input, running python within the shell,
importing cliapp and checking that cliapp.__version__ has the expected value.

It has also been tested by building and running obnam (which runtime depends on cliapp)
with nix-shell -I "nixpkgs=$HOME/repos/nixpkgs" --pure -p obnam and then using obnam in the resulting shell to perform a backup.

Hope this helps,
Richard Ipsum

@jagajaga
Copy link
Member

Please properly name commit's name following our guide in CONTRIBUTING.md.

Also fetch from git rather than from tarball
@richardipsum richardipsum changed the title Update cliapp to latest release cliapp: 1.20140719 -> 1.20150305 Oct 30, 2015
@richardipsum
Copy link
Contributor Author

Sorry, now updated as requested.

jagajaga added a commit that referenced this pull request Oct 30, 2015
@jagajaga jagajaga merged commit 21e2d0b into NixOS:master Oct 30, 2015
@Janik-Haag Janik-Haag added the 12. first-time contribution This PR is the author's first one; please be gentle! label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
12. first-time contribution This PR is the author's first one; please be gentle!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants