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

Dave Love <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]



--- Comment #3 from Dave Love <[email protected]> ---
This either needs to run python3 explicitly or buildrequire %_bindir/python (if
that's still allowable).  Currently %check fails completely and it's not
immediately obvious, due to ||:.  Can you adjust the source not to try to run
tests which are expected to fail, or somehow count the failures, and remove the
||:?  (I remember seeing a number of failures when I've run the tests on el7
which I never investigated, which might be more-or-less specific to el7, and I
can believe running in mock could cause some.)  If that isn't possible, at
least add a comment about failures in the %check stanza.

The .so objects in the package-specific directory are fine for internal, use by
the way, regardless of what the review tool says.


-- 
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
_______________________________________________
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]

Reply via email to