]> git.proxmox.com Git - mirror_zfs.git/commitdiff
Replace EXTRA_DIST with dist_noinst_DATA
authorBrian Behlendorf <behlendorf1@llnl.gov>
Thu, 26 May 2022 16:24:50 +0000 (09:24 -0700)
committerGitHub <noreply@github.com>
Thu, 26 May 2022 16:24:50 +0000 (09:24 -0700)
The EXTRA_DIST variable is ignored when used in the FALSE conditional
of a Makefile.am.  This results in the `make dist` target omitting
these files from the generated tarball unless CONFIG_USER is defined.
This issue can be avoided by switching to use the dist_noinst_DATA
variable which is handled as expected by autoconf.

This change also adds support for --with-config=dist as an alias
for --with-config=srpm and updates the GitHub workflows to use it.

Reviewed-by: Ahelenia ZiemiaƄska <nabijaczleweli@nabijaczleweli.xyz>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #13459
Closes #13505

27 files changed:
.github/workflows/zfs-tests-functional.yml
.github/workflows/zfs-tests-sanity.yml
.github/workflows/zloop.yml
Makefile.am
cmd/Makefile.am
cmd/zed/Makefile.am
cmd/zed/zed.d/Makefile.am
cmd/zpool/Makefile.am
config/Substfiles.am
config/zfs-build.m4
contrib/bpftrace/Makefile.am
contrib/dracut/Makefile.am
contrib/initramfs/Makefile.am
contrib/pyzfs/Makefile.am
contrib/zcp/Makefile.am
etc/Makefile.am
lib/libnvpair/Makefile.am
lib/libuutil/Makefile.am
lib/libzfs/Makefile.am
lib/libzfs_core/Makefile.am
lib/libzfsbootenv/Makefile.am
man/Makefile.am
rpm/Makefile.am
scripts/Makefile.am
tests/Makefile.am
tests/zfs-tests/cmd/Makefile.am
tests/zfs-tests/tests/Makefile.am

index 0273610af0453ad7c66e5f50b228477bfa9199df..328cb97f10e4ed166cae3235bd2536a1aa03b7ad 100644 (file)
@@ -28,7 +28,7 @@ jobs:
         ./autogen.sh
     - name: Configure
       run: |
-        ./configure --enable-debug --enable-debuginfo --enable-asan --enable-ubsan
+        ./configure --enable-debug --enable-debuginfo --enable-asan --enable-ubsan --with-config=dist
     - name: Make
       run: |
         make -j$(nproc) --no-print-directory --silent pkg-utils pkg-kmod
index 73606f909e10b1c9bfd889064cd0d5ff727fa1b2..4c15cecf58d8dcf4a0372691a825ccf0bf4eda0d 100644 (file)
@@ -24,7 +24,7 @@ jobs:
         ./autogen.sh
     - name: Configure
       run: |
-        ./configure --enable-debug --enable-debuginfo --enable-asan --enable-ubsan
+        ./configure --enable-debug --enable-debuginfo --enable-asan --enable-ubsan --with-config=dist
     - name: Make
       run: |
         make -j$(nproc) --no-print-directory --silent pkg-utils pkg-kmod
index d49eeae1653c2009f1b18be8c5789947017c1877..64fe96a3ab6b3e837bc92ae53dbb66ebdb89b9b3 100644 (file)
@@ -23,7 +23,7 @@ jobs:
         ./autogen.sh
     - name: Configure
       run: |
-        ./configure --enable-debug --enable-debuginfo --enable-asan --enable-ubsan
+        ./configure --enable-debug --enable-debuginfo --enable-asan --enable-ubsan --with-config=dist
     - name: Make
       run: |
         make -j$(nproc) --no-print-directory --silent pkg-utils pkg-kmod
index 0a73a63870bc42c22d5d0854664438c533139c96..54d300e7d40be8e385f331d155c24e30c334353c 100644 (file)
@@ -1,5 +1,5 @@
 CLEANFILES =
