Skip to content

Prep for 4.14 and 5.0.5#765

Closed
yadij wants to merge 2 commits into
squid-cache:masterfrom
yadij:release-prep
Closed

Prep for 4.14 and 5.0.5#765
yadij wants to merge 2 commits into
squid-cache:masterfrom
yadij:release-prep

Conversation

@yadij
Copy link
Copy Markdown
Contributor

@yadij yadij commented Jan 31, 2021

No description provided.

@yadij yadij added M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels S-waiting-for-more-reviewers needs a reviewer and/or a second opinion and removed M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels labels Jan 31, 2021
@yadij yadij requested review from kinkie and rousskov February 2, 2021 02:31
@yadij
Copy link
Copy Markdown
Contributor Author

yadij commented Feb 2, 2021

@rousskov, @kinkie could one of you guys please review this ASAP for anubis fast-track. If not I will have to bump the releases out to Feb 9.

Copy link
Copy Markdown
Contributor

@rousskov rousskov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am approving this simple change without a close review to put it on the fast track, per @yadij request. @kinkie, please feel free to request changes, of course!

However, I would like to use this opportunity to say that, IMO, copyright year changes in v4 (8e56335) and v5 (04f296e) should not have been committed until the resolution of #763 (review) discussion. It is very unpleasant for me to see changes shoveled into the official code like that.

Comment thread ChangeLog Outdated
@yadij
Copy link
Copy Markdown
Contributor Author

yadij commented Feb 2, 2021

I am approving this simple change without a close review to put it on the fast track, per @yadij request. @kinkie, please feel free to request changes, of course!

However, I would like to use this opportunity to say that, IMO, copyright year changes in v4 and v5 should not have been committed until the resolution of #763 (review) discussion. It is very unpleasant for me to see changes shoveled into the official code like that.

FTR: The above statement implies that the copyright renewal is an unusual occurrence. Where in fact it is a regular event since the copyright restructuring project in 2014. It is performed annually by squid-core approved automation.

Co-authored-by: Alex Rousskov <rousskov@measurement-factory.com>
@yadij yadij added M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels and removed M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels labels Feb 2, 2021
@yadij
Copy link
Copy Markdown
Contributor Author

yadij commented Feb 2, 2021

Marking for Anubis with the requested ChangeLog modification plus re-do of build checks.

@yadij yadij added M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels and removed S-waiting-for-more-reviewers needs a reviewer and/or a second opinion labels Feb 2, 2021
@rousskov
Copy link
Copy Markdown
Contributor

rousskov commented Feb 2, 2021

Alex: IMO, copyright year changes in v4 and v5 should not have been committed until the resolution of #763 (review) discussion. It is very unpleasant for me to see changes shoveled into the official code like that.

Amos: The above statement implies that the copyright renewal is an unusual occurrence

There is no such implication.

it is a regular event since the copyright restructuring project in 2014. It is performed annually by squid-core approved automation.

If you think that this technicality can somehow justify what happened, then, technically, Squid vX modifications that are also relevant to the master branch must happen in master first. The existence of a modification script is not relevant to that key principle, of course.

@yadij yadij removed the request for review from kinkie February 3, 2021 11:10
@squid-anubis squid-anubis added the M-waiting-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels label Feb 3, 2021
squid-anubis pushed a commit that referenced this pull request Feb 3, 2021
@squid-anubis squid-anubis added M-merged https://github.com/measurement-factory/anubis#pull-request-labels and removed M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels M-waiting-staging-checks https://github.com/measurement-factory/anubis#pull-request-labels labels Feb 3, 2021
@yadij
Copy link
Copy Markdown
Contributor Author

yadij commented Feb 4, 2021

Alex: IMO, copyright year changes in v4 and v5 should not have been committed until the resolution of #763 (review) discussion. It is very unpleasant for me to see changes shoveled into the official code like that.

Amos: The above statement implies that the copyright renewal is an unusual occurrence

There is no such implication.

Then please clarify exactly what you meant by that last sentence. It is filled with emotive negative wording.

This whole discussion gives the impression that something surprising has happened that you are only now

and the topic of the paragraph is about the routine maintenance commits on v4 and v5 branches.

it is a regular event since the copyright restructuring project in 2014. It is performed annually by squid-core approved automation.

If you think that this technicality can somehow justify what happened, then, technically, Squid vX modifications that are also relevant to the master branch must happen in master first. The existence of a modification script is not relevant to that key principle, of course.

The automatic maintenance on each branch is independent. The scripts/source-maintenance.sh in each vN branch are what is run and the resulting changes merged to that vN branch after the branch maintainer (a.k.a. me) reviews.

You should well know how things work with Squid code management. I'm not sure why I have to explain this, except that you called seeing the scripts in operation "unpleasant". So ... ??!

squidadm pushed a commit to squidadm/squid that referenced this pull request Feb 4, 2021
squidadm pushed a commit to squidadm/squid that referenced this pull request Feb 4, 2021
@rousskov
Copy link
Copy Markdown
Contributor

rousskov commented Feb 4, 2021

Then please clarify exactly what you meant by that last sentence.

When a core developer raises an issue about changing the copyright year, I expect the maintainer not to change the copyright year in the official branches until the issue is at least discussed. To me, it is just basic courtesy -- something that cannot be codified, but should be practiced and expected by everybody, especially folks on the core team. If others do not see how such actions can feel very unpleasant, I do not know how to explain it.

This whole discussion gives the impression that something surprising has happened that you are only now

Unfortunately, I cannot say I was really surprised. Similar cooperation difficulties is, sadly, a recurring Squid Project theme.

@rousskov rousskov mentioned this pull request Apr 4, 2021
@yadij yadij mentioned this pull request Mar 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

M-merged https://github.com/measurement-factory/anubis#pull-request-labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants