search

madjar / nox

Tools to make nix nicer to use
MIT License
305 stars 35 forks source link

Nothing changed #37

Open davidak opened 8 years ago

davidak commented 8 years ago 
$ nox-review pr 12491
=== Reviewing PR 12491 : [WIP] rarcrack: init at 0.2
==> Creating nixpkgs repo in /home/davidak/.nox/nixpkgs
==> Fetching base (master)
==> Fetching PR
==> Fetching extra history for merging
==> Merging PR into base
Auto-merging pkgs/top-level/all-packages.nix
Nothing changed

i have updated that commit several times but there are changes (like you see on github) https://github.com/NixOS/nixpkgs/pull/12491

reproducible on darwin and linux https://travis-ci.org/NixOS/nixpkgs/builds/104322996

grahamc commented 8 years ago

Another as seen here: https://github.com/NixOS/nixpkgs/pull/13444 / https://travis-ci.org/NixOS/nixpkgs/builds/111716384

Mathnerd314 commented 8 years ago

Well, sauce-connect didn't rebuild because it was a fixed-output derivation and the hash didn't change. The first I'm not sure about.

davidak commented 8 years ago

And again with https://github.com/NixOS/nixpkgs/pull/16159

$ nix-channel --update
downloading Nix expressions from ‘http://nixos.org/channels/nixpkgs-unstable//nixexprs.tar.xz’...
downloading ‘http://nixos.org/channels/nixpkgs-unstable//nixexprs.tar.xz’... [6420/7423 KiB, 3169.1 KiB/s]
unpacking channels...
$ nix-shell -p nox --run "nox-review pr 16159"
=== Reviewing PR 16159 : Neovim darwin fixes
==> Fetching base (master)
==> Fetching PR
==> Fetching extra history for merging
==> Merging PR into base
Nothing changed