Page MenuHomeFreeBSD

Make it possible not to send MFH emails
Needs ReviewPublic

Authored by adamw on Jun 19 2019, 12:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 22 2024, 5:09 PM
Unknown Object (File)
Feb 13 2024, 4:47 AM
Unknown Object (File)
Feb 3 2024, 11:42 PM
Unknown Object (File)
Dec 25 2023, 4:40 AM
Unknown Object (File)
Dec 20 2023, 2:07 AM
Unknown Object (File)
Dec 6 2023, 9:25 PM
Unknown Object (File)
Nov 9 2023, 4:32 PM
Unknown Object (File)
Nov 2 2023, 2:41 PM
Subscribers

Details

Reviewers
delphij
Group Reviewers
ports secteam
portmgr
Summary

This treats

MFH:     2019Q2 (any text)

as a signal not to send the MFH request email.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Oops, there was a typo there.

I don't know about this, I think it is better to send an email that is ignored than silently doing things.

This revision is now accepted and ready to land.Jun 19 2019, 3:49 PM
In D20698#447294, @mat wrote:

I don't know about this, I think it is better to send an email that is ignored than silently doing things.

I think ultimately what happens on quarterly branch is still being covered by commit mail? This only affects MFH approval email, which seems to be redundant if it's a hat commit and the intention was not to ask an approval in the first place.

In D20698#447294, @mat wrote:

I don't know about this, I think it is better to send an email that is ignored than silently doing things.

I think ultimately what happens on quarterly branch is still being covered by commit mail? This only affects MFH approval email, which seems to be redundant if it's a hat commit and the intention was not to ask an approval in the first place.

Then if the MFH line contains anything else than a branch name, say:

MFH: 2019Q2 (security update blanket approval)

There is no need to send an email because it will already have gotten committed.

Maybe it should echo the fact that no MFH email is being sent. (So need two tests.)

In D20698#447300, @mat wrote:

Then if the MFH line contains anything else than a branch name, say:

MFH: 2019Q2 (security update blanket approval)

There is no need to send an email because it will already have gotten committed.

That's what the above patch does.

Maybe it should echo the fact that no MFH email is being sent. (So need two tests.)

That's a great idea. Instant feedback would help alleviate the behaving-silently thing.

In D20698#447300, @mat wrote:

Then if the MFH line contains anything else than a branch name, say:

MFH: 2019Q2 (security update blanket approval)

There is no need to send an email because it will already have gotten committed.

That's what the above patch does.

That's what it does only when you write "with hat". What I mean is that if you add a comment on the MFH line, whatever the comment is, an email should not be sent.

adamw edited the summary of this revision. (Show Details)

I see what you mean. Here it takes anything between parentheses as a sign to exit.

This revision now requires review to proceed.Jun 20 2019, 12:27 PM
hooks/scripts/mfh.sh
9โ€“10

I'm not sure you need to change this line, it does not change anything wrt what gets in MFH_LINE in the end.

10โ€“12

Splitting it in two to get a message out:

if [ "$MFH_LINE" = "" ]; then
        exit 0
fi
if echo "$MFH_LINE" | grep -q '(.*)'; then
        echo "Comment detected on the MFH line," 1>&2
        echo "not sending the MFH email." 1>&2
        exit 0
fi

I think we can safely abandon this? MFH mails are not sent anymore anyway and ports migrated to git.