Skip to content

Commit

Permalink
Merge pull request #107715 from NixOS/revert-106172-nix-gitignore-perf
Browse files Browse the repository at this point in the history
Reverts #106172 ef3ed45

This change causes issues with negative patterns. Reverting now until those can be resolved.
  • Loading branch information
kevincox committed Jan 3, 2021
2 parents 3f67972 + 11e522c commit bfa497b
Showing 1 changed file with 12 additions and 14 deletions.
26 changes: 12 additions & 14 deletions pkgs/build-support/nix-gitignore/default.nix
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,14 @@ let
in rec {
# [["good/relative/source/file" true] ["bad.tmpfile" false]] -> root -> path
filterPattern = patterns: root:
let
filters = map (pair: relPath: if match (head pair) relPath == null then true else last pair) patterns;
in
name: _type:
let
relPath = lib.removePrefix ((toString root) + "/") name;
in foldl' (acc: f: if acc == true then f relPath else acc) true filters;
(name: _type:
let
relPath = lib.removePrefix ((toString root) + "/") name;
matches = pair: (match (head pair) relPath) != null;
matched = map (pair: [(matches pair) (last pair)]) patterns;
in
last (last ([[true true]] ++ (filter head matched)))
);

# string -> [[regex bool]]
gitignoreToPatterns = gitignore:
Expand Down Expand Up @@ -90,9 +91,7 @@ in rec {
(filter (l: !isList l && !isComment l)
(split "\n" gitignore));

gitignoreFilter = ign: let
patterns = gitignoreToPatterns ign;
in root: filterPattern patterns root;
gitignoreFilter = ign: root: filterPattern (gitignoreToPatterns ign) root;

# string|[string|file] (→ [string|file] → [string]) -> string
gitignoreCompileIgnore = file_str_patterns: root:
Expand All @@ -101,10 +100,9 @@ in rec {
str_patterns = map (onPath readFile) (lib.toList file_str_patterns);
in concatStringsSep "\n" str_patterns;

gitignoreFilterPure = filter: patterns: root: let
compiledFilter = gitignoreCompileIgnore patterns root;
filterFn = gitignoreFilter compiledFilter;
in name: type: filterFn root name type && filter name type;
gitignoreFilterPure = filter: patterns: root: name: type:
gitignoreFilter (gitignoreCompileIgnore patterns root) root name type
&& filter name type;

# This is a very hacky way of programming this!
# A better way would be to reuse existing filtering by making multiple gitignore functions per each root.
Expand Down

0 comments on commit bfa497b

Please sign in to comment.