Skip to content

Commit

Permalink
38 papercuts
Browse files Browse the repository at this point in the history
  • Loading branch information
Enzime committed Oct 31, 2024
1 parent 1781245 commit 2ddbcf5
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ jobs:
skipAddingSubstituter: ${{ runner.environment == 'self-hosted' }}
- name: nix run .#add-subflakes-to-store
run: nix-shell --pure -I nixpkgs=flake:nixpkgs -p '(import ./shell.nix { }).packages.${builtins.currentSystem}.add-subflakes-to-store' --command add-subflakes-to-store
- run: nix run .#check
- run: nix run --print-build-logs .#check -- --print-build-logs

build:
timeout-minutes: 7200
Expand Down
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
- Fixed `packages'.check` failing to build on Linux
- Refactored `packages'.check` to not modify the local directory
- Fixed `packages'.check` not consistently operating on same directory
- Removed workaround for `users.users.builder.shell` being defined twice even when it's set to the same derivation
- Fixed `shell` not getting updated for users on macOS
- Removed `vscode-colorize` as it crashes the Extension Host on some repos
Expand Down
13 changes: 9 additions & 4 deletions flake.nix
Original file line number Diff line number Diff line change
Expand Up @@ -423,10 +423,15 @@
text = ''
set -x
patch < ${./files/no-ifd.diff}
PATCHED=$(nix flake metadata "''${1:-$PWD}" --json | jq -r '.path')
patch -R < ${./files/no-ifd.diff}
nix flake check --print-build-logs "$PATCHED"
nix flake check ${
pkgs.stdenvNoCC.mkDerivation {
name = "patched-self";
# WORKAROUND: `toString ./.` works on macOS but not Linux
src = toString self;
patches = [ ./files/no-ifd.diff ];
installPhase = "cp -r . $out";
}
} "$@"
'';
};

Expand Down

0 comments on commit 2ddbcf5

Please sign in to comment.