Page MenuHomeFreeBSD

devel/py-gitpython: Update to 2.1.9
ClosedPublic

Authored by ygy on Mar 29 2018, 6:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 12, 12:44 AM
Unknown Object (File)
Feb 26 2024, 8:42 PM
Unknown Object (File)
Jan 21 2024, 9:37 AM
Unknown Object (File)
Dec 31 2023, 10:33 AM
Unknown Object (File)
Dec 25 2023, 1:58 PM
Unknown Object (File)
Dec 23 2023, 12:03 AM
Unknown Object (File)
Dec 1 2023, 2:17 AM
Unknown Object (File)
Nov 2 2023, 9:02 PM

Details

Summary
- Update devel/py-gitpython to 2.1.9
- Take maintainership

PR: 227017
Approved by: woodsb02, che@bein.link (maintainer timeout)
Differential_Revision: https://reviews.freebsd.org/D14899
Test Plan

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

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

ygy planned changes to this revision.Mar 29 2018, 6:31 PM

Hi ygy,
This review is marked as "changes planned" - was this deliberate? If so, can you describe the problems you have found which are planning to address?
Cheers,
Ben

Hi ygy,
This review is marked as "changes planned" - was this deliberate? If so, can you describe the problems you have found which are planning to address?
Cheers,
Ben

I'm waiting for a (possible) maintainer timeout to take it over :-) It's been two timeouts.

Then just change the maintainer and wait for one timeout, it is enough.

ygy added a reviewer: woodsb02.

I am not sure what the "3 consecutive times" adds here, could you explain?

In D14899#326603, @mat wrote:

I am not sure what the "3 consecutive times" adds here, could you explain?

From https://www.freebsd.org/doc/en/books/porters-handbook/makefile-maintainer.html:

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?

In D14899#327870, @ygy wrote:
In D14899#326603, @mat wrote:

I am not sure what the "3 consecutive times" adds here, could you explain?

From https://www.freebsd.org/doc/en/books/porters-handbook/makefile-maintainer.html:

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.

In D14899#327968, @mat wrote:
In D14899#327870, @ygy wrote:
In D14899#326603, @mat wrote:

I am not sure what the "3 consecutive times" adds here, could you explain?

From https://www.freebsd.org/doc/en/books/porters-handbook/makefile-maintainer.html:

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.

Oh now I understand. I will rewrite the message then, thanks for the explanation!

Hi Guangyuan,
According to https://www.freshports.org/devel/py-gitpython, there are 3 ports which depend on this port:
devel/py-git-up
devel/py-git_semver
security/w3af

Can you please:

  1. 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
  2. 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.
  3. 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).

Thanks,
Ben

In D14899#328651, @mat wrote:

Why bump dependant ports?

Fair challenge - for the record Guangyuan this is related to section 5.2.3.1 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.

@woodsb02 @mat Noted and updated the test plan. I will include these in future reviews.

ygy removed a subscriber: woodsb02.

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: che@bein.link (maintainer timeout)
Approved by: woodsb02 (ports)

Nice work Guangyuan!

This revision is now accepted and ready to land.Jun 7 2018, 10:49 PM

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: che@bein.link (maintainer timeout)
Approved by: woodsb02 (ports)

It is purely a cosmetic preference, I prefer to put everyone on one line, keeps things short.

This revision was automatically updated to reflect the committed changes.