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

cde: init at 0.1 #24225

Merged
merged 2 commits into from Mar 25, 2017
Merged

cde: init at 0.1 #24225

merged 2 commits into from Mar 25, 2017

Conversation

rlupton20
Copy link
Contributor

Motivation for this change

CDE was an attempt to capture a binary and all it's dependencies, to create a portable package - it works by monitoring system calls and grabbing any files that are mentioned. It's quite an old package and doesn't seem to have moved much recently, but it still works well.

It works particularly well with nix, because you can use it along with nix-shell to create standalone tarballs for software that isn't available on old systems (which is more or less how I can across this - just installing nix wasn't an option).

Allows a mini nix to be transported to other systems in a neat way.

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.

stdenv.mkDerivation {
name = "cde-0.1";
src = fetchgit {
url = "https://github.com/pgbovine/CDE.git";
Copy link
Contributor

Choose a reason for hiding this comment

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

Use fetchFromGitHub

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for fixing - was planning on doing it tomorrow evening but you beat me to it 😄.

@ndowens
Copy link
Contributor

ndowens commented Mar 23, 2017

😀

@ndowens ndowens merged commit 5f61654 into NixOS:master Mar 25, 2017
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

2 participants