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

charliecloud: init at 0.2.2 #34484

Closed
wants to merge 1 commit into from
Closed

charliecloud: init at 0.2.2 #34484

wants to merge 1 commit into from

Conversation

bzizou
Copy link
Contributor

@bzizou bzizou commented Feb 1, 2018

Motivation for this change

Want to use charliecloud :-)

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
    • 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 nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

bats_version = "0.4.0";

srcs =
[ (fetchurl {
Copy link
Member

Choose a reason for hiding this comment

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

Please use fetchFromGithub

sha256 = "0djq3j6q7zj4ignf1b87k64bg4h5aw8l0mxnshagsaskwvmhakpi";
})
(fetchurl {
url = "https://github.com/sstephenson/bats/archive/v${bats_version}.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.

bats is already available in nixpkgs. Would it be possible to use?


patches = [ ./CONDUCT.md_not_present_into_bats_release.patch ];

buildInputs = [ bash python ];
Copy link
Member

Choose a reason for hiding this comment

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

I guess bash is provided by stdenv so, it is not necessary to include it.

@@ -14479,6 +14479,8 @@ with pkgs;
amdappsdk = amdappsdk28;
};

charliecloud = callPackage ../applications/virtualization/charliecloud { };

Copy link
Member

Choose a reason for hiding this comment

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

Too much spaces

@pSub
Copy link
Member

pSub commented Jul 13, 2018

@bzizou Are you still interested in this? If so, please fix the mentioned issues.

@bzizou
Copy link
Contributor Author

bzizou commented Jul 16, 2018

@pSub Yes, sorry! I check that asap...

@bzizou
Copy link
Contributor Author

bzizou commented Apr 3, 2019

Charliecloud has a lot of new versions and changes. I'll do another PR with the latest.

@bzizou bzizou closed this Apr 3, 2019
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

5 participants