sys-kernel/linux-firmware: Flip order of if blocks in src_prepare.
authorUlrich Müller <ulm@gentoo.org>
Sun, 19 May 2019 06:34:37 +0000 (08:34 +0200)
committerUlrich Müller <ulm@gentoo.org>
Sun, 19 May 2019 06:34:37 +0000 (08:34 +0200)
Otherwise, the second rm will fail because of previously removed files.
Remove the ewarn message which isn't useful after separation of flags.

Package-Manager: Portage-2.3.66, Repoman-2.3.12
Signed-off-by: Ulrich Müller <ulm@gentoo.org>
sys-kernel/linux-firmware/linux-firmware-20190514.ebuild

index 45c50b3372de54e3d5c9466519028a4b62e0cd6c..d89f8c43e2e47e331b2295d096859d48ea5654f2 100644 (file)
@@ -228,6 +228,11 @@ src_prepare() {
        # remove sources and documentation (wildcards are expanded)
        rm -r ${source_files[@]} || die
 
+       if use !unknown-license; then
+               # remove files in the unknown_license blacklist
+               rm "${unknown_license[@]}" || die
+       fi
+
        if use !redistributable; then
                # remove files _not_ in the free_software whitelist
                local file remove=()
@@ -235,15 +240,6 @@ src_prepare() {
                        has "${file#./}" "${free_software[@]}" || remove+=("${file}")
                done < <(find * ! -type d -print0 || die)
                printf "%s\0" "${remove[@]}" | xargs -0 rm || die
-
-               if use unknown-license; then
-                       ewarn 'The "unknown-license" flag is set, while "-redistributable"'
-                       ewarn 'asks for free software only. Ignoring "unknown-license".'
-               fi
-       fi
-       if use !unknown-license; then
-               # remove files in the unknown_license blacklist
-               rm "${unknown_license[@]}" || die
        fi
 
        echo "# Remove files that shall not be installed from this list." > ${PN}.conf