Hi Martin,

I looked at the underlying code in fedora-review and oh my,
what a beautiful mess. It parses output from RPM line-by-line and splits
the lines on spaces - which works fine until you run across a package
where the files have spaces in their names and then the "take nth field"
logic breaks completely.

I cooked up a more resilient patch here:
https://forge.fedoraproject.org/packaging/FedoraReview/pulls/563

Cheers,
Artur FI
-- 
_______________________________________________
devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]
Do not reply to spam, report it: 
https://forge.fedoraproject.org/infra/tickets/issues/new

Reply via email to