Bug 1713856 - pyproj needs proj to upgrade to version 6.x before it can be upgraded.
Summary: pyproj needs proj to upgrade to version 6.x before it can be upgraded.
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: proj
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Devrim Gündüz
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1686878
TreeView+ depends on / blocked
 
Reported: 2019-05-25 08:45 UTC by Jos de Kloe
Modified: 2019-11-10 11:49 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-11-10 11:49:07 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Jos de Kloe 2019-05-25 08:45:08 UTC
Description of problem:

upstream proj has released new versions 6.0.0 and 6.1.0, and looking at the 6.0.0 release notes a lot has changed since the 5.x series.

pyproj upstream is following these changes and can no longer be build using proj 5.x, so currently I cannot update this package anymore.

This bug is just to note the current situation, and to help me track when proj gets upgraded to the new version.

Comment 1 Ben Cotton 2019-08-13 17:08:25 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 31 development cycle.
Changing version to '31'.

Comment 2 Ben Cotton 2019-08-13 19:18:17 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 31 development cycle.
Changing version to 31.

Comment 3 Jos de Kloe 2019-08-14 07:43:25 UTC
still waiting for news here. If the maintainer could add a note on what the plans are that would be appreciated.

Comment 4 Orion Poplawski 2019-08-16 03:19:46 UTC
Looks like libgeotiff 1.5 requires proj 6 as well.  Devrim, are you still around?

Comment 5 Orion Poplawski 2019-08-17 04:47:01 UTC
PR - https://src.fedoraproject.org/rpms/proj/pull-request/6

Comment 6 Devrim Gündüz 2019-09-01 22:31:29 UTC
Hi,

