Skip to content

Commit

Permalink
Back out one of the changes in #26178 to get warnings/warnings.chpl w…
Browse files Browse the repository at this point in the history
…orking (#26180)

[reviewed by @DanilaFe ]

While paratesting my branch tonight, I found that

  chplenv/chplconfig/warnings/warnings.chpl

was failing due to PR #26178. Though I'm not that familiar with this
logic, it seems that reverting half of the changes in it keeps this test
working as expected, though it may suggest we need to do more to both
achieve the PR's aim and also keep errors for cases like this.
  • Loading branch information
bradcray authored Oct 31, 2024
2 parents 07d6adc + 308ca8b commit 56a218a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion util/chplenv/overrides.py
Original file line number Diff line number Diff line change
Expand Up @@ -198,7 +198,7 @@ def skip_line(self, line, linenum):

# Check for syntax errors
try:
var, val = [f.strip() for f in line.split('=', maxsplit=1)]
var, val = [f.strip() for f in line.split('=')]
except ValueError:
self.warnings.append(
(
Expand Down

0 comments on commit 56a218a

Please sign in to comment.