Skip to content

Commit

Permalink
Make cpp assembly file extensions case sensitive again
Browse files Browse the repository at this point in the history
This fixes an issue introduced by PR bazelbuild#14005 where .s and .S
extensions were handled case-insensitive on Windows so the action
assemble was triggered instead of preprocess_assemble.
  • Loading branch information
Simon Bjorklen authored and Simon Bjorklen committed Oct 19, 2021
1 parent 1811e82 commit f37ff44
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -99,8 +99,8 @@ public ImmutableList<String> getExtensions() {

@Override
public boolean apply(String path) {
return (OS.endsWith(path, ext) && !PIC_ASSEMBLER.matches(path))
|| OS.endsWith(path, ".asm");
return (path.endsWith(ext) && !PIC_ASSEMBLER.matches(path))
|| path.endsWith(".asm");
}

@Override
Expand Down

0 comments on commit f37ff44

Please sign in to comment.