SPRM: https://radez.fedorapeople.org/python-PuLP-1.6.1-2.fc25.src.rpm Spec: https://radez.fedorapeople.org/python-pulp.spec Description: LP modeler written in python. More info https://pypi.python.org/pypi/PuLP Fedora Account System Username: radez
Koji Build here: https://koji.fedoraproject.org/koji/taskinfo?taskID=17021666
*** Bug 1373821 has been marked as a duplicate of this bug. ***
fedora-review complains about spec/SRPM name diff: - Spec file name must match the spec package %{name}, in the format %{name}.spec. Note: python-pulp.spec should be python-PuLP.spec See: http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Spec_file_name I'd recommend to use the lowercase.
I agree with you that I would prefer the lower case. The upstream appears to use lowercase. Though, there is already a project called pulp in fedora, dnf search pulp shows a couple dozen hits. While there is not an actual package called python-pulp is seems that it would be confusing to the end user. What do you think? Should I just make it python-pulp or should I make the differentiation with python-PuLP? I'm not sure I have a strong opinion either way, which is probably why it mismatches in the spec name and package name now. For now I'll go with you're recommendation on the lower case and if you think otherwise I'll switch it back.
Actually just put both versions up there for which ever way is decided to be used: https://radez.fedorapeople.org/python-pulp.spec https://radez.fedorapeople.org/python-pulp-1.6.1-2.fc25.src.rpm https://koji.fedoraproject.org/koji/taskinfo?taskID=17172028 https://radez.fedorapeople.org/python-PuLP.spec https://radez.fedorapeople.org/python-PuLP-1.6.1-2.fc25.src.rpm https://koji.fedoraproject.org/koji/taskinfo?taskID=17172000
https://fedoraproject.org/wiki/Packaging:Naming?rd=Packaging:NamingGuidelines#Python_modules suggests "take into account the name of the module used when importing it in Python scripts" In this case, it's "import pulp" so python-pulp it is.
Great, in that case let's use these links to proceed: https://radez.fedorapeople.org/python-pulp.spec https://radez.fedorapeople.org/python-pulp-1.6.1-2.fc25.src.rpm https://koji.fedoraproject.org/koji/taskinfo?taskID=17172028
> License: ASL 2.0 is not correct, fedora-review detected the license as MIT/X11 (BSD like) Here is the full licensecheck.txt: MIT/X11 (BSD like) ------------------ PuLP-1.6.1/LICENSE PuLP-1.6.1/PKG-INFO PuLP-1.6.1/src/pulp/__init__.py PuLP-1.6.1/src/pulp/amply.py PuLP-1.6.1/src/pulp/constants.py PuLP-1.6.1/src/pulp/pulp.py PuLP-1.6.1/src/pulp/solvers.py PuLP-1.6.1/src/pulp/sparse.py Unknown or generated -------------------- PuLP-1.6.1/AUTHORS PuLP-1.6.1/HISTORY PuLP-1.6.1/INSTALL PuLP-1.6.1/MANIFEST.in PuLP-1.6.1/README.md PuLP-1.6.1/docs.cfg PuLP-1.6.1/examples/AmericanSteelProblem.py PuLP-1.6.1/examples/BeerDistributionProblem.py PuLP-1.6.1/examples/BeerDistributionProblem_resolve.py PuLP-1.6.1/examples/CG.py PuLP-1.6.1/examples/CGcolumnwise.py PuLP-1.6.1/examples/ComputerPlantProblem.py PuLP-1.6.1/examples/SpongeRollProblem1.py PuLP-1.6.1/examples/SpongeRollProblem2.py PuLP-1.6.1/examples/SpongeRollProblem3.py PuLP-1.6.1/examples/SpongeRollProblem4.py PuLP-1.6.1/examples/SpongeRollProblem5.py PuLP-1.6.1/examples/SpongeRollProblem6.py PuLP-1.6.1/examples/Sudoku1.py PuLP-1.6.1/examples/Sudoku2.py PuLP-1.6.1/examples/WhiskasModel1.py PuLP-1.6.1/examples/WhiskasModel2.py PuLP-1.6.1/examples/furniture.py PuLP-1.6.1/examples/test1.py PuLP-1.6.1/examples/test2.py PuLP-1.6.1/examples/test3.py PuLP-1.6.1/examples/test4.py PuLP-1.6.1/examples/test5.py PuLP-1.6.1/examples/test6.py PuLP-1.6.1/examples/test7.py PuLP-1.6.1/examples/wedding.py PuLP-1.6.1/ez_setup.py PuLP-1.6.1/setup.cfg PuLP-1.6.1/setup.py PuLP-1.6.1/src/pulp/pulp.cfg.buildout PuLP-1.6.1/src/pulp/pulp.cfg.linux PuLP-1.6.1/src/pulp/pulp.cfg.osx PuLP-1.6.1/src/pulp/pulp.cfg.win PuLP-1.6.1/src/pulp/tests.py
Updated these links: https://radez.fedorapeople.org/python-pulp.spec https://radez.fedorapeople.org/python-pulp-1.6.1-2.fc25.src.rpm license is now set to MIT/X11 (BSD like)
ok, fedora-review is not that smart "MIT/X11 (BSD like)" is not listed at https://fedoraproject.org/wiki/Licensing:Main#Software_License_List It should be simply License: MIT LICENSE text matches https://fedoraproject.org/wiki/Licensing:MIT?rd=Licensing/MIT#Modern_Style_with_sublicense
Updated these links: https://radez.fedorapeople.org/python-pulp.spec https://radez.fedorapeople.org/python-pulp-1.6.1-2.fc25.src.rpm license is now set to just MIT
One more thing, sorry for more ping-pong, naming of py2/3 version of binaries is not following https://fedoraproject.org/wiki/Packaging:Python#Naming If shipping both, they need to be named -X and -X.Y I'm not sure what is the use-case for pulptest/pulpdoctest except for developers, so we could either drop them all OR even easier, just keep py3 binaries only as in the example spec in Packaging:Python: %install %py2_install %py3_install
Oops, one more :( We also have a file conflict with python2-pulp-common subpackage of pulp mega RPM https://koji.fedoraproject.org/koji/packageinfo?packageID=21756 from project https://github.com/pulp/pulp [!]: Package does not own files or directories owned by other packages. Note: Dirs in package are owned also by: /usr/lib/python2.7/site- packages/pulp(python2-pulp-common) I guess they are to blame for taking over the namespace which they did not reserve on PyPI, https://pypi.python.org/pypi/pulp redirects to https://pypi.python.org/pypi/PuLP Not sure what we can do now, any ideas Haikel?
Well, among criteria used to untie, we look at upstream activity but both are reasonably maintained and PuLP has reserved name on PyPI. Technical solutions are: 1. rename either of the modules, and fix all software using it. Not very practical as both projects are used. 2. have python-pulp conflict python2-pulp-common (+ comment explaining why) and I'd rename the package to python-PuLP to avoid more confusion. I'd recommend option 2.
I've updated these files according to Haikel's recommendation: https://radez.fedorapeople.org/python-PuLP.spec https://radez.fedorapeople.org/python-PuLP-1.6.1-2.fc25.src.rpm
This matches 2nd case, name conflict of not compatible libs : https://fedoraproject.org/wiki/Packaging:Conflicts#Library_Name_Conflicts Guideline requires to ask one of the upstreams to rename itself, which is unlikely, so we need FPC ticket, I've opened: https://fedorahosted.org/fpc/ticket/671 and FYI bug 1414000 for current pulp RPM maintainers.
(In reply to Alan Pevec from comment #16) > This matches 2nd case, name conflict of not compatible libs : > https://fedoraproject.org/wiki/Packaging:Conflicts#Library_Name_Conflicts > > Guideline requires to ask one of the upstreams to rename itself, which is > unlikely, Even if you think it's unlikely, you should still try asking nicely. You never know if they're amenable or not unless there have been previous failed attempts.
Asked nicely in https://pulp.plan.io/issues/2531 got nice reply in https://fedorahosted.org/fpc/ticket/671#comment:4 so we are proceeding with Conflicts: solution in this review, while waiting for FPC approval.
My understanding from https://pagure.io/packaging-committee/issue/671#comment-436821 is that we can go ahead with the package, declaring an explicit Conflicts: (as it is done in the package currently on RDO). Could we move ahead with the package review? This one is blocking openstack-congress in RDO for Fedora Rawhide.
Sure thing, I'll take some time today to get the conflict put in to get it moved forward.
This is an automatic check from review-stats script. This review request ticket hasn't been updated for some time, but it seems that the review is still being working out by you. If this is right, please respond to this comment clearing the NEEDINFO flag and try to reach out the submitter to proceed with the review. If you're not interested in reviewing this ticket anymore, please clear the fedora-review flag and reset the assignee, so that a new reviewer can take this ticket. Without any reply, this request will shortly be resetted.
Dan, are you still interested to add this package to Fedora? Links to proposed spec at radez.fedorapeople.org are now 404. Honestly, I forgot what was this dependency for :)
FWIW pulpproject.org is since their 3.0.0 release publishing into non-conflicting namespaces in pypi: https://pypi.org/user/pulp/ It was also retired in Fedora https://src.fedoraproject.org/rpms/pulp/c/740d2a375360c4858ad601155f45584c8ce71fb3
I don't think we need this anymore. I don't remember what it was for either :)