-
Notifications
You must be signed in to change notification settings - Fork 72
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.
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
pv_major/pv_minor selectors type mismatch #122
Comments
csonto
pushed a commit
that referenced
this issue
Jul 31, 2023
We incorrectly marked pv_major and pv_minor fields as being of string type, even though the values were already correctly handled as integers internally. This confused -S|--select that tried to compare string values instead of integers. Reported here: #122
Thanks for the report! The report types for pv_major and pv_minor were incorrect, this is now fixed with: 0e9fe27 We don't report PV's major:minor as a pair, just the separate pv_major and pv_minor fields. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Both pv_major and pv_minor are of type string, but apparently they're not treated properly in selection - they seem to be the string composed of single character with the code equal to their value. I guess this is not how it's supposed to work:
I would expect them either to be numbers (
-S "pv_minor = 2"
) or be strings with plain values (-S "pv_minor =~ '^18$'"
).As a side note - can I match the "major:minor" pair given in that format somehow?
The text was updated successfully, but these errors were encountered: