curl install breaks build directoey

[Tim Tassonis]

The install install instructions:

find docs \( -name Makefile\* \
          -o -name \*.1       \
          -o -name \*.3 \)    \
          -exec rm {} \;      &&
install -v -d -m755 /usr/share/doc/curl-7.47.0 &&
cp -v -R docs/*     /usr/share/doc/curl-7.47.0

works ok, but have the side effect that the build directory gets broken, e.g. a following make (or make install) will miss files and fail. Also, it will still copy a /usr/share/doc/curl-7.47.0/examples/.deps folder that is unneeded. Therefore I suggest to change this to:

install -v -d -m755 /usr/share/doc/curl-7.47.0 &&
cp -v -R docs/* /usr/share/doc/curl-7.47.0 &&
find /usr/share/doc/curl-7.47.0 \( -name Makefile\* \
-o -name \*.1 \
-o -name \*.3 \) \
-exec rm {} \; &&
rm -rf /usr/share/doc/curl-7.47.0/examples/.deps

This will leave the build directory unharmed and do the necessary cleanups in the destination directory.

[bdubbs@…]

I recommend marking this as wontfix. The assumption in LFS and BLFS is that you always start with a clean tarball extraction. If you want to deviate from that, you are on your own.

[Fernando de Oliveira]

Yes!

Thanks.

[Tim Tassonis]

Fine me with, although I don't understand the refusal to change one single line that won't harm anything. The reason I'm suggesting this is that one simple typo in those three lines will force you to fully unpack, ./configure and make the whole package again, only to be able to call "make install" again.

I won't bother commenting on your refusal to accept that it's a good idea to remove the unneeded .deps Directory in the resulting docucment...