Re: cygport: 'announce' feature branch

[Corinna Vinschen <corinna-cygwin at cygwin dot com>]

On Nov  3 18:10, Yaakov Selkowitz wrote:
> On Tue, 2015-11-03 at 18:23 -0500, Ken Brown wrote:
> > On 11/3/2015 3:06 AM, Yaakov Selkowitz wrote:
> > > I just created a new 'announce' feature branch in cygport git:
> > >
> > > https://github.com/cygwinports/cygport/tree/announce
> > >
> > > The new 'announce' command generates an announcement template from
> > > the .cygport file based on NAME/VERSION/RELEASE, PKG_NAMES, and
> > > DESCRIPTION, to which you can easily add release-specific information,
> > > and then sends the message via an SMTP server.  Note that this adds
> > 
> > This is a nice feature.  Thanks.
> > 
> > > dependencies on perl-Authen-SASL, perl-MIME-tools, and
> > 
> > perl-MIME-tools isn't in the distro yet.
> 
> Good point, I just moved it in from Ports.
> 
> > If there's no mailserver configured, I'd like to see cygport save the 
> > mbox file in the current directory and give an informational message, 
> > rather than saving it in /tmp and giving an error message.  Users might 
> > want to simply paste it into an email, and the error message seems a 
> > little unfriendly.
> 
> First, I have already changed SMTP_SERVER to default to localhost,
> matching git's behaviour.  (If it wasn't already painfully obvious, this
> is all modelled after git send-email.)  Therefore, the error message now
> reflects an actual failure to send the message.  As for the location of
> the mbox file, it's something I'm willing to consider.

I agree with Ken.  $PWD or perhaps $HOME would be nice.  $PWD seems
the most obvious choice.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

Attachment: pgphtX3hCXnyB.pgp
Description: PGP signature