Bug 1671697 - pyxattr is missing provides python3.7dist(xattr)
Summary: pyxattr is missing provides python3.7dist(xattr)
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: pyxattr
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Marcin Zajaczkowski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-02-01 11:01 UTC by Alfredo Moralejo
Modified: 2019-02-02 23:22 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2019-02-02 23:22:59 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Alfredo Moralejo 2019-02-01 11:01:37 UTC
Description of problem:

After enabling automatic dependencies generation in Fedora, packages which import xattr are adding:

Requires: python3.7dist(xattr)

However, pyxattr is providing

python3.7dist(pyxattr)

So packages which depend on it fail to install.

I'm not sure what's the best to manually add that provides as i don't find a macro for python3.7dist only for python3dist

Version-Release number of selected component (if applicable):

pyxattr-0.5.6-2.fc29

Comment 1 Miro Hrončok 2019-02-01 11:15:33 UTC
See:

 https://pypi.org/project/pyxattr/
 https://pypi.org/project/xattr/

Those are 2 things, two different Python packages.

RPMs that have "Requires: python3.7dist(xattr)" actually have upstream information on depending on xattr and pip would install xattr.

Our pyxattr package is an equivalent of the pyxattr PyPI package.


It seems that both packages are imported as xattr. I don't know about their compatibility with each other.

Comment 2 Miro Hrončok 2019-02-01 11:17:53 UTC
The automatic provides/requires are based on package names, not module names. A package name does not need to == the module name. For example djangorestframework package has the rest_framework module.

Comment 3 Miro Hrončok 2019-02-02 23:22:59 UTC
Anyway, nothing wrong with pyxattr here.

If your upstreams depend on pyxattr and not xattr, fix them. Don't disable the generator [1] please, but rather patch setup.py or the other upstream file that setup.py uses to fetch the dependency information from (I guess your setup.pys usually just use pbr to do "magic").


If you actually need xattr and not pyxattr, you'll need to package that, as it is not packaged in Fedora yet. You would most likely get conflicting files with pyxattr, that's unfortunate.

Maybe pyxattr can be dropped once xattr is packaged, I'm not sure. See [2] for compatibility.


[1] https://src.fedoraproject.org/rpms/python-pifpaf/c/e833ec140755d6b4ab5818c873d651fae22b4c8f?branch=master
[2] https://github.com/xattr/xattr/blob/master/xattr/pyxattr_compat.py


Note You need to log in before you can comment on or make changes to this bug.