Skip to content

Commit

Permalink
Back out one of the changes in chapel-lang#26178 to get warnings/warn…
Browse files Browse the repository at this point in the history
…ings.chpl working

While paratesting my branch tonight, I found that

  chplenv/chplconfig/warnings/warnings.chpl

was failing due to PR chapel-lang#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.

---
Signed-off-by: Brad Chamberlain <[email protected]>
  • Loading branch information
bradcray committed Oct 31, 2024
1 parent 07d6adc commit 308ca8b
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 308ca8b

Please sign in to comment.