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

Categories

(Testing :: Mozbase, defect)

defect
Not set

Tracking

(Not tracked)

REOPENED

People

(Reporter: Gijs, Unassigned)

Details

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.