Date: Sat, 7 Aug 2004 02:29:18 +0300 From: Giorgos Keramidas <keramida@ceid.upatras.gr> To: Peter Pentchev <roam@ringlet.net> Cc: freebsd-doc@freebsd.org Subject: Re: docs/69086: Porters Handbook: How to convert from CR/LF to LF using REINPLACE_CMD Message-ID: <20040806232918.GC79083@gothmog.gr> In-Reply-To: <200408061550.i76FoTY1013972@freefall.freebsd.org> References: <200408061550.i76FoTY1013972@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Nice :)
xargs is one of my favorite UNIX tools too. If this builds fine
(I haven't tested it, just gave it a quick look), then please commit it.
On 2004-08-06 15:50, Peter Pentchev <roam@ringlet.net> wrote:
> Or how about the following, which uses the &windows; entity properly? :)
>
> G'luck,
> Peter
>
> Index: doc/en_US.ISO8859-1/books/porters-handbook/book.sgml
> ===================================================================
> RCS file: /home/ncvs/doc/en_US.ISO8859-1/books/porters-handbook/book.sgml,v
> retrieving revision 1.470
> diff -u -r1.470 book.sgml
> --- doc/en_US.ISO8859-1/books/porters-handbook/book.sgml 5 Aug 2004 10:04:30 -0000 1.470
> +++ doc/en_US.ISO8859-1/books/porters-handbook/book.sgml 6 Aug 2004 15:40:46 -0000
> @@ -18,6 +18,8 @@
> %mailing-lists;
> <!ENTITY % freebsd PUBLIC "-//FreeBSD//ENTITIES DocBook Miscellaneous FreeBSD Entities//EN">
> %freebsd;
> +<!ENTITY % trademarks PUBLIC "-//FreeBSD//ENTITIES DocBook Trademark Entities//EN">
> +%trademarks;
> <!ENTITY % urls PUBLIC "-//FreeBSD//ENTITIES DocBook URL Entities//EN">
> %urls;
> ]>
> @@ -682,6 +684,25 @@
> lines!); define <literal>USE_AUTOCONF_VER=213</literal> and take the
> diffs of <filename>configure.in</filename>.</para>
>
> + <para>Quite often, there is a situation when the software being
> + ported, especially if it is primarily developed on &windows;, uses
> + the CR/LF convention for most of its source files. This may cause
> + problems with further patching, compiler warnings, scripts
> + execution (<command>/bin/sh^M</command> not found), etc. To
> + quickly convert those files from CR/LF to just LF, you can do
> + something like this:</para>
> +
> + <programlisting>USE_REINPLACE= yes
> +
> +post-extract:
> + @${FIND} -E ${WRKDIR} -type f -iregex ".*\.(c|cpp|h|txt)" -print0 | \
> + ${XARGS} -0 ${REINPLACE_CMD} -e 's/[[:cntrl:]]*$$//' '{}' \;</programlisting>
> +
> + <para>Of course, if you need to process each and every file,
> + <option>-iregex</option> above can be omitted. Be aware that this
> + piece of code will strip all trailing control characters from each
> + line of processed file (except <literal>\n</literal>).</para>
> +
> <para>Also, if you had to delete a file, then you can do it in the
> <maketarget>post-extract</maketarget> target rather than as part of
> the patch. Once you are happy with the resulting diff, please split
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040806232918.GC79083>