Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#7432 closed defect (wontfix)

curl install breaks build directoey

Reported by: Tim Tassonis Owned by: blfs-book@…
Priority: low Milestone: 7.9
Component: BOOK Version: SVN
Severity: minor Keywords:
Cc:

Description

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.

Change History (3)

comment:1 by bdubbs@…, 6 years ago

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.

comment:2 by Fernando de Oliveira, 6 years ago

Resolution: wontfix
Status: newclosed

Yes!

Thanks.

comment:3 by Tim Tassonis, 6 years ago

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...

Note: See TracTickets for help on using tickets.