Skip to content
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

Add query formats for displaying tag numbers and names #1551

Merged
merged 1 commit into from Mar 15, 2021

Conversation

pmatilai
Copy link
Member

Though I added these ages ago along with the arraysize thing, but guess
not.

Though I added these ages ago along with the arraysize thing, but guess
not.
@pmatilai pmatilai added the RFE label Feb 19, 2021
@ffesti
Copy link
Contributor

ffesti commented Feb 23, 2021

The patch looks good. But I wonder how useful the new formats really are. I have difficulty coming up with another use case other than what's in the test case. That doesn't mean I am against adding them. But the whole discussion started with packages having tags not yet supported by the system's rpm version. While this gives at least some way to show the tags in the package we should probably also add a way to query by tags number for those cases.

@pmatilai
Copy link
Member Author

Yeah these formats aren't particularly interesting on their own, I'd say they're more like building blocks towards generalizing something like the --xml format which is half built-in when it arguably should be presentable with just a funky --qf format.

The "real" use-case would be something like --qf "%{=*:tagnum} %{*}" for a straight dump of the entire header without any borken xml, but that doesn't work at the moment, due to whatever limitations in the header formatting code.

@pmatilai pmatilai merged commit 4c25053 into rpm-software-management:master Mar 15, 2021
@pmatilai pmatilai deleted the tagfmt-pr branch June 21, 2021 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants