[Bug 66164] build/install from source changes file permissions throughout my home directory

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=66164

--- Comment #1 from Jaikiran Pai <ja...@apache.org> ---
This appears to be a genuine issue. I had a look at our build.xml which does
this and there a couple of places where we blindly update these permissions on
files/dirs of the "dist.dir". I have a proposed fix for this which looks as
below, but looking at the commit history, the current behaviour has been around
for more than 2 decades, so I will wait for a few days to see if anyone has any
inputs/objections to the change below, before I commit it:

diff --git a/build.xml b/build.xml
index 8d401aac7..48378b078 100644
--- a/build.xml
+++ b/build.xml
@@ -1057,14 +1057,6 @@
       <include name="*.pl"/>
     </fixcrlf>

-    <setpermissions mode="755" failonerror="${setpermissions.fail}"
-                    nonPosixMode="${setpermissions.nonposixmode}">
-      <dirset dir="${dist.dir}"/>
-    </setpermissions>
-    <setpermissions mode="644" failonerror="${setpermissions.fail}"
-                    nonPosixMode="${setpermissions.nonposixmode}">
-      <fileset dir="${dist.dir}"/>
-    </setpermissions>
     <setpermissions mode="755" failonerror="${setpermissions.fail}"
                     nonPosixMode="${setpermissions.nonposixmode}">
       <fileset dir="${dist.bin}">
@@ -1158,14 +1150,6 @@
       </fileset>
     </copy>

-    <setpermissions mode="755" failonerror="${setpermissions.fail}"
-                    nonPosixMode="${setpermissions.nonposixmode}">
-      <dirset dir="${dist.dir}"/>
-    </setpermissions>
-    <setpermissions mode="644" failonerror="${setpermissions.fail}"
-                    nonPosixMode="${setpermissions.nonposixmode}">
-      <fileset dir="${dist.dir}"/>
-    </setpermissions>
     <setpermissions mode="755" failonerror="${setpermissions.fail}"
                     nonPosixMode="${setpermissions.nonposixmode}">
       <fileset dir="${dist.bin}">

-- 
You are receiving this mail because:
You are the assignee for the bug.