[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Acl-devel] [PATCH 2/2] build: ship a pkgconfig file for libattr
From: |
Mike Frysinger |
Subject: |
Re: [Acl-devel] [PATCH 2/2] build: ship a pkgconfig file for libattr |
Date: |
Fri, 10 Jan 2014 12:50:50 -0500 |
User-agent: |
KMail/1.13.7 (Linux/3.12.1; KDE/4.6.5; x86_64; ; ) |
On Friday 10 January 2014 12:46:03 Jan Engelhardt wrote:
> On Friday 2014-01-10 18:40, Mike Frysinger wrote:
> >On Friday 10 January 2014 09:47:23 Jan Engelhardt wrote:
> >> --- a/Makefile.am
> >> +++ b/Makefile.am
> >>
> >> -DLOCALEDIR=\"$(localedir)\" \
> >> -DSYSCONFDIR=\"$(sysconfdir)\"
> >>
> >> +pkgconfdir = ${libdir}/pkgconfig
> >
> >this should be $(libdir) and not ${libdir}
>
> It does not really matter - using {} to reference variables is
> accepted by make implementations, and probably even inspired by sh and
> the P family of scripting languages which have been using it for ages.
i'm aware. however, the code base uses $(...) everywhere and it should not
deviate for no good reason (of which i see none here).
-mike
signature.asc
Description: This is a digitally signed message part.
Re: [Acl-devel] [PATCH 1/2] build: make use of an aux-dir to stow away helper scripts, Mike Frysinger, 2014/01/10
Re: [Acl-devel] [PATCH] build: drop attrincludedir, use pkgincludedir, Mike Frysinger, 2014/01/10