Skip to content

Commit

Permalink
reftest: fix sed-cmd regexp to handle escaped '\'
Browse files Browse the repository at this point in the history
  • Loading branch information
rjbou committed Sep 12, 2023
1 parent 98e8eb9 commit 92b2bba
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion master_changes.md
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ users)

### Engine
* With real path resolved for all opam temp dir, remove `/private` from mac temp dir regexp [#5654 @rjbou]
* Reimplement `sed-cmd` command regexp, to handle prefixed commands with path not only in subprocess, but anywere in output [#5657 @rjbou]
* Reimplement `sed-cmd` command regexp, to handle prefixed commands with path not only in subprocess, but anywere in output [#5657 #5607 @rjbou]

## Github Actions
* Add coreutils install for cheksum validation tests [#5560 @rjbou]
Expand Down
5 changes: 3 additions & 2 deletions tests/reftests/run.ml
Original file line number Diff line number Diff line change
Expand Up @@ -468,8 +468,9 @@ module Parse = struct
*)
opt_quoted @@ [
alpha; char ':';
rep1 @@ seq [ char '\\'; rep1 @@ diff any (with_quote_set "\\") ];
char '\\';
rep1 @@ seq [ char '\\'; opt @@ char '\\';
rep1 @@ diff any (with_quote_set "\\") ];
char '\\'; opt @@ char '\\';
Re.str cmd;
opt @@ Re.str ".exe";
] in
Expand Down

0 comments on commit 92b2bba

Please sign in to comment.