Follow-up to #815 (fixed in #816).
- The shorthand key list includes
extra. pipenv's PipenvMarkers has 11 keys and no extra, so extra = "== 'x'" should be ignored.
- A shorthand value with no operator, for example
python_version = "3.10", is emitted as the marker text python_version 3.10, which is not valid marker syntax. pipenv builds <key> <value>, and when the combined marker does not parse it drops the whole marker, so the requirement stays unconditional.
Source: https://github.com/pypa/pipenv/blob/c1739c9b/pipenv/utils/markers.py (PipenvMarkers.from_pipfile).
Fix: drop extra from the list and skip the marker when a shorthand value has no operator. One regression covers both.
Follow-up to #815 (fixed in #816).
extra. pipenv'sPipenvMarkershas 11 keys and noextra, soextra = "== 'x'"should be ignored.python_version = "3.10", is emitted as the marker textpython_version 3.10, which is not valid marker syntax. pipenv builds<key> <value>, and when the combined marker does not parse it drops the whole marker, so the requirement stays unconditional.Source: https://github.com/pypa/pipenv/blob/c1739c9b/pipenv/utils/markers.py (
PipenvMarkers.from_pipfile).Fix: drop
extrafrom the list and skip the marker when a shorthand value has no operator. One regression covers both.