-EXTRA_DIST =
+dist_noinst_DATA =
 INSTALL_DATA_HOOKS =
 ALL_LOCAL =
 CLEAN_LOCAL =
@@ -37,26 +37,26 @@ extradir = $(prefix)/src/zfs-$(VERSION)
 extra_HEADERS = zfs.release.in zfs_config.h.in
 endif
 
-EXTRA_DIST += autogen.sh copy-builtin
-EXTRA_DIST += AUTHORS CODE_OF_CONDUCT.md COPYRIGHT LICENSE META NEWS NOTICE
-EXTRA_DIST += README.md RELEASES.md
-EXTRA_DIST += module/lua/README.zfs module/os/linux/spl/README.md
+dist_noinst_DATA += autogen.sh copy-builtin
+dist_noinst_DATA += AUTHORS CODE_OF_CONDUCT.md COPYRIGHT LICENSE META NEWS NOTICE
+dist_noinst_DATA += README.md RELEASES.md
+dist_noinst_DATA += module/lua/README.zfs module/os/linux/spl/README.md
 
 # Include all the extra licensing information for modules
-EXTRA_DIST += module/icp/algs/skein/THIRDPARTYLICENSE
-EXTRA_DIST += module/icp/algs/skein/THIRDPARTYLICENSE.descrip
-EXTRA_DIST += module/icp/asm-x86_64/aes/THIRDPARTYLICENSE.gladman
-EXTRA_DIST += module/icp/asm-x86_64/aes/THIRDPARTYLICENSE.gladman.descrip
-EXTRA_DIST += module/icp/asm-x86_64/aes/THIRDPARTYLICENSE.openssl
-EXTRA_DIST += module/icp/asm-x86_64/aes/THIRDPARTYLICENSE.openssl.descrip
-EXTRA_DIST += module/icp/asm-x86_64/modes/THIRDPARTYLICENSE.cryptogams
-EXTRA_DIST += module/icp/asm-x86_64/modes/THIRDPARTYLICENSE.cryptogams.descrip
-EXTRA_DIST += module/icp/asm-x86_64/modes/THIRDPARTYLICENSE.openssl
-EXTRA_DIST += module/icp/asm-x86_64/modes/THIRDPARTYLICENSE.openssl.descrip
-EXTRA_DIST += module/os/linux/spl/THIRDPARTYLICENSE.gplv2
-EXTRA_DIST += module/os/linux/spl/THIRDPARTYLICENSE.gplv2.descrip
-EXTRA_DIST += module/zfs/THIRDPARTYLICENSE.cityhash
-EXTRA_DIST += module/zfs/THIRDPARTYLICENSE.cityhash.descrip
+dist_noinst_DATA += module/icp/algs/skein/THIRDPARTYLICENSE
+dist_noinst_DATA += module/icp/algs/skein/THIRDPARTYLICENSE.descrip
+dist_noinst_DATA += module/icp/asm-x86_64/aes/THIRDPARTYLICENSE.gladman
+dist_noinst_DATA += module/icp/asm-x86_64/aes/THIRDPARTYLICENSE.gladman.descrip
+dist_noinst_DATA += module/icp/asm-x86_64/aes/THIRDPARTYLICENSE.openssl
+dist_noinst_DATA += module/icp/asm-x86_64/aes/THIRDPARTYLICENSE.openssl.descrip
+dist_noinst_DATA += module/icp/asm-x86_64/modes/THIRDPARTYLICENSE.cryptogams
+dist_noinst_DATA += module/icp/asm-x86_64/modes/THIRDPARTYLICENSE.cryptogams.descrip
+dist_noinst_DATA += module/icp/asm-x86_64/modes/THIRDPARTYLICENSE.openssl
+dist_noinst_DATA += module/icp/asm-x86_64/modes/THIRDPARTYLICENSE.openssl.descrip
+dist_noinst_DATA += module/os/linux/spl/THIRDPARTYLICENSE.gplv2
+dist_noinst_DATA += module/os/linux/spl/THIRDPARTYLICENSE.gplv2.descrip
+dist_noinst_DATA += module/zfs/THIRDPARTYLICENSE.cityhash
+dist_noinst_DATA += module/zfs/THIRDPARTYLICENSE.cityhash.descrip
 
 @CODE_COVERAGE_RULES@
 
