Hi Stefan, thanks for reviewing this. I missed the fact that filesets had a similar attribute. Hope everything is consistent now.
Gintas 2018-05-28 17:00 GMT+02:00 Stefan Bodewig <bode...@apache.org>: > Hi Gintas > > you should probably check and document how the new followlinks attribute > interacts with fileset's followsymlinks attribute. > > Please add something to WHATSNEW. > > Some additional notes inline. > > On 2018-05-23, <gin...@apache.org> wrote: > > > http://git-wip-us.apache.org/repos/asf/ant/blob/35a84fea/ > manual/Types/selectors.html > > ---------------------------------------------------------------------- > > diff --git a/manual/Types/selectors.html b/manual/Types/selectors.html > > index 955a1b2..e3289af 100644 > > --- a/manual/Types/selectors.html > > +++ b/manual/Types/selectors.html > > @@ -925,6 +925,11 @@ > > <td>Username of the expected owner</td> > > <td>Yes</td> > > </tr> > > + <tr> > > + <td>followlinks</td> > > + <td>Must the selector follow symbolic links?</td> > > + <td>No; defaults to <q>false</q> (was <q>true</q> before Ant > 1.10.4)</td> > > + </tr> > > </table> > > Why change the default? > > > http://git-wip-us.apache.org/repos/asf/ant/blob/35a84fea/ > src/main/org/apache/tools/ant/types/selectors/OwnedBySelector.java > > > + /** > > + * Sets the "follow links" flag. > > + * @param followLinks the user name > > + */ > > + public void setFollowLinks(String followLinks) { > > + this.followLinks = PropertyHelper.toBoolean(followLinks); > > + } > > public void setFollowLinks(boolean followLinks) { > this.followLinks = followLinks; > } > > does the same and looks clearer to me. Same for the other attribute > setters. > > Stefan > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org > For additional commands, e-mail: dev-h...@ant.apache.org > >