<div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> On 24/02/16 07:37, Kyle Guinn wrote:<br>
> > On 2/23/16, Andrzej Telszewski <<a href="mailto:atelszewski@gmail.com" target="_blank">atelszewski@gmail.com</a>> wrote:<br>
> >> On 05/02/16 13:48, Martijn Dekker wrote:<br>
> >>> 2. The following horrible, broken and obsolete eyesore:<br>
> >>><br>
> >>> find -L . \<br>
> >>>    \( -perm 777 -o -perm 775 -o -perm 750 -o -perm 711 -o -perm<br>
> >>> 555 \ -o -perm 511 \) -exec chmod 755 {} \; -o \<br>
> >>>    \( -perm 666 -o -perm 664 -o -perm 640 -o -perm 600 -o -perm<br>
> >>> 444 \ -o -perm 440 -o -perm 400 \) -exec chmod 644 {} \;<br>
> >>><br>
> >>> can be replaced by:<br>
> >>><br>
> >>>      chmod -R a-st,u+rwX,go-w+rX .<br>
> >><br>
> >> I advocate this solution.<br>
> >> Can we switch for it?<br>
> >> Or at least would the SlackBuild be accepted if I used this<br>
> >> construct?<br>
> ><br>
> > Even shorter and simpler to understand:<br>
> > chmod -R u+w,go-w,a+rX-st .<br>
> ><br><br>
These 'hammer everything' solutions change permissions on files that<br>
the find command specifically misses, which may have been set on purpose<br>
by upstream.<br>
<br>
Note that I'm not adverse to submissions using a single chmod line,<br>
many of my scripts did this in the past. It requires an extra level<br>
of awareness though, that might be a trap for new players. For the<br>
template, we should leave the find command in place. Bulk swapping<br>
them out in the repo is not an option.<br></blockquote><div><br></div><div><br></div><div><br></div><div>Erik,</div><div><br></div><div>Are you opposed to this solution?</div><div><br></div><span class="lG" style="font-size:13px;line-height:19.5px">find</span><span style="font-size:13px;line-height:19.5px"> -L . \</span><br style="font-size:13px;line-height:19.5px"><span style="font-size:13px;line-height:19.5px"> \( -perm 777 -o -perm 775 -o -perm 750 -o -perm 711 -o -perm 555 \</span><br style="font-size:13px;line-height:19.5px"><span style="font-size:13px;line-height:19.5px">  -o -perm 511 \) -print0 | \</span><br style="font-size:13px;line-height:19.5px"><span style="font-size:13px;line-height:19.5px">  xargs -0 chmod 755</span><br style="font-size:13px;line-height:19.5px"><span class="lG" style="font-size:13px;line-height:19.5px">find</span><span style="font-size:13px;line-height:19.5px"> -L . \</span><br style="font-size:13px;line-height:19.5px"><span style="font-size:13px;line-height:19.5px"> \( -perm 666 -o -perm 664 -o -perm 640 -o -perm 600 -o -perm 444 \</span><br style="font-size:13px;line-height:19.5px"><span style="font-size:13px;line-height:19.5px">  -o -perm 440 -o -perm 400 \) -print0 | \</span><br style="font-size:13px;line-height:19.5px"><div><span style="font-size:13px;line-height:19.5px">  xargs -0 chmod 644</span></div><div><br></div><div>It was proposed by B. Watson here:</div><div><br></div><div><a href="https://lists.slackbuilds.org/pipermail/slackbuilds-users/2015-November/015210.html">https://lists.slackbuilds.org/pipermail/slackbuilds-users/2015-November/015210.html</a></div><div><br></div><div>It seems to work the same as the current template, but is just less resource intensive ... but I don't know for sure. </div></div></div>