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

Benson Muite <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |POST
              Flags|fedora-review?              |fedora-review+



--- Comment #11 from Benson Muite <[email protected]> ---
(In reply to Felix Wang from comment #8)
> Thanks for reviewing.
> 
> > a) Consider packaging the documentation, for an example see:
> > https://src.fedoraproject.org/rpms/python-androguard/blob/rawhide/f/python-androguard.spec
> 
> Added doc.

Thanks. 
> 
> > b) To ensure all directories are owned, please add:
> > Requires: python3-libs
> 
> I think it does not need to add this, see aother reviewer's comment about
> this, https://bugzilla.redhat.com/show_bug.cgi?id=2418755#c2


$ dnf repoquery whatowns /usr/lib/python3.14
Updating and loading repositories:
 Fedora 43 - x86_64                                                            
            100% |   1.4 MiB/s |  58.5 MiB |  00m42s
 Fedora 43 - x86_64 - Updates                                                  
            100% | 560.7 KiB/s |  22.5 MiB |  00m41s
Repositories loaded.
python3-libs-0:3.14.0-1.fc43.i686
python3-libs-0:3.14.0-1.fc43.x86_64
python3-libs-0:3.14.2-1.fc43.i686
python3-libs-0:3.14.2-1.fc43.x86_64

$ dnf repoquery whatowns /usr/lib/python3.14/site-packages
Updating and loading repositories:
Repositories loaded.
python3-bluechi-0:1.1.0-3.fc43.noarch
python3-bluechi-0:1.2.1-1.fc43.noarch
python3-libs-0:3.14.0-1.fc43.i686
python3-libs-0:3.14.0-1.fc43.x86_64
python3-libs-0:3.14.2-1.fc43.i686
python3-libs-0:3.14.2-1.fc43.x86_64

Assume python(abi) is python3-libs - the runtime libraries
https://src.fedoraproject.org/rpms/python3.14/blob/rawhide/f/python3.14.spec#_533

Not sure if it is a bug in the review tool, or packaging. There is a bug
reported
for fedora-review:

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

Not a blocker, but possibly helpful to add with a link to the ticket.

> 
> > c) Please fix the spelling error by changing
> 
> Fixed spelling.

Thanks.

> 
> ---
> 
> Spec URL: https://topazus.fedorapeople.org/python-uncertainties.spec
> SRPM URL:
> https://topazus.fedorapeople.org/python-uncertainties-3.2.3-1.fc44.src.rpm


Approved.

Can you make me co-maintainer of:
https://src.fedoraproject.org/rpms/groonga
https://src.fedoraproject.org/rpms/qxmpp
https://src.fedoraproject.org/rpms/kaidan
would like to update them to latest releases.


-- 
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=2424162

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

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