On 08/ 4/14 08:36 AM, Thomas Klausner wrote:
On Mon, Aug 04, 2014 at 04:48:58PM +0200, walter harms wrote:
Am 04.08.2014 16:39, schrieb Laurent Carlier:
Signed-off-by: Laurent Carlier <[email protected]>
---
man/Makefile.am | 2 ++
man/twm.man | 2 +-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/man/Makefile.am b/man/Makefile.am
index 7d45968..f59d5b5 100644
--- a/man/Makefile.am
+++ b/man/Makefile.am
@@ -33,6 +33,8 @@ CLEANFILES = $(appman_DATA)
SUFFIXES = .$(APP_MAN_SUFFIX) .man
+MAN_SUBSTS += -e 's|__datadir__|$(datadir)|g'
+
I never noticed that one can use "|" as replacement for "/" in sed.
Since the documentation of sed describes "/", why "|" in this case ?
You can use any letter instead of '/'.
'|' is often used when replacing paths, since it's not very common in them :)
Right - since this case is going to expand to a path containing / characters,
using | avoids having to find some way to escape the / in $(datadir) when
make substitutes it. '|' is used all over the X makefiles for pathname
substitutions with sed for this reason, so this fits our style.
--
-Alan Coopersmith- [email protected]
Oracle Solaris Engineering - http://blogs.oracle.com/alanc
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
