Fix --exclude-ignore option.

This fixes https://savannah.gnu.org/bugs/?64387.

* src/exclist.c (info_attach_exclist): Always use file->flags.
(exclude_vcs_ignores): Pass flags from struct vcs_ignore_file
to excfile_add.

* tests/exclude19.at: New test.
* tests/exclude20.at: New test.
* tests/Makefile.am: Add new tests.
* tests/testsuite.at: Add new tests.

diff --git a/src/exclist.c b/src/exclist.c
index 967e74ad112337f3b66e7749e61d4fc1491d3a3d..b4e0f9191741e6ffdbd55e43707110b2f19fee4f 100644 (file)
--- a/src/exclist.c
+++ b/src/exclist.c
@@ -118,8 +118,7 @@ info_attach_exclist (struct tar_stat_info *dir)
 
          ent = xmalloc (sizeof (*ent));
          ent->excluded = ex;
-         ent->flags = file->flags == EXCL_DEFAULT
-                      ? file->flags : vcsfile->flags;
+         ent->flags = file->flags;
          ent->prev = tail;
          ent->next = NULL;
 
@@ -333,5 +332,5 @@ exclude_vcs_ignores (void)
   struct vcs_ignore_file *p;
 
   for (p = vcs_ignore_files; p->filename; p++)
-    excfile_add (p->filename, EXCL_DEFAULT);
+    excfile_add (p->filename, p->flags);
 }

diff --git a/tests/Makefile.am b/tests/Makefile.am
index fde5e918573b1d0165c17ee1d2f6ca9a232eb2f9..5745f69c48d384acc7309702f5784269f03525ce 100644 (file)
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -108,6 +108,8 @@ TESTSUITE_AT = \
  exclude16.at\
  exclude17.at\
  exclude18.at\
+ exclude19.at\
+ exclude20.at\
  extrac01.at\
  extrac02.at\
  extrac03.at\

diff --git a/tests/exclude18.at b/tests/exclude18.at
index c5be12c3b6a29ba7fa1dcff540b5aaf60ae6f806..d232c8c44f72b2f3b32ca9b4ebc7a1af802f0adb 100644 (file)
--- a/tests/exclude18.at
+++ b/tests/exclude18.at
@@ -28,7 +28,7 @@
 # extracting.
 
 AT_SETUP([--exclude-vcs extract list compare])
-AT_KEYWORDS([exclude-vcs extract list compare exclude18])
+AT_KEYWORDS([exclude exclude-vcs extract list compare exclude18])
 
 AT_TAR_CHECK([
 AT_SORT_PREREQ

diff --git a/tests/exclude19.at b/tests/exclude19.at
new file mode 100644 (file)
index 0000000..9422607
--- /dev/null
+++ b/tests/exclude19.at
@@ -0,0 +1,47 @@
+# Process this file with autom4te to create testsuite. -*- Autotest -*-
+
+# Test suite for GNU tar.
+# Copyright 2024 Free Software Foundation, Inc.
+
+# This file is part of GNU tar.
+
+# GNU tar is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+
+# GNU tar is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+# Test --exclude-ignore option.
+# Ref: https://savannah.gnu.org/bugs/?64387
+
+AT_SETUP([--exclude-ignore])
+AT_KEYWORDS([exclude exclude-ignore extract exclude19])
+
+AT_TAR_CHECK([
+AT_SORT_PREREQ
+mkdir dir dir/subdir
+echo 1 > dir/file.ext
+echo 2 > dir/subdir/file2.ext
+echo '*.ext' > dir/ignorefile
+
+tar -cf archive.tar --exclude-ignore=ignorefile dir
+rm -rf dir
+tar xfv archive.tar | sort
+],
+[0],
+[dir/
+dir/ignorefile
+dir/subdir/
+dir/subdir/file2.ext
+])
+
+AT_CLEANUP
+
+

diff --git a/tests/exclude20.at b/tests/exclude20.at
new file mode 100644 (file)
index 0000000..0b1b681
--- /dev/null
+++ b/tests/exclude20.at
@@ -0,0 +1,43 @@
+# Process this file with autom4te to create testsuite. -*- Autotest -*-
+
+# Test suite for GNU tar.
+# Copyright 2024 Free Software Foundation, Inc.
+
+# This file is part of GNU tar.
+
+# GNU tar is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 3 of the License, or
+# (at your option) any later version.
+
+# GNU tar is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+AT_SETUP([--exclude-ignore-recursive])
+AT_KEYWORDS([exclude exclude-ignore-recursive extract exclude20])
+
+AT_TAR_CHECK([
+AT_SORT_PREREQ
+mkdir dir dir/subdir
+echo 1 > dir/file.ext
+echo 2 > dir/subdir/file2.ext
+echo '*.ext' > dir/ignorefile
+
+tar -cf archive.tar --exclude-ignore-recursive=ignorefile dir
+rm -rf dir
+tar xfv archive.tar | sort
+],
+[0],
+[dir/
+dir/ignorefile
+dir/subdir/
+])
+
+AT_CLEANUP
+
+

diff --git a/tests/testsuite.at b/tests/testsuite.at
index 935352e27133944227cdbfba663e0363b734db89..8a0d39c31a3ef4b6fe0687bd677f6f5ab3e5cc8a 100644 (file)
--- a/tests/testsuite.at
+++ b/tests/testsuite.at
@@ -315,6 +315,8 @@ m4_include([exclude15.at])
 m4_include([exclude16.at])
 m4_include([exclude17.at])
 m4_include([exclude18.at])
+m4_include([exclude19.at])
+m4_include([exclude20.at])
 
 AT_BANNER([Deletions])
 m4_include([delete01.at])