Commit d158eb0
committed
Remove hash_options from requirement to avoid packvers parsing issues
The compatibility check is already ignoring the `hash_options` [1],
while a `Requirement` is still constructed in [2] and [3].
This causes the application to crash and fail to create a result.
Remove the `hash_options` for now until
`packvers.requirements.Requirement` supports this option.
[1]: https://github.com/aboutcode-org/python-inspector/blob/3ad3925840e8dad16ad12b3917f04ec3172590ad/src/_packagedcode/pypi.py#L999
[2]: https://github.com/aboutcode-org/python-inspector/blob/3ad3925840e8dad16ad12b3917f04ec3172590ad/src/python_inspector/api.py#L484
[3]: https://github.com/aboutcode-org/python-inspector/blob/3ad3925840e8dad16ad12b3917f04ec3172590ad/src/python_inspector/resolution.py#L259
Related to #243.
Signed-off-by: Marcel Bochtler <[email protected]>1 parent ff07d0e commit d158eb0
File tree
4 files changed
+563
-2
lines changed- src/_packagedcode
- tests
- data
4 files changed
+563
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
534 | 534 | | |
535 | 535 | | |
536 | 536 | | |
537 | | - | |
| 537 | + | |
538 | 538 | | |
539 | 539 | | |
540 | 540 | | |
| |||
945 | 945 | | |
946 | 946 | | |
947 | 947 | | |
| 948 | + | |
| 949 | + | |
948 | 950 | | |
949 | 951 | | |
950 | 952 | | |
| |||
973 | 975 | | |
974 | 976 | | |
975 | 977 | | |
976 | | - | |
| 978 | + | |
977 | 979 | | |
978 | 980 | | |
979 | 981 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
0 commit comments