index 9959760b523eab7e37ecf794439c9c4dc9983cbb..65de980da308b818fa739277b5da00b9f404bdaa 100644 (file)
@@ -10,7 +10,7 @@ mounthelper_PROGRAMS =
 sbin_SCRIPTS      += fsck.zfs
 SHELLCHECKSCRIPTS += fsck.zfs
 CLEANFILES        += fsck.zfs
-EXTRA_DIST        += %D%/fsck.zfs.in
+dist_noinst_DATA  += %D%/fsck.zfs.in
 $(call SUBST,fsck.zfs,%D%/)
 
 
@@ -100,9 +100,9 @@ endif
 
 
 if USING_PYTHON
-bin_SCRIPTS += arc_summary     arcstat        dbufstat
-CLEANFILES  += arc_summary     arcstat        dbufstat
-EXTRA_DIST  += %D%/arc_summary %D%/arcstat.in %D%/dbufstat.in
+bin_SCRIPTS      += arc_summary     arcstat        dbufstat
+CLEANFILES       += arc_summary     arcstat        dbufstat
+dist_noinst_DATA += %D%/arc_summary %D%/arcstat.in %D%/dbufstat.in
 
 $(call SUBST,arcstat,%D%/)
 $(call SUBST,dbufstat,%D%/)
index 6eea1a135f56298b658d61e2470de3876752ba66..c437ff51dd2b09e2f458c243fcd11d72c2bb7ecc 100644 (file)
@@ -43,4 +43,4 @@ zed_LDADD = \
 zed_LDADD += -lrt $(LIBATOMIC_LIBS) $(LIBUDEV_LIBS) $(LIBUUID_LIBS)
 zed_LDFLAGS = -pthread
 
-EXTRA_DIST += $(addprefix %D%/,agents/README.md)
+dist_noinst_DATA += %D%/agents/README.md
index 122d2c0c51bccbe329482e3f9f35c8a88d8d0356..c65b43fb027eadd627be4b3b614f1b1acd5ded24 100644 (file)
@@ -38,7 +38,7 @@ zedconfdefaults = \
        vdev_attach-led.sh \
        vdev_clear-led.sh
 
-EXTRA_DIST += $(addprefix %D%/,README)
+dist_noinst_DATA += %D%/README
 
 INSTALL_DATA_HOOKS += zed-install-data-hook
 zed-install-data-hook:
index 9ba72eadf3a7953ea0d466ccc4d375f22283c7bd..3c7c8a9aebe260d4e077d9945aa5d2ba7a2f8a50 100644 (file)
@@ -38,7 +38,7 @@ zpool_LDADD += -lgeom
 endif
 zpool_LDADD += -lm $(LIBBLKID_LIBS) $(LIBUUID_LIBS)
 
-EXTRA_DIST += $(addprefix %D%/,zpool.d/README compatibility.d)
+dist_noinst_DATA += %D%/zpool.d/README
 
 SHELLCHECKSCRIPTS += $(dist_zpoolexec_SCRIPTS)
 zpoolexecdir = $(zfsexecdir)/zpool.d
index 733073dd20ca649fe8d7734875780e1529fc9514..38e870b2f501a5ce07fa676cf09d3cace68dcae2 100644 (file)
@@ -41,6 +41,6 @@ endef
 
 SUBSTFILES =
 CLEANFILES += $(SUBSTFILES)
-EXTRA_DIST += $(SUBSTFILES:=.in)
+dist_noinst_DATA += $(SUBSTFILES:=.in)
 
 $(call SUBST,%,)
