Gentoo Archives: gentoo-dev

From: "Michał Górny" <mgorny@g.o>
To: gentoo-dev@l.g.o
Cc: reavertm@g.o, "Michał Górny" <mgorny@g.o>
Subject: [gentoo-dev] [PATCH autotools-utils 9/9] Explain .la removal reasons in output.
Date: Tue, 13 Sep 2011 14:13:16
Message-Id: 1315923033-23397-9-git-send-email-mgorny@gentoo.org
In Reply to: [gentoo-dev] [PATCH autotools-utils 1/9] Fix handling whitespace in filenames when looking for .la files. by "Michał Górny"
1 ---
2 eclass/autotools-utils.eclass | 11 ++++++-----
3 1 files changed, 6 insertions(+), 5 deletions(-)
4
5 diff --git a/eclass/autotools-utils.eclass b/eclass/autotools-utils.eclass
6 index 2e01dcc..495244b 100644
7 --- a/eclass/autotools-utils.eclass
8 +++ b/eclass/autotools-utils.eclass
9 @@ -186,16 +186,17 @@ remove_libtool_files() {
10 # - they are covered by a .pc file already,
11 # - they don't provide any new information (no libs & no flags).
12 local removing
13 - if [[ ${removing_all} ]]; then removing=1
14 - elif [[ ! -f ${archivefile} ]]; then removing=1
15 - elif has "$(basename "${f}")" "${pc_libs[@]}"; then removing=1
16 + if [[ ${removing_all} ]]; then removing='forced'
17 + elif [[ ! -f ${archivefile} ]]; then removing='no static archive'
18 + elif has "$(basename "${f}")" "${pc_libs[@]}"; then
19 + removing='covered by .pc'
20 elif [[ ! $(sed -n -e \
21 "s/^\(dependency_libs\|inherited_linker_flags\)='\(.*\)'$/\2/p" \
22 - "${f}") ]]; then removing=1
23 + "${f}") ]]; then removing='no libs & flags'
24 fi
25
26 if [[ ${removing} ]]; then
27 - einfo "Removing unnecessary ${f#${D%/}}"
28 + einfo "Removing unnecessary ${f#${D%/}} (${removing})"
29 rm -f "${f}" || die
30 fi
31 done
32 --
33 1.7.6.1