From mboxrd@z Thu Jan 1 00:00:00 1970 From: Olaf Hering Subject: Re: [PATCH 25/46] Use configure --mandir=DIR to set MANDIR Date: Tue, 23 Sep 2014 14:23:24 +0200 Message-ID: <20140923122324.GC10033@aepfle.de> References: <1411390835-7348-1-git-send-email-olaf@aepfle.de> <1411390835-7348-26-git-send-email-olaf@aepfle.de> <1411464725.14989.29.camel@kazak.uk.xensource.com> <1411464989.14989.33.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1411464989.14989.33.camel@kazak.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: Keir Fraser , David Scott , Stefano Stabellini , George Dunlap , Tim Deegan , Ian Jackson , xen-devel@lists.xen.org, Jan Beulich , Samuel Thibault List-Id: xen-devel@lists.xenproject.org On Tue, Sep 23, Ian Campbell wrote: > Maybe we are just using it wrong and we should be doing in Paths.mk.in: > prefix=@prefix@ > datarootdir=@datarootdir@ > mandir=@mandir@ > ... > MANDIR=\$mandir > > (or even just switching our code to $mandir). Perhaps looking at what > automake would put in Makefile.in for a generic project would be > informative? > > That would be a bigger change though and I'd be OK with deferring for > now. With automake the resulting variables are all lowercase. Perhaps thats the reason why configure uses also lowercase. Just xen uses uppercase for all variables. So my options are either: expand to lowercase variables and assign to existing uppercase variables later in that file. Or do a treewide conversion from upper to lowercase. Looks like the simplest change would be to expand all variables from configure in Paths.mk and assign them later to existing variables. Olaf