(In reply to Orion Poplawski from comment #4)
> Looks like libgeotiff 1.5 requires proj 6 as well.  Devrim, are you still
> around?

6.2.0 is is just out. Please let's start with 6.2.0.

I am on it.

Comment 7 Fedora Update System 2019-09-02 00:28:48 UTC
FEDORA-2019-29c418b1d7 has been submitted as an update to Fedora 31. https://bodhi.fedoraproject.org/updates/FEDORA-2019-29c418b1d7

Comment 8 Devrim Gündüz 2019-09-02 01:15:29 UTC
Hi Orion,

(In reply to Orion Poplawski from comment #4)
> Looks like libgeotiff 1.5 requires proj 6 as well.  Devrim, are you still
> around?

That said, I pushed 1.5.1 to EPEL 8. I'll leave Fedora to Volker.

Comment 9 Fedora Update System 2019-09-02 23:56:24 UTC
proj-6.2.0-1.fc31 has been pushed to the Fedora 31 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2019-29c418b1d7

Comment 10 Miro Hrončok 2019-09-04 12:49:59 UTC
The update actually broke pyproj.

Problem: conflicting requests
- nothing provides libproj.so.13()(64bit) needed by python3-cartopy-0.17.0-6.fc31.x86_64
Problem: package python3-geopandas-0.5.1-1.fc31.noarch requires python3.7dist(pyproj), but none of the providers can be installed
- conflicting requests
- nothing provides libproj.so.13()(64bit) needed by python3-pyproj-1.9.6-3.fc31.x86_64
- nothing provides proj-epsg needed by python3-pyproj-1.9.6-3.fc31.x86_64
- nothing provides proj-nad needed by python3-pyproj-1.9.6-3.fc31.x86_64

Comment 11 Devrim Gündüz 2019-09-04 13:12:34 UTC
Hi,

Sorry for the breakage. I am working on fixing it.

Regards, Devrim

Comment 12 Devrim Gündüz 2019-09-04 14:02:31 UTC
Hi Miro,

I also tried to update to 2.3.1, but it failed:

https://koji.fedoraproject.org/koji/taskinfo?taskID=37456340

https://kojipkgs.fedoraproject.org//work/tasks/6340/37456340/build.log

Can you please take a look?

Sorry about that.

Regards, Devrim

Comment 13 Miro Hrončok 2019-09-04 14:13:41 UTC
You need to recythonize the sources. See https://pagure.io/packaging-committee/issue/893

Comment 14 Devrim Gündüz 2019-09-04 14:47:19 UTC
Hi Miro,

I am not a Python/Cython guy -- and read the link and sublinks as  well :) Still, not quite sure what I need to do.


Is that something like

for file in `find %{buildroot} -iname "*.py"`
do
cython $i
done

or?

Sorry again.

Devrim

Comment 16 Jos de Kloe 2019-09-05 09:46:13 UTC
Hi Devrim and Miro,

first a big thank you for stepping in and doing the pyproj update.
The new proj for rawhide seems not yet available in my local mirror, but if I download the proj 6.2.0 generated rpms from koji and manually insert them in mock I can locally rebuild this new version.

There may be one small issue left. If I use rpmlint on the generated rpm files it complains about one file in the docs:

    python3-pyproj.x86_64: E: wrong-script-interpreter /usr/share/doc/python3-pyproj/docs/conf.py /usr/bin/env python

If I try to generate docs manually in the unpacked source tree /builddir/build/BUILD/pyproj-2.3.1/docs/ using the provided Makefile, it becomes clear that also the Makefile still points to plain python without version.

So my questions:
* should generation of docs (html or other) be added to the specfile? I find no statements on this in the packaging guidelines.
* if so, should we patch the conf.py and Makefile in the docs dir to point to python3
* if not, should we still patch these files in case users wish to generate the documentation themselves?

What do you think?

Comment 17 Miro Hrončok 2019-09-05 09:57:23 UTC
(In reply to Jos de Kloe from comment #16)
> Hi Devrim and Miro,
> 
> first a big thank you for stepping in and doing the pyproj update.
> The new proj for rawhide seems not yet available in my local mirror, but if
> I download the proj 6.2.0 generated rpms from koji and manually insert them
> in mock I can locally rebuild this new version.
> 
> There may be one small issue left. If I use rpmlint on the generated rpm
> files it complains about one file in the docs:
> 
>     python3-pyproj.x86_64: E: wrong-script-interpreter
> /usr/share/doc/python3-pyproj/docs/conf.py /usr/bin/env python

That shebang would ideally be removed, yes.

> If I try to generate docs manually in the unpacked source tree
> /builddir/build/BUILD/pyproj-2.3.1/docs/ using the provided Makefile, it
> becomes clear that also the Makefile still points to plain python without
> version.
> 
> So my questions:
> * should generation of docs (html or other) be added to the specfile? I find
> no statements on this in the packaging guidelines.

It might. Not should.

> * if so, should we patch the conf.py and Makefile in the docs dir to point
> to python3

You should remove the shebang (preferably in upstream). For Makefile, I believe it just uses the "sphinx-build" command, doesn't it?

> * if not, should we still patch these files in case users wish to generate
> the documentation themselves?

You should still remove the shebang. Otherwise I wouldn't patch anything.

Comment 18 Devrim Gündüz 2019-09-05 11:31:03 UTC
(In reply to Miro Hrončok from comment #15)
> https://src.fedoraproject.org/rpms/pyproj/pull-request/2

Sorry, forgot to thank you.

Regards, Devrim

Comment 19 Miro Hrončok 2019-09-05 11:33:26 UTC
No problem :)

Comment 20 Jos de Kloe 2019-09-06 09:05:42 UTC
I proposed a patch upstream for the python->python3 issue, which was accepted and merged.
Next I'll take a look at the spec file to patch the issue for this current version.

Also I will look again at the documentation generation. The provided makefile uses "python -msphinx" to generate the documentation, not sphinx-build.

Comment 21 Jos de Kloe 2019-09-08 10:56:15 UTC
Fix for plain python usage and documentation generation has been added now to the rawhide version.

Comment 22 Jos de Kloe 2019-11-10 11:49:07 UTC
The latest version 2.4.1 of pyproj has been build succesfully now for rawhide, so closing this bug.


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