index 582c479f5b5bc0fa7d23b383c374d8bbdc891f41..b40776da7a70714f0944a49a1a746ef816e8478d 100644 (file)
@@ -268,6 +268,7 @@ AC_DEFUN([ZFS_AC_CONFIG], [
                user)   ZFS_AC_CONFIG_USER   ;;
                all)    ZFS_AC_CONFIG_USER
                        ZFS_AC_CONFIG_KERNEL ;;
+               dist)                        ;;
                srpm)                        ;;
                *)
                AC_MSG_RESULT([Error!])
index 5382f1e1de420098a5e9e4076b9589670e32ee82..4f649cf5433eb53d9373a9c28f2d39e7c6047f10 100644 (file)
@@ -1,5 +1,3 @@
-EXTRA_DIST += $(addprefix %D%/, \
-       taskqlatency.bt \
-       zfs-trace.sh)
+dist_noinst_DATA += %D%/taskqlatency.bt %D%/zfs-trace.sh
 
 SHELLCHECKSCRIPTS += %D%/zfs-trace.sh
index f0c103b0a4da2ef36e18e5510aa208ed6b515512..73ca52b66316c054f806c1777794a8e8a0226401 100644 (file)
@@ -24,4 +24,4 @@ SHELLCHECKSCRIPTS += $(pkgdracut_02_SCRIPTS) $(pkgdracut_90_SCRIPTS)
 # Provided by /bin/sleep, and, again, every implementation of that supports this
 $(call SHELLCHECK_OPTS,$(pkgdracut_90_SCRIPTS)): CHECKBASHISMS_IGNORE = -e 'sleep only takes one integer' -e 'sleep 0.'
 
-EXTRA_DIST += $(addprefix %D%/,README.md)
+dist_noinst_DATA += %D%/README.md
index 6917b517f33088b89c41b7ae2fb53225c1e2f262..a583341ea2b14c04a02316d6a6d1389e9157ff11 100644 (file)
@@ -36,4 +36,4 @@ SHELLCHECKSCRIPTS   += $(i_t_check_scripts)
 $(call SHELLCHECK_OPTS,$(i_t_check_scripts)): SHELLCHECK_SHELL = sh
 
 
-EXTRA_DIST += $(addprefix %D%/,README.md)
+dist_noinst_DATA += %D%/README.md
index 505cd3c8b35564d9e809450609cf6f2126438004..06d9a09d7f1eec1fd6cbc769c0182abec62bcbe2 100644 (file)
@@ -1,4 +1,4 @@
-EXTRA_DIST += $(addprefix %D%/,libzfs_core README LICENSE docs)
+dist_noinst_DATA += %D%/libzfs_core %D%/README %D%/LICENSE %D%/docs
 SUBSTFILES += %D%/setup.py
 
 if PYZFS_ENABLED
index 759a9d529dc8eb919861d833e79dad5f7c7b479f..fc3f01a233cb55d326b3cc18579fac0092a798eb 100644 (file)
@@ -1 +1 @@
-EXTRA_DIST += $(addprefix %D%/,autosnap.lua)
+dist_noinst_DATA += %D%/autosnap.lua
index a41f04c90420ece90b417c088c909f47a9b413ec..53064eb6f6ea350c068c40fc446db7d871bbd4d8 100644 (file)
@@ -2,7 +2,7 @@ sudoersddir = $(sysconfdir)/sudoers.d
 sudoersd_DATA = \
        %D%/sudoers.d/zfs
 
-EXTRA_DIST += $(sudoersd_DATA)
+dist_noinst_DATA += $(sudoersd_DATA)
 
 
 sysconf_zfsdir = $(sysconfdir)/zfs
@@ -32,7 +32,7 @@ $(call SHELLCHECK_OPTS,$(initconf_DATA)): SHELLCHECK_SHELL = sh
 
 
 if INIT_SYSV
-EXTRA_DIST += $(addprefix %D%/,init.d/README.md)
+dist_noinst_DATA += %D%/init.d/README.md
 
 init_SCRIPTS = \
        %D%/init.d/zfs-import \
