Re: [PATCH] Update listingblock monospace fix to support all docbook versions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 20 Jul 2007, Brian Gernhardt wrote:


On Jul 20, 2007, at 8:06 AM, Julian Phillips wrote:

+while ($line = <>) {
+	$line =~ s/^#GIT#SET#MAN#FONT#/.ft/;
+	print $line;
+}

Why not use `perl -pe "s/^#GIT#SET#MAN#FONT#/.ft/"` or an equivalent sed command in the Makefile instead of having this as a separate script?

Only because my perl foo is too weak to know the perl form, and I didn't know how to get sed to replace unicode characters - which the original version of the script did. I rewrote the script in my lunchbreak after I failed to find any better way to solve the problem, and it didn't occur to me to move it to sed at that point.

However, given the other issues with docbook xsl 1.72 I think it would cleaner to simply say that git doesn't support 1.72. I wouldn't be surprised if other projects weren't taking that approach.

--
Julian

 ---
It's computer hardware, of course it's worth having <g>
	-- Espy on #Debian
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux