[Bug 2168858] Review Request: python-rapidfuzz - Rapid fuzzy string matching in Python and C++ using the Levenshtein Distance

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #9 from Miro Hrončok <mhroncok@xxxxxxxxxx> ---
> (In reply to Miro Hrončok from comment #3)
> > (In reply to Karolina Surma from comment #1)
> > > Starting with question marks as I'm not familiar with arch packages. 
> > > 
> > > Fedora Review points out this one: 
> > > 
> > > - Header files in -devel subpackage, if present.
> > >   Note: python3-rapidfuzz : /usr/lib64/python3.11/site-
> > >   packages/rapidfuzz/rapidfuzz.h
> > >   See: https://docs.fedoraproject.org/en-US/packaging-
> > >   guidelines/#_devel_packages
> > 
> > Is this file required for the package to function? If not, it should
> > probably not be installed (that should be fixable upstream). If it is, it
> > needs to stay.
> > 
>  I believe it needs to stay. I've tried to remove it, but the tests failed
> with ModuleNotFoundError: No module named 'rapidfuzz.distance.metrics_cpp'.

How did you try to remove it? (If you deleted it before building the extension
module, the ModuleNotFoundError is quite expected.)


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2168858
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
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/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux