Open Bug 926464 Opened 6 years ago Updated 2 years ago
Manifest parser should recognize capitalized constants: skip-if = True doesn't work, skip-if = true does
As per summary. Especially because the moz.build documentation says you can use the constants "True", "False" and "None", and the test manifest documentation on MDN doesn't say anything, so I assumed I should use the capitalized versions and just wasted a couple of hours of try machine time and my own sanity trying to figure out what was up. </rant>
Not sure if we should actually support it, but we should error out fast
I'd support erroring better here and fixing the docs. I did document the expression syntax in the manifestparser docs: http://mozbase.readthedocs.org/en/latest/manifestdestiny.html#manifest-conditional-expressions
Mass closing bugs with no activity in 2+ years. If this bug is important to you, please re-open.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
This is still a problem which needs to be fixed. The mentioned docs can now be found at: https://firefox-source-docs.mozilla.org/mozbase/manifestparser.html#manifest-conditional-expressions
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
You need to log in before you can comment on or make changes to this bug.