Hi Geertjan,

thanks for doing this work.

Unfortunately I have a comment regarding your commit. All those newly added
files, named .DS_Store, do not belong into the commit. Actually they should
be ignored be the .gitignore file as it os a macOS specific file (see [1]).

Beyond this I haven't checked anything else in the commit.

Regards
Martin

[1] https://en.wikipedia.org/wiki/.DS_Store

On Sun, Sep 24, 2017 at 11:53 PM, Geertjan Wielenga <
[email protected]> wrote:

> Yes, definitely.
>
> Also, by the way, several other modules are affected, inc performance.java,
> among many others. (The tool is able to print out a list of all changed
> files, pretty handy.)
>
> Gj
>
> On Sun, Sep 24, 2017 at 11:49 PM, Dave Schoorl <[email protected]> wrote:
>
> > I think that's OK. But should the commit text not start with
> [NETBEANS-54]
> > for traceability with the Jira issue?
> >
> > /Dave
> >
> > >
> > >     Op 24 september 2017 om 23:42 schreef Geertjan Wielenga <
> > [email protected]>:
> > >
> > >     Hi all,
> > >
> > >     Today Jan Lahoda finetuned the ASF license converter tool to catch
> > more
> > >     Oracle licenses:
> > >
> > >     https://github.com/apache/incubator-netbeans-tools/commit/
> > 6756d08c0b7b76295a480b5e0335a86c68ae11c2
> > >
> > >     That finetuning results in 1239 files being converted to Apache,
> > which is
> > >     pretty cool, i.e., the more we can automate the better. From what I
> > can
> > >     tell, all the relicensed files thanks to this particular finetuning
> > are in
> > >     the java.examples and javafx2.samples modules.
> > >
> > >     I'll be pushing these license changes in the same way as previously
> > --
> > >     i.e., directly to incubator-netbeans, and not via a fork. Hope
> > that's OK
> > >     with everyone -- I don't consider these license header changes as
> > something
> > >     that needs to be reviewed, hence not doing these via a fork but
> > making them
> > >     directly available to incubator-netbeans.
> > >
> > >     Hope that's OK with everyone.
> > >
> > >     Gj
> > >
> >
> >
> >
> > Met vriendelijke groet,
> >
> > Dave Schoorl | Java ontwikkelaar |BKWI http://www.bkwi.nl/ | 06-52749699
> > | Afwezig op feestdagen en weekenden
> > St. Jacobsstraat 400-420, 3511 BT Utrecht | Postbus 1407, 3500 BK Utrecht
> > | www.bkwi.nl http://www.bkwi.nl/
> >
>

Reply via email to