public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Brian Inglis <Brian.Inglis@Shaw.ca>
To: cygwin-apps@cygwin.com
Subject: Re: Automatic announcement generation by calm
Date: Mon, 8 Jan 2024 06:42:34 -0700	[thread overview]
Message-ID: <48d33665-ca88-4699-9dd5-5030fc131e1c@Shaw.ca> (raw)
In-Reply-To: <5d19b39d-26e9-45a6-a003-6b9f403a306f@towo.net>

On 2024-01-08 06:01, Thomas Wolff via Cygwin-apps wrote:
> Am 08/01/2024 um 13:35 schrieb Corinna Vinschen via Cygwin-apps:
>> On Jan  7 16:12, Jon Turney via Cygwin-apps wrote:
>>> This is an experimental facility, currently only available for packages
>>> deployed from the build service [1] (that is, not for self-built packages
>>> uploaded with 'cygport up' via sftp)
>>>
>>> When the token "announce" is present for a build job (in addition to
>>> "deploy"), after a successful deploy, calm will automatically generate and
>>> send an announce email.
>>>
>>> The mail follows a similar format to that generated by "cygport announce",
>>> containing a list of packages and the description, with the following
>>> addition:

Maybe also append:
	"
	For more information, see the project home page:

		$HOMEPAGE
	"

>>> * If the cygport defines the variable "ANNOUNCE", it's evaluated contents
>>> will be appended to the generated mail.

Evaluated how - cygport variable expansion - commands?

Could ANNOUNCE variable contain a source file name or a URI?

>>> * Otherwise, if the source package contains an ANNOUNCE file [2], it's
>>> contents will be appended.

Could ANNOUNCE be a symlink to $NAME-$PVR.$ARCH/origsrc/$SRC_DIR/NEWS?

>>> * Otherwise, if the source package contains a README or ${PN}.README file,
>>> lines that look like part of a changelog, between one starting with '----
>>> ${PVR}' and the next starting '----', will be extracted and appended
>>>
>>> (None of these seem like a particularly great way of doing things, but they
>>> match some historical patterns.  As always, suggestions about improvements
>>> are welcome.)
>>>
>>>
>>> In accordance with our long-standing policy of treating maintainer email
>>> addresses as private information, the mail is sent from cygwin-no-reply and
>>> bcc'ed to the uploader.

How about email headers:

	From: "Cygwin $NAME Maintainer" <cygwin-no-reply@cygwin.com>
	To: Cygwin Announcements <cygwin-announce@cygwin.com>
	Bcc: $SMTP_SENDER
	Reply-To: Cygwin <cygwin@cygwin.com>
	Subject: Updated: $NAME $PVR

>>> For testing purposes, if the token "mock" (yes, I am running out of synonyms
>>> for "test"...) is also present, the mail will be only sent to the uploader,
>>> not the announce list.
>>>
>>>
>>> [1] https://cygwin.com/packaging/build.html
>>> [2] Note that this isn't currently part of the default value of CYGWIN_FILES
>>> [3], so needs to be explicitly listed there to be included in the source
>>> package
>>> [3] https://cygwin.github.io/cygport/src_prep_cygpart.html#CYGWIN_FILES
>> :+1:
>>
>> Unfortunately I started the OpenSSH 9.6 build before reading this here,
>> but that's some great addition.

> I'd also appreciate to prefix the mail with an "[ANNOUNCEMENT] " tag as for the 
> mails forwarded from cygwin-announcement to cygwin before that was stopped, to 
> enhance the overview in users' mailboxes.

Spammy looking compared to "Updated: ..."?

Maybe also update cygport ... announce to be as close as possible to calm?

-- 
Take care. Thanks, Brian Inglis              Calgary, Alberta, Canada

La perfection est atteinte                   Perfection is achieved
non pas lorsqu'il n'y a plus rien à ajouter  not when there is no more to add
mais lorsqu'il n'y a plus rien à retirer     but when there is no more to cut
                                 -- Antoine de Saint-Exupéry

  reply	other threads:[~2024-01-08 13:42 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-07 16:12 Jon Turney
2024-01-08 12:35 ` Corinna Vinschen
2024-01-08 13:01   ` Thomas Wolff
2024-01-08 13:42     ` Brian Inglis [this message]
2024-01-14 18:59       ` Jon Turney

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=48d33665-ca88-4699-9dd5-5030fc131e1c@Shaw.ca \
    --to=brian.inglis@shaw.ca \
    --cc=cygwin-apps@cygwin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).