Skip to content

Commit a5e40d8

Browse files
zwennamasahir0y
authored andcommitted
builddeb: Consolidate consecutive chmod calls into one
No need to call chmod three times when it can do everything at once. Signed-off-by: Sven Joachim <svenjoac@gmx.de> Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
1 parent ec61452 commit a5e40d8

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

scripts/package/builddeb

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,9 @@ create_package() {
4242
else
4343
chown -R root:root "$pdir"
4444
fi
45-
chmod -R go-w "$pdir"
46-
# in case we are in a restrictive umask environment like 0077
47-
chmod -R a+rX "$pdir"
48-
# in case we build in a setuid/setgid directory
49-
chmod -R ug-s "$pdir"
45+
# a+rX in case we are in a restrictive umask environment like 0077
46+
# ug-s in case we build in a setuid/setgid directory
47+
chmod -R go-w,a+rX,ug-s "$pdir"
5048

5149
# Create the package
5250
dpkg-gencontrol -p$pname -P"$pdir"

0 commit comments

Comments
 (0)