diff mbox series

[v4,1/2] xfsprogs: fix permissions on files installed by libtoolize

Message ID 20241004115704.2105777-2-aalbersh@redhat.com (mailing list archive)
State Not Applicable, archived
Headers show
Series Minor fixes for xfsprogs | expand

Commit Message

Andrey Albershteyn Oct. 4, 2024, 11:57 a.m. UTC 
Libtoolize installs some set of AUX files from its system package.
Not all distributions have the same permissions set on these files.
For example, read-only libtoolize system package will copy those
files without write permissions. This causes build to fail as next
line copies ./include/install-sh over ./install-sh which is not
writable.

Fix this by setting permission explicitly on files copied by
libtoolize.

Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Andrey Albershteyn <aalbersh@redhat.com>
---
 Makefile | 3 +++
 1 file changed, 3 insertions(+)

Comments

Christoph Hellwig Oct. 8, 2024, 4:34 a.m. UTC | #1 
Looks good:

Reviewed-by: Christoph Hellwig <hch@lst.de>
diff mbox series

Patch

diff --git a/Makefile b/Makefile
index 4e768526c6fe..c40728d9a455 100644
--- a/Makefile
+++ b/Makefile
@@ -109,6 +109,9 @@  endif
 
 configure: configure.ac
 	libtoolize -c -i -f
+	chmod 755 config.guess config.sub install-sh
+	chmod 644 ltmain.sh m4/libtool.m4 m4/ltoptions.m4 m4/ltsugar.m4 \
+		m4/ltversion.m4 m4/lt~obsolete.m4
 	cp include/install-sh .
 	aclocal -I m4
 	autoconf