Bug 17314 - libXau's Makefile.am should have proper man creation rules
Summary: libXau's Makefile.am should have proper man creation rules
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Lib/Xau (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-26 13:43 UTC by Damien Thébault
Modified: 2008-08-26 14:20 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
libXau 1.0.3 Makefile.am patch (947 bytes, patch)
2008-08-26 13:43 UTC, Damien Thébault
no flags Details | Splinter Review

Description Damien Thébault 2008-08-26 13:43:55 UTC
Created attachment 18528 [details] [review]
libXau 1.0.3 Makefile.am patch

If I run "make -n" in the libXau directory, I get the following error:

[...]
true '  [LD        ]  libXau.la'
/bin/sh ./libtool  --tag=CC --mode=link gcc -Wall -Wpointer-arith -Wstrict-prototypes 	-Wmissing-prototypes -Wmissing-declarations 	-Wnested-externs -fno-strict-aliasing    -g -O2   -o libXau.la -rpath /usr/local/lib -version-number 6:0:0 -no-undefined AuDispose.lo AuFileName.lo AuGetAddr.lo AuGetBest.lo AuLock.lo AuRead.lo AuUnlock.lo AuWrite.lo   
sed 's/__xorgversion__/"libXau 1.0.3" "X Version 11"/' \
	< ./Xau.man > Xau.3
make[1]: *** No rule to make target `XauFileName.3', needed by `all-am'.  Stop.
make[1]: Leaving directory `/tmp/libXau-1.0.3'
make: *** [all] Error 2

Then, if I look at Makefile.am, there is not rule for those files, only a "shadows.DONE" rule creating the needed files.
In addition, this shadows.DONE rule is just a shell loop, so it's really something that can be done with Makefile rules, with multiple benefits: rules requiring those files will not fail, make -n will not fail, and it'll be possible to create those files in parallel (but the speed is not really an issue).

See the attached patch, which provide a simple rule to create those files instead of using the shadows.DONE rule.
Comment 1 Alan Coopersmith 2008-08-26 14:20:40 UTC
Thanks - fix committed to git master for libXau 1.0.4 release:

commit 753bf38ceca4a0c8d6c6c1845b4387e4b3c9e4f0
Author: Damien Thébault <damien.thebault@gmail.com>
Date:   Tue Aug 26 14:16:44 2008 -0700

    Bug 17314: libXau's Makefile.am should have proper man creation rules
    
    X.Org Bugzilla #17314 <http://bugs.freedesktop.org/show_bug.cgi?id=17314>
    Patch #18528 <http://bugs.freedesktop.org/attachment.cgi?id=18528>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.