Bug 241369 - www/py-cheroot: Update to 6.5.6
Summary: www/py-cheroot: Update to 6.5.6
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Kai Knoblich
URL: https://github.com/cherrypy/cheroot/b...
Keywords:
Depends on:
Blocks:
 
Reported: 2019-10-20 13:59 UTC by Keith Gaughan
Modified: 2019-10-30 00:09 UTC (History)
2 users (show)

See Also:
k: maintainer-feedback+
kai: merge-quarterly+


Attachments
Patch (817 bytes, patch)
2019-10-20 13:59 UTC, Keith Gaughan
no flags Details | Diff
py-cheroot-6.5.6-revised.patch (8.84 KB, patch)
2019-10-27 17:56 UTC, Kai Knoblich
kai: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Keith Gaughan 2019-10-20 13:59:17 UTC
Created attachment 208462 [details]
Patch

This isn't the most recent version (that would be 8.2.1), but it is fairly recent. There's a significant amount of version churn in the upstream project currently, and I'm waiting on that to settle down before updating further.
Comment 1 Keith Gaughan 2019-10-20 14:00:58 UTC
Further, 6.5.6 is the most recent signed release: https://github.com/cherrypy/cheroot/releases/tag/v6.5.6
Comment 2 Kai Knoblich freebsd_committer freebsd_triage 2019-10-27 17:56:48 UTC
Created attachment 208640 [details]
py-cheroot-6.5.6-revised.patch

(In reply to Keith Gaughan from comment #0)

Hi Keith,

thank you for the patch. Attached is a revised version that fixes some issues:

- The port devel/py-setuptools_scm_git_archive is required to build otherwise the build process fails in a early stage.

- The port devel/py-backports.functools_lru_cache is also required for runtime but only for Python < 3.3. To make this happen we need to backport a patch from the 6.6.0 release for "setup.cfg".

- Since ports r515635 we need also to patch "setup.py" with a patch backported from the 8.0 release.

- At last but not least: Because the port installs a script (= ${PREFIX}/bin/cheroot) outside the Python environment we need to add the keyword "concurrent" to make the port concurrent safe.

If you're fine with the changes and www/py-cheroot works as expected I'm happy then to commit it as soon as possible.
Comment 3 Keith Gaughan 2019-10-27 18:10:00 UTC
Yeah, that's all fine by me. Thanks for that! I'm not sure why the build didn't break for me. I'd known about the LRU cache issue, but  had mentally dismissed it due to the oldest (3 series) version in ports being 3.5, and entirely forgot about 2.7.

+2 to the revised patch!
Comment 4 commit-hook freebsd_committer freebsd_triage 2019-10-28 08:09:47 UTC
A commit references this bug:

Author: kai
Date: Mon Oct 28 08:09:35 UTC 2019
New revision: 515848
URL: https://svnweb.freebsd.org/changeset/ports/515848

Log:
  www/py-cheroot: Update to 6.5.6

  * There is currently a signification amount of version churn in the upstream
    repository so use the most recent signed release for now.

  * Since release 6.3.0 the port installs a CLI tool outside the Python
    environment thus make it concurrent safe.

  * Also backport two patches to retain compatibility with recent versions of
    devel/py-setuptools and avoid failures at runtime with Python 3 while I'm
    here. [1]

  Changelog since 6.0.0:

  https://github.com/cherrypy/cheroot/blob/v6.5.6/CHANGES.rst

  PR:		241369
  Submitted by:	Keith Gaughan <k@stereochro.me> (maintainer)
  Approved by:	Keith Gaughan <k@stereochro.me> (maintainer) [1]
  MFH:		2019Q4

Changes:
  head/www/py-cheroot/Makefile
  head/www/py-cheroot/distinfo
  head/www/py-cheroot/files/
  head/www/py-cheroot/files/patch-setup.cfg
  head/www/py-cheroot/files/patch-setup.py
Comment 5 Kai Knoblich freebsd_committer freebsd_triage 2019-10-28 08:31:04 UTC
(In reply to Keith Gaughan from comment #3)

Thank you for the quick approval. The changes were committed to the head branch.

I'll try to merge them into the 2019Q4 branch if I get approval from the ports-secteam.
Comment 6 Kai Knoblich freebsd_committer freebsd_triage 2019-10-28 08:32:35 UTC
Comment on attachment 208640 [details]
py-cheroot-6.5.6-revised.patch

^ Triage: maintainer-approval was given in comment #3
Comment 7 commit-hook freebsd_committer freebsd_triage 2019-10-30 00:06:38 UTC
A commit references this bug:

Author: kai
Date: Wed Oct 30 00:05:59 UTC 2019
New revision: 516042
URL: https://svnweb.freebsd.org/changeset/ports/516042

Log:
  MFH: r515848

  www/py-cheroot: Update to 6.5.6

  * There is currently a signification amount of version churn in the upstream
    repository so use the most recent signed release for now.

  * Since release 6.3.0 the port installs a CLI tool outside the Python
    environment thus make it concurrent safe.

  * Also backport two patches to retain compatibility with recent versions of
    devel/py-setuptools and avoid failures at runtime with Python 3 while I'm
    here. [1]

  Changelog since 6.0.0:

  https://github.com/cherrypy/cheroot/blob/v6.5.6/CHANGES.rst

  PR:		241369
  Submitted by:	Keith Gaughan <k@stereochro.me> (maintainer)
  Approved by:	Keith Gaughan <k@stereochro.me> (maintainer) [1]

  Approved by:	ports-secteam (miwi)

Changes:
_U  branches/2019Q4/
  branches/2019Q4/www/py-cheroot/Makefile
  branches/2019Q4/www/py-cheroot/distinfo
  branches/2019Q4/www/py-cheroot/files/
Comment 8 Kai Knoblich freebsd_committer freebsd_triage 2019-10-30 00:08:38 UTC
Committed to the 2019Q4 branch, all done!