DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:from:date:to:subject:message-id:references
	:mime-version:content-type:in-reply-to; q=dns; s=default; b=Malh
	Il6EsFzJsk0gh92SzedhLiTM7C93VTScvq4mrWjPUTOT5lNjM8MES8r4nldo7bUS
	GQLB8hptibt9yNB5IJIO2F0KAW/IEpeDN254DAJUGAI6FrD+S+9mWS91Oci60Mkl
	8F0XAlOk5ptWVpgIeXRMicPAElD5+qOmSqi/4gI=
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
From: Stephen Carrier <carrier@berkeley.edu>
Date: Wed, 9 Oct 2019 13:03:38 -0700
To: cygwin@cygwin.com
Subject: Re: msmtp 1.8.5 built using ${prefix}/${SYSCONFDIR}
Message-ID: <20191009200338.GA3790@zebra>
References: <20191002004739.GA5215@zebra> <a43f1570-c87b-6b83-c8bf-b824a1aacc40@SystematicSw.ab.ca> <20191002181838.GA3281@zebra> <eaffbcc1-415d-3cb3-a3f4-445c186d7f27@SystematicSw.ab.ca>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <eaffbcc1-415d-3cb3-a3f4-445c186d7f27@SystematicSw.ab.ca>
User-Agent: Mutt/1.9.1 (2017-09-22)

On Wed, Oct 02, 2019 at 01:38:13PM -0600, Brian Inglis wrote:
> On 2019-10-02 12:18, Stephen Carrier wrote:
> > On Tue, Oct 01, 2019 at 08:43:40PM -0600, Brian Inglis wrote:
> >> On 2019-10-01 18:47, Stephen Carrier wrote:
> >>> The latest verion of the msmtp package: 1.8.5+20190811+git7912c76-1 .
> >>> Differs from the previous version: 1.6.6-1 .
> >>>
> >>> In that the location of the system-wide configuration file is changed
> >>> from /etc/msmtprc to /usr/etc/msmtprc.
> >>>
> >>> I think this has the potential to break installations during an upgrade.
> >>>
> >>> In my case it only confused me as I set up a new system.
> >>>
> >>> Please consider changing this default back to what it was for the
> >>> convenience of those who will otherwise need to figure this out.
> >>
> >> Definitely not a standard location and nothing there currently on my system.
> >> The package contents shows only server configuration, nothing on /usr/etc/, no
> >> defaults provided in /etc/defaults/usr/etc/, and /usr/bin/msmtp-config defaults
> >> the CONFDIR to /etc, so unless you have a symlink, link or junction from /etc to
> >> /usr/etc, or changed the location during the server config, /etc/ should have
> >> been used.
> > 
> > This is not what I meant.  The package doesn't provide a configuration
> > file.  msmtp-config generates one in /etc/msmtprc, exactly as I would
> > want and expect since that's where to find the configuration file for
> > previous versions.
> > 
> > The problem is that the binary looks for its configuration file in a
> > different place:
> 
> > $ msmtp --version
> > msmtp version 1.8.5
> > Platform: x86_64-pc-cygwin
> > ....
> > System configuration file name: /usr/etc/msmtprc
> 
> > and only after I moved the configuration file to that nonstandard location
> > would msmtp start working instead of complaining that it couldn't find
> > any configuration file.
> > 
> > This seems like something that could be fixed by tweaking the package
> > build scripts; I think it would be great to do that.
> 
> So this is a discrepancy between the built exe and the config script, which
> should be resolved by having the built exe use the standard SYSCONFDIR, by
> removing ${prefix} from its definition.
> [changing subject]

Could the package maintainer join in?  I don't see how discussing the
minutae of the build process actually helps without the maintainer's
participation.

Pre-existing msmtp installations use /etc/msmtprc, and the current
configuration script creates a valid configuration in that same location.
Clearly, fixing the binary and not the configuration script is the way
forward, unless there is compelling reason to move the configuration
file from where it used to be.

Thanks,

Stephen Carrier

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple