Bug 252180

Summary: devel/py-cookiecutter: adjust dependencies
Product: Ports & Packages Reporter: Charlie Li <vishwin>
Component: Individual Port(s)Assignee: Matthew Seaman <matthew>
Status: Closed FIXED    
Severity: Affects Many People Flags: bugzilla: maintainer-feedback? (matthew)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Bug Depends on: 250941    
Bug Blocks:    
Attachments:
Description Flags
v1 none

Description Charlie Li freebsd_committer freebsd_triage 2020-12-27 02:08:58 UTC
Created attachment 220973 [details]
v1

When I tried to even print the help message, cookiecutter complains of missing and outdated dependencies. Match what is in setup.py.

This includes a line for www/py-requests>=2.23.0, which is being addressed in another PR.
Comment 1 Matthew Seaman freebsd_committer freebsd_triage 2020-12-27 15:54:33 UTC
Thank you very much for the report.  I guess we're waiting on 250941 for now, although I wonder if the absolute latest version of py-requests really is required? Python module versions in dependencies tend to be "what the developers tested with" which usually equates to "what was current when they ran the tests" and frequently don't include all of the dependency versions that could work just fine.
Comment 2 commit-hook freebsd_committer freebsd_triage 2020-12-28 10:14:48 UTC
A commit references this bug:

Author: matthew
Date: Mon Dec 28 10:14:38 UTC 2020
New revision: 559453
URL: https://svnweb.freebsd.org/changeset/ports/559453

Log:
  Fix `make test` to achieve 100% pass rate by converting entirely
  unnecessary references to '/bin/bash' to use '/bin/sh' instead.

  Update dependency versions to match 'setup.cfg' _except_ for
  py-requests, where the ports currently has only 2.22.0 available, but
  upstream is requiring at least 2.23.0. (Waiting on PR 250941) [1]

  PR:             252180
  Submitted by:   Charlie	Li

Changes:
  head/devel/py-cookiecutter/Makefile
  head/devel/py-cookiecutter/files/
  head/devel/py-cookiecutter/files/patch-tests_test-generate-files-permissions_input{{cookiecutter.permissions}}_script.sh
  head/devel/py-cookiecutter/files/patch-tests_test-pyshellhooks_hooks_post__gen__project.sh
  head/devel/py-cookiecutter/files/patch-tests_test-pyshellhooks_hooks_pre__gen__project.sh
  head/devel/py-cookiecutter/files/patch-tests_test-shellhooks_hooks_post__gen__project.sh
  head/devel/py-cookiecutter/files/patch-tests_test-shellhooks_hooks_pre__gen__project.sh
  head/devel/py-cookiecutter/files/patch-tests_test__hooks.py
Comment 3 Matthew Seaman freebsd_committer freebsd_triage 2020-12-28 10:21:10 UTC
Most of the dependency versions now match what upstream requires.  Still waiting on PR 250941 for the devel/py-requests port to be updated to >2.23.0, but in the mean time do the best we can by requiring at least 2.22.0

After removing some gratuitous and entirely unnecessary use of '/bin/bash' -- the test scripts are along the lines of 'touch foo ; echo bar' -- we now have 100% of the tests passing.

I'll leave this PR open until I can sort out the py-requests dependency version properly.
Comment 4 Charlie Li freebsd_committer freebsd_triage 2020-12-28 10:55:28 UTC
koobs@ has commandeered the rest of the devel/py-requests stuff after some hints to progress forward. We've at least got their test suite under control so it's now onto consumers.

This port only requires at least 2.23.0 as stated in setup.cfg, but for the sake of other consumers in the tree in addition to shutting up portscout, that port's PR is for 2.25.1.
Comment 5 commit-hook freebsd_committer freebsd_triage 2021-05-22 20:53:05 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=c4f9aff50af6e368137c3c81300d58e0892b078b

commit c4f9aff50af6e368137c3c81300d58e0892b078b
Author:     Matthew Seaman <matthew@FreeBSD.org>
AuthorDate: 2021-05-22 20:48:04 +0000
Commit:     Matthew Seaman <matthew@FreeBSD.org>
CommitDate: 2021-05-22 20:52:24 +0000

    devel/py-cookiecutter: Amend py-requests dependency version

    With 7d02ece23c1a68 updating www/py-requests to the current version,
    we can correctly match the version requirements from upstream.

    PR:     252180
    Reported by:    vishwin

 devel/py-cookiecutter/Makefile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Comment 6 Matthew Seaman freebsd_committer freebsd_triage 2021-05-22 20:53:28 UTC
Committed, thanks!