index 6d9345f5289e3970cfc8bf8b25c92295e183c0cf..87b8d32aa175e1524417be4a5fbe801441323243 100644 (file)
@@ -32,4 +32,4 @@ endif
 
 libnvpair_la_LDFLAGS += -version-info 3:0:0
 
-EXTRA_DIST += $(addprefix %D%/,libnvpair.abi libnvpair.suppr)
+dist_noinst_DATA += %D%/libnvpair.abi %D%/libnvpair.suppr
index cb89d423dc480d4d6a4bbf2681903fabf1997840..339f9a0647453fc265ab44a1c8c76699915ed526 100644 (file)
@@ -26,4 +26,4 @@ endif
 
 libuutil_la_LDFLAGS += -version-info 3:0:0
 
-EXTRA_DIST += $(addprefix %D%/,libuutil.abi libuutil.suppr)
+dist_noinst_DATA += %D%/libuutil.abi %D%/libuutil.suppr
index 07414c4a308b86c9ceba7182358fd0eb3ea6336a..f5eb84679204535edd3eeb84c6cd51d1e9cc77bc 100644 (file)
@@ -76,5 +76,5 @@ libzfs_la_LDFLAGS += -version-info 5:0:1
 
 pkgconfig_DATA += %D%/libzfs.pc
 
-EXTRA_DIST += $(addprefix %D%/,libzfs.abi libzfs.suppr)
-EXTRA_DIST += $(addprefix %D%/,THIRDPARTYLICENSE.openssl THIRDPARTYLICENSE.openssl.descrip)
+dist_noinst_DATA += %D%/libzfs.abi %D%/libzfs.suppr
+dist_noinst_DATA += %D%/THIRDPARTYLICENSE.openssl %D%/THIRDPARTYLICENSE.openssl.descrip
index 9370da4e3e74f0c6e23594331de0d0d5678df0a5..d1c6fb86d186626ebc2792161236790239c606e7 100644 (file)
@@ -43,4 +43,4 @@ libzfs_core_la_LDFLAGS += -version-info 3:0:0
 
 pkgconfig_DATA += %D%/libzfs_core.pc
 
-EXTRA_DIST += $(addprefix %D%/,libzfs_core.abi libzfs_core.suppr)
+dist_noinst_DATA += %D%/libzfs_core.abi %D%/libzfs_core.suppr
index 7b87bfe6786bc62403d1b96b7c945e8d4738b3d2..118f154821fcbd2c4e376c0de6b77579a437bee6 100644 (file)
@@ -26,4 +26,4 @@ libzfsbootenv_la_LDFLAGS += -version-info 1:0:0
 
 pkgconfig_DATA += %D%/libzfsbootenv.pc
 
-EXTRA_DIST += $(addprefix %D%/,libzfsbootenv.abi libzfsbootenv.suppr)
+dist_noinst_DATA += %D%/libzfsbootenv.abi %D%/libzfsbootenv.suppr
index 362ee9b7e149aab7893863b38ec70f5f1c26a9a0..8fa21d2fd23efc82baddecb36d1343760043caf4 100644 (file)
@@ -1,4 +1,4 @@
-EXTRA_DIST += \
+dist_noinst_man_MANS = \
        %D%/man1/cstyle.1
 
 dist_man_MANS = \
@@ -105,8 +105,9 @@ nodist_man_MANS = \
        %D%/man8/zed.8 \
        %D%/man8/zfs-mount-generator.8
 
-SUBSTFILES += $(nodist_man_MANS)
+dist_noinst_DATA += $(dist_noinst_man_MANS) $(dist_man_MANS)
 
+SUBSTFILES += $(nodist_man_MANS)
 
 CHECKS += mancheck
 mancheck:
index 216cb731fea96eea5e91bddb2cdcab55dc6a0839..af7b25021eac8b7036223c8e9f19b9821ab3b8cc 100644 (file)
@@ -1,4 +1,4 @@
-EXTRA_DIST += \
+dist_noinst_DATA += \
        %D%/generic/zfs-dkms.spec.in \
        %D%/generic/zfs-kmod.spec.in \
        %D%/generic/zfs.spec.in \
