brp-remove-la-files: Remove symlinks, too

As we check for file contents delete symlink before looking at regular
files.

Resolves: #3304

scripts/brp-remove-la-files

@@ -5,6 +5,12 @@ if [ -z "$RPM_BUILD_ROOT" ] || [ "$RPM_BUILD_ROOT" = "/" ]; then
   exit 0
 fi
 
-find "$RPM_BUILD_ROOT" -type f -name '*.la' 2>/dev/null -print0 |
-  xargs -0 grep --fixed-strings '.la - a libtool library file' --files-with-matches --null |
-  xargs -0 rm --force
+# Check and remove symlinks first then regular files
+for FILETYPE in l f
+do
+
+  find "$RPM_BUILD_ROOT" -type $FILETYPE -name '*.la' 2>/dev/null -print0 |
+    xargs -0 grep --fixed-strings '.la - a libtool library file' --files-with-matches --null |
+    xargs -0 rm --force
+
+done

tests/CMakeLists.txt

@@ -51,6 +51,7 @@ set(TESTSUITE_AT
 	rpmconfig.at
 	rpmconfig2.at
 	rpmconfig3.at
+	rpmbrp.at
 )
 
 FILE(REMOVE ${CMAKE_CURRENT_BINARY_DIR}/rpmtests.at)

tests/rpmbrp.at

@@ -0,0 +1,28 @@
+#    rpmbrp.at: rpm build root policy scripts tests
+
+AT_BANNER([RPM build root policy scripts])
+
+# ------------------------------
+# Check brp-remove-la-files
+AT_SETUP([brp-remove-la-files])
+AT_KEYWORDS([build brp])
+RPMTEST_CHECK([
+TD=/tmp/brp-remove-la-files
+
+runroot_other rm -rf $TD
+runroot_other mkdir -p $TD
+runroot_other echo ".la - a libtool library file" > $TD/test1.la
+runroot_other echo ".la - a not libtool library file" > $TD/test2.la
+runroot_other echo "other file" > $TD/test.txt
+runroot_other ln -s test1.la $TD/test.la
+runroot_other ln -s test2.la $TD/testfalse.la
+runroot_other --setenv RPM_BUILD_ROOT $TD ${RPM_CONFIGDIR_PATH}/brp-remove-la-files 
+runroot_other ls -1 $TD/*
+],
+[0],
+[/tmp/brp-remove-la-files/test.txt
+/tmp/brp-remove-la-files/test2.la
+/tmp/brp-remove-la-files/testfalse.la
+],
+[ignore])
+RPMTEST_CLEANUP