Spec URL: https://lbrabec.fedorapeople.org/python-xunitparser.spec SRPM URL: https://lbrabec.fedorapeople.org/python-xunitparser-1.3.3-1.fc24.src.rpm Description: Read JUnit/XUnit XML files and map them to Python objects Fedora Account System Username: lbrabec
This is my first package, so I need sponsor. We need this package for Taskotron [1], to parse XML output of pytest and unittest. [1] https://taskotron.fedoraproject.org/
The package looks good to me generally - pretty small/clean. Why no Python 3 though? It seems to build fine
I initially included python3 but I found that `python3 setup.py test` fails in several testcases. I decided not to include python3 until the testsuite passes without failures or errors.
*Unofficial comments - I too am waiting for a sponsor * Minor, mostly personal choice point first: you don't need the "sum" global - just include the summary in the header as usual and then use %{summary} where you need it. Though having said that, the python sample spec file does it your way. I just like keeping extra macros to a minimum. The Group tag is not needed [1] Ideally the description would be a little bit more than just the summary rpmlint is clean, which is good license looks fine [1] https://fedoraproject.org/wiki/Packaging:Guidelines#Tags_and_Sections
I updated the files: Spec URL: https://lbrabec.fedorapeople.org/python-xunitparser.spec SRPM URL: https://lbrabec.fedorapeople.org/python-xunitparser-1.3.3-1.fc24.src.rpm
In bug #1333525 it is mentioned that the python3 failures have been reported to upstream. There isn't anything about the status of the fixes, though.
Has the duplication between this and #1333525 been figured out? It looks like slightly more progress has been made on that review but it's yet to be completed
As an aside, you may want to ping ignatenkobrain on IRC (Freenode) and see about becoming a co-maintainer, as he also submitted a review for this package, and his was granted.
(In reply to John Dulaney from comment #8) > As an aside, you may want to ping ignatenkobrain on IRC (Freenode) and see > about becoming a co-maintainer, as he also submitted a review for this > package, and his was granted. That's already been taken care of. Not sure why this hasn't been closed as duplicate yet. Closing, please re-open if I misunderstood something *** This bug has been marked as a duplicate of bug 1333525 ***