index 6f1dc9ccdba5f3df3817c0d8955e7c57b5c4a6fe..79719e621b69237e94b603bec5330432eaabde83 100644 (file)
@@ -6,7 +6,7 @@ dist_scripts_SCRIPTS = \
        %D%/zimport.sh \
        %D%/zloop.sh
 
-EXTRA_SCRIPTS = \
+dist_noinst_SCRIPTS = \
        %D%/commitcheck.sh \
        %D%/common.sh.in \
        %D%/dkms.mkconf \
@@ -18,14 +18,13 @@ EXTRA_SCRIPTS = \
        %D%/paxcheck.sh \
        %D%/zfs-tests-color.sh
 
-EXTRA_DIST += \
+dist_noinst_DATA += \
        %D%/cstyle.pl \
        %D%/enum-extract.pl \
        %D%/zfs2zol-patch.sed \
-       %D%/zol2zfs-patch.sed \
-       $(EXTRA_SCRIPTS)
+       %D%/zol2zfs-patch.sed
 
-SHELLCHECKSCRIPTS += $(dist_scripts_SCRIPTS) $(EXTRA_SCRIPTS)
+SHELLCHECKSCRIPTS += $(dist_scripts_SCRIPTS) $(dist_noinst_SCRIPTS)
 
 define SCRIPTS_EXTRA_ENVIRONMENT
 
index d6ca957eccc80ff6879d659cde58156aa44c2782..2e633041ab597fd2be348daa17223de265aba5ee 100644 (file)
@@ -25,6 +25,6 @@ dist_scripts_runfiles_DATA = \
        %D%/runfiles/sunos.run
 
 
-EXTRA_DIST += $(addprefix %D%/,README.md)
+dist_noinst_DATA += %D%/README.md
 
 SHELLCHECKSCRIPTS += $(shell find $(srcdir)/%D% -name '*.sh')
index 71edd4939424c12fada963adfbfeae9780904b53..e3c9874dcd5487e7942fff963bcc02dd8b64f4b4 100644 (file)
@@ -41,7 +41,7 @@ scripts_zfs_tests_bin_PROGRAMS += %D%/draid
        libnvpair.la
 %C%_draid_LDADD += $(ZLIB_LIBS)
 
-EXTRA_DIST += $(addprefix %D%/,file/file_common.h)
+dist_noinst_DATA += %D%/file/file_common.h
 scripts_zfs_tests_bin_PROGRAMS += %D%/file_append %D%/file_check %D%/file_trunc %D%/file_write %D%/largest_file %D%/randwritecomp
 %C%_file_append_SOURCES   = %D%/file/file_append.c
 %C%_file_check_SOURCES    = %D%/file/file_check.c
@@ -116,7 +116,7 @@ scripts_zfs_tests_bin_PROGRAMS += %D%/xattrtest
 scripts_zfs_tests_bin_PROGRAMS += %D%/zed_fd_spill-zedlet
 
 
-EXTRA_DIST += $(addprefix %D%/,linux_dos_attributes/dos_attributes.h)
+dist_noinst_DATA += %D%/linux_dos_attributes/dos_attributes.h
 scripts_zfs_tests_bin_PROGRAMS  += %D%/read_dos_attributes %D%/write_dos_attributes
 %C%_read_dos_attributes_SOURCES  = %D%/linux_dos_attributes/read_dos_attributes.c
 %C%_write_dos_attributes_SOURCES = %D%/linux_dos_attributes/write_dos_attributes.c
index 49f9f3aaa7d6397ecfab59e2acf5c81e68ab7a02..a91a24d1668072c685c21696690ed8b8e4a5ee61 100644 (file)
@@ -1,5 +1,5 @@
 CLEANFILES =
-EXTRA_DIST =
+dist_noinst_DATA =
 include $(top_srcdir)/config/Substfiles.am