On Thu, Apr 18, 2024 at 09:55:20AM -0600, Anthony J. Bentley wrote:
> Stuart Henderson writes:
> > Perhaps consider replacing pre-patch with this which I think is a little
> > clearer:
> >
> > FIX_CRLF_FILES = `find . -type f`
>
> Can this be tweaked to limit it to text files, without making it too
> complicated?
>
> I only see a single binary file in the distfile, but still...
I see this message now after I already committed the diff. I think
that makes sense, yes. Limiting it to .txt, .c and .h files (the files
we're patching) also works.
Comments or OK?
Caspar
Index: Makefile
===================================================================
RCS file: /cvs/ports/converters/k2pdfopt/Makefile,v
diff -u -p -r1.13 Makefile
--- Makefile 18 Apr 2024 16:15:44 -0000 1.13
+++ Makefile 18 Apr 2024 16:28:35 -0000
@@ -38,7 +38,7 @@ RUN_DEPENDS += print/ghostscript/gnu
WRKDIST = ${WRKDIR}/k2pdfopt_v${V}
-FIX_CRLF_FILES = `find . -type f`
+FIX_CRLF_FILES = `find . -type f \( -name '*.txt' -o -name '*.[ch]' \)`
post-install:
${INSTALL_MAN} ${WRKDIR}/debian/k2pdfopt.1 ${PREFIX}/man/man1
