Age | Commit message (Collapse) | Author | Files | Lines | |
---|---|---|---|---|---|
2017-11-25 | build-remote: Fix missing log output | aszlig | 1 | -1/+4 | |
The storeUri variable in the build-remote hook is declared very much to the start of the main function and a bunch of lines later, the same variable gets checked via hasPrefix() but it gets assigned *after* that check when the most suitable machine for the build was choosen. So I guess this was just a typo in d16fd2497374671c92cb877f9570d65783a7 and what we really want is to either checkd the prefix *after* assigning storeUri or use bestMachine->storeUri directly. I choose the latter, because the former could introduce even more regressions if the try block where the variable gets assigned terminates early. Nevertheless, the reason why the log output didn't work is because hasPrefix() checked for "ssh://" in front of storeUri, but if the storeUri isn't set correctly (or at all), we don't get the log file descriptor set up properly, leading to no log output. I've adjusted the remote-builds test to include a regression test for this, so that we can make sure we get a build output when using remote builds. In addition to that I've tested this with two of my build farms and the build logs are emitted correctly again. Signed-off-by: aszlig <aszlig@nix.build> | |||||
2017-05-01 | Remove $NIX_BUILD_HOOK and $NIX_CURRENT_LOAD | Eelco Dolstra | 1 | -1/+0 | |
This is to simplify remote build configuration. These environment variables predate nix.conf. The build hook now has a sensible default (namely build-remote). The current load is kept in the Nix state directory now. | |||||
2017-02-21 | useChroot -> useSandbox | Eelco Dolstra | 1 | -1/+1 | |
2016-11-10 | build-remote: Implement in C++ | Shea Levy | 1 | -0/+1 | |
2015-02-16 | Test chroot building | Eelco Dolstra | 1 | -0/+1 | |
2014-12-11 | Urgh | Eelco Dolstra | 1 | -1/+1 | |
2014-12-05 | Shut up a warning | Eelco Dolstra | 1 | -1/+1 | |
2014-09-05 | tests/remote-builds.nix: Time out faster | Eelco Dolstra | 1 | -2/+3 | |
2014-07-24 | tests/remote-builds.nix: Test failing build | Eelco Dolstra | 1 | -1/+4 | |
2014-07-24 | tests/remote-builds.nix: Don't try cache.nixos.org | Eelco Dolstra | 1 | -0/+1 | |
2014-07-14 | build-remote.pl: Fix building multiple output derivations | Eelco Dolstra | 1 | -2/+3 | |
We were importing paths without sorting them topologically, leading to "path is not valid" errors. See e.g. http://hydra.nixos.org/build/12451761 | |||||
2014-04-15 | Fix test evaluation | Eelco Dolstra | 1 | -2/+2 | |
2013-10-11 | Adjust to the NixOS/Nixpkgs merge | Eelco Dolstra | 1 | -1/+1 | |
2013-03-08 | Revert "Prevent config.h from being clobbered" | Eelco Dolstra | 1 | -2/+2 | |
This reverts commit 28bba8c44f484eae38e8a15dcec73cfa999156f6. | |||||
2013-03-07 | Prevent config.h from being clobbered | Eelco Dolstra | 1 | -2/+2 | |
2013-01-21 | Fix the VM tests | Eelco Dolstra | 1 | -2/+2 | |
2012-03-12 | Fix tests | Eelco Dolstra | 1 | -2/+2 | |
2011-11-25 | * Fix race. | Eelco Dolstra | 1 | -0/+1 | |
2011-10-11 | * Move the remote building test from the NixOS tree to the Nix tree. | Eelco Dolstra | 1 | -0/+97 | |