[PATCH] Update manpage with the proper location of system.twmrc file

Alan Coopersmith alan.coopersmith at oracle.com
Mon Aug 4 08:52:08 PDT 2014


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 <lordheavym at gmail.com>
>>> ---
>>>   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-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list