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

junkie: init at 2.8.0 #57510

Merged
merged 2 commits into from Mar 15, 2019

Conversation

Projects
None yet
3 participants
@rixed
Copy link
Contributor

rixed commented Mar 12, 2019

Motivation for this change

Addition of a new networking tool: a programmable sniffer/DPI

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option 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/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

Show resolved Hide resolved pkgs/tools/networking/junkie/default.nix Outdated
Show resolved Hide resolved pkgs/tools/networking/junkie/default.nix
Show resolved Hide resolved pkgs/tools/networking/junkie/default.nix Outdated
Show resolved Hide resolved pkgs/tools/networking/junkie/default.nix Outdated

@rixed rixed force-pushed the rixed:junkie branch from 1207f52 to 1881bd8 Mar 15, 2019

@rixed

This comment has been minimized.

Copy link
Contributor Author

rixed commented Mar 15, 2019

Thank you so much for all the comments. That was my first attempt at packaging anything for Nix since I tested Nix about 7 or 8 years ago :)

@Infinisil

This comment has been minimized.

Copy link
Contributor

Infinisil commented Mar 15, 2019

Nice :D

Only thing I could complain about now is that the commits should be switched around, adding yourself to maintainers should come first because otherwise the package commit would reference your name which isn't present at that point yet

@rixed

This comment has been minimized.

Copy link
Contributor Author

rixed commented Mar 15, 2019

Only thing I could complain about now is that the commits should be switched around, adding yourself to maintainers should come first

Yes, I think that's how it is, at least according to my git log. Also according to github commit tab, assuming it displays the commit in the same order than git log. Am I missing something?

@Infinisil

This comment has been minimized.

Copy link
Contributor

Infinisil commented Mar 15, 2019

Ohh damn, it's GitHub's weird behavior where it doesn't display the commits in the right order sometimes when force pushing, see https://help.github.com/en/articles/why-are-my-commits-in-the-wrong-order

Alright everything good then :)

@rixed

This comment has been minimized.

Copy link
Contributor Author

rixed commented Mar 15, 2019

Oh I see. I was unaware of this behaviour.

@Infinisil Infinisil merged commit b81e42e into NixOS:master Mar 15, 2019

10 checks passed

grahamcofborg-eval ^.^!
Details
grahamcofborg-eval-check-maintainers matching changed paths to changed attrs...
Details
grahamcofborg-eval-check-meta config.nix: checkMeta = true
Details
grahamcofborg-eval-nixos-manual nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="ofborg"; } ./nixos/release.nix -A manual
Details
grahamcofborg-eval-nixos-options nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="ofborg"; } ./nixos/release.nix -A options
Details
grahamcofborg-eval-nixpkgs-manual nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="ofborg"; } ./pkgs/top-level/release.nix -A manual
Details
grahamcofborg-eval-nixpkgs-tarball nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="ofborg"; } ./pkgs/top-level/release.nix -A tarball
Details
grahamcofborg-eval-nixpkgs-unstable-jobset nix-instantiate --arg nixpkgs { outPath=./.; revCount=999999; shortRev="ofborg"; } ./pkgs/top-level/release.nix -A unstable
Details
grahamcofborg-eval-package-list nix-env -qa --json --file .
Details
grahamcofborg-eval-package-list-no-aliases nix-env -qa --json --file . --arg config { allowAliases = false; }
Details

@rixed rixed deleted the rixed:junkie branch Mar 15, 2019

@Infinisil

This comment has been minimized.

Copy link
Contributor

Infinisil commented Mar 15, 2019

Does not build..

these derivations will be built:
  /nix/store/7p2h5w4glas3qx62grf80rd0svjmxqdg-source.drv
  /nix/store/1fqiw4zl8wm2rv4z99h61jj3v5qhmf19-junkie-2.8.0.drv
building '/nix/store/7p2h5w4glas3qx62grf80rd0svjmxqdg-source.drv'...

trying https://github.com/rixed/junkie/archive/v2.8.0.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   120    0   120    0     0    217      0 --:--:-- --:--:-- --:--:--   217
100 1233k    0 1233k    0     0   548k      0 --:--:--  0:00:02 --:--:--  997k
unpacking source archive /build/v2.8.0.tar.gz
hash mismatch in fixed-output derivation '/nix/store/rlyg71knykygmb5w3x4vvirl8x1jy8ik-source':
  wanted: sha256:16729cfxzvcq07vksrhgp89n7xh0z8rmyxwbm3k90kvdw3b7pdf9
  got:    sha256:0kfdjgch667gfb3qpiadd2dj3fxc7r19nr620gffb1ahca02wq31
cannot build derivation '/nix/store/1fqiw4zl8wm2rv4z99h61jj3v5qhmf19-junkie-2.8.0.drv': 1 dependencies couldn't be built
error: build of '/nix/store/1fqiw4zl8wm2rv4z99h61jj3v5qhmf19-junkie-2.8.0.drv' failed
@rixed

This comment has been minimized.

Copy link
Contributor Author

rixed commented Mar 16, 2019

OMG. Fixed in #57737

I would swear it used to work but now of course I failed to repro it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.