[gentoo-portage-dev] [PATCH 2/3] ecompress: Fix duplicate pre-compressed file warning

[Michał Górny]

Fix the pre-compressed file warning to be reported only once, now that
files are processed in parallel.

Signed-off-by: Michał Górny <mgorny@g.o>
---
 bin/ecompress      |  6 ++++++
 bin/ecompress-file | 13 +++----------
 2 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/bin/ecompress b/bin/ecompress
index d5ff3796c..e4fdc3934 100755
--- a/bin/ecompress
+++ b/bin/ecompress
@@ -150,6 +150,12 @@ find "${ED}" -name '*.ecompress' -delete -print0 |
 	___parallel_xargs -0 "${PORTAGE_BIN_PATH}"/ecompress-file
 ret=${?}
 
+if [[ -f ${T}/.ecompress_had_precompressed ]]; then
+	eqawarn "One or more compressed files were found in docompress-ed directories."
+	eqawarn "Please fix the ebuild not to install compressed files (manpages,"
+	eqawarn "documentation) when automatic compression is used."
+fi
+
 fix_symlinks
 : $(( ret |= ${?} ))
 [[ $ret -ne 0 ]] && __helpers_die "${0##*/} failed"
diff --git a/bin/ecompress-file b/bin/ecompress-file
index 18269c91b..27695da1b 100755
--- a/bin/ecompress-file
+++ b/bin/ecompress-file
@@ -14,7 +14,6 @@ compress_file() {
 	set +f
 	mask_ext_re="^(${mask_ext_re:1})\$"
 	local filtered_args=()
-	local had_precompressed=
 	for x in "$@" ; do
 		[[ ${x##*.} =~ $mask_ext_re ]] && continue
 		[[ -s ${x} ]] || continue
@@ -24,15 +23,15 @@ compress_file() {
 			*.gz|*.Z)
 				gunzip -f "${x}" || __helpers_die "gunzip failed"
 				x=${x%.*}
-				had_precompressed=1;;
+				touch "${T}/.ecompress_had_precompressed";;
 			*.bz2)
 				bunzip2 -f "${x}" || __helpers_die "bunzip2 failed"
 				x=${x%.bz2}
-				had_precompressed=1;;
+				touch "${T}/.ecompress_had_precompressed";;
 			*.lzma|*.xz)
 				unxz -f "${x}" || __helpers_die "unxz failed"
 				x=${x%.*}
-				had_precompressed=1;;
+				touch "${T}/.ecompress_had_precompressed";;
 		esac
 
 		filtered_args+=( "$x" )
@@ -40,12 +39,6 @@ compress_file() {
 	[[ ${#filtered_args[@]} -eq 0 ]] && return 0
 	set -- "${filtered_args[@]}"
 
-	if [[ ${had_precompressed} ]]; then
-		eqawarn "One or more compressed files were found in docompress-ed directories."
-		eqawarn "Please fix the ebuild not to install compressed files (manpages,"
-		eqawarn "documentation) when automatic compression is used."
-	fi
-
 	# If a compressed version of the file already exists, simply
 	# delete it so that the compressor doesn't whine (bzip2 will
 	# complain and skip, gzip will prompt for input)
-- 
2.19.0