Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Maybe provide different strictness / warning levels #598

Open
andychu opened this issue Feb 10, 2020 · 0 comments
Open

Maybe provide different strictness / warning levels #598

andychu opened this issue Feb 10, 2020 · 0 comments
Labels
strict

Comments

@andychu
Copy link
Contributor

andychu commented Feb 10, 2020

shopt -s strict_echo seems likely "break too much" ?

It should be:

  • on for strict:all
  • OFF for oil:basic -- this is the mode where you don't want to break too much
  • on for oil:all

Likewise, strict_backslash and strict_glob might be too restrictive.

  • Right now strict_word_eval and strict_arith issue warnings, so they're inconsistent

Idea:

OIL_STRICT_WARN=3    # warn at levels 1, 2, and 3
OIL_STRICT_DIE=2    # die at levels 1 and 2

OIL_STRICT_WARN=max  # all warnings

OIL_STRICT_DIE=max  # always fail
@andychu andychu added the strict label Feb 10, 2020
andychu pushed a commit that referenced this issue Mar 21, 2020
[errors] Fix error location for arith_expr.VarRef.
017f7a9 
This code path was tickled by ble.sh.

Addresses issue #664.

Also turn off warnings since they come up too much.  #598 to provide
strictness levels could help.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
strict
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@andychu