- Update devel/py-gitpython to 2.1.9 - Take maintainership PR: 227017 Approved by: woodsb02, firstname.lastname@example.org (maintainer timeout) Differential_Revision: https://reviews.freebsd.org/D14899
testport: OK (11.1-RELEASE amd64, 10.4-RELEASE i386)
bulk build: OK (11.1-RELEASE amd64) for:
devel/py-git-up devel/py-git_semver security/w3af
If the maintainer does not respond within three months, or if there have been three consecutive timeouts, then that maintainer is considered absent without leave, and can be replaced as the maintainer of the particular port in question.
I realized that the statement could probably be omitted here, but I just wanted to address that. Do you have a better suggestion for this circumstance?
Well, this sentence is not there for that purpose. This sentence means that if there are three consecutive timeouts, the maintainership can be reset to whatever is needed, most of the time, ports@ or any generic maintainer like perl@ or python@, and the previous maintainer cannot complain about it.
But any patch can be applied after only one maintainer timeout, including one which changes the maintainership.
According to https://www.freshports.org/devel/py-gitpython, there are 3 ports which depend on this port:
Can you please:
- Bump the PORTREVISION variable of these dependent ports (adding a line to the Makefile if it is not already there), and include these changes in this review
- Run poudriere bulk -t on these dependent ports with your updated version of devel/py-gitpython, and bumped PORTREVISIONS, and report the success in the “test plan” section of this review.
- Have you been able to conduct any runtime testing of this updated port to confirm it functions ok (not always possible depending on familiarity with a port, but good to document in the “test plan” section one way or the other).
Fair challenge - for the record Guangyuan this is related to section 188.8.131.52 of the porters handbook.
My thought process was to ensure the dependent ports get upgraded by pkg to ensure they keep working if they have any compiled code which depends on the old version of devel/py-gitpython.
However, given this python port does not install shared libraries, I don’t believe this step (item 1 in my list above) is required.
Approved for you to commit.
In the commit message, I believe it is preferred to have one line for each person who approved/reviewed. So please copy the “approved by:” line in the commit template, to look like:
Approved by: email@example.com (maintainer timeout)
Approved by: woodsb02 (ports)
Nice work Guangyuan!