[PATCH 2/4] fix hardcoded value of XPRINTDIR
Daniel Stone
daniel at fooishbar.org
Tue Jul 3 16:05:37 PDT 2007
On Tue, Jul 03, 2007 at 02:20:22PM -0700, Dan Nicholson wrote:
> On 7/3/07, Ademar de Souza Reis Jr <ademar at mandriva.com.br> wrote:
> > (now using the variable already set on configure: xpconfigdir)
> > ---
> > hw/xprint/Makefile.am | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/hw/xprint/Makefile.am b/hw/xprint/Makefile.am
> > index dc8764a..a687f35 100644
> > --- a/hw/xprint/Makefile.am
> > +++ b/hw/xprint/Makefile.am
> > @@ -4,7 +4,7 @@ bin_PROGRAMS = Xprt
> >
> > Xprt_CFLAGS = @DIX_CFLAGS@ @XPRINT_CFLAGS@ \
> > -DXPRINT -DPRINT_ONLY_SERVER -D_XP_PRINT_SERVER_ \
> > - -DXPRINTDIR=\"$(libdir)/X11/xserver\" \
> > + -DXPRINTDIR=\"@xpconfigdir@\" \
> > -DXPRASTERDDX -DXPPCLDDX -DXPMONOPCLDDX -DXPPSDDX \
> > -DXFree86Server
>
> Since xpconfigdir is AC_SUBSTed, wouldn't it be better to use
> $(xpconfigdir) there so it's substituted by make? That would ensure
> that all variables are resolved and you don't inadvertantly encode
> ${somevar} into xprint.
You need AC_DEFINE_DIR, instead of AC_SUBST.
Cheers,
Daniel
More information about the xorg
mailing list