https://bugzilla.redhat.com/show_bug.cgi?id=2388154



--- Comment #6 from Gordon Messmer <[email protected]> ---
> there are three instances of the command "cd ../../" and all of them need to 
> be removed

N/M... mistake on my part.

Package seems mostly good, other than:

Needs to update to the current version.

Current version has more tests that require network access and must be
disabled. The patch in my git repo addresses this, and has been offered
upstream.

It would be nice to add a bcond to optionally enable network tests.

Dependency on pyo3-async-runtimes0.25, and I'm waiting on creation of repos for
that package.

There is one use of %define that should instead be %global (or should include a
justification)


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2388154

Report this comment as SPAM: 
https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202388154%23c6

-- 
_______________________________________________
package-review 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://pagure.io/fedora-infrastructure/new_issue

Reply via email to