On 2018-12-03 17:16, Alan Bateman wrote:
On 03/12/2018 16:10, Magnus Ihse Bursie wrote:
The commands for hg log is missing -l1, which will limit the log to
just the revision specified. Instead, all revisions from repo
creation will now be included, and the script fails to work.
I will publish a separate changeset with missed copyright year
updates in the build system.
Bug: https://bugs.openjdk.java.net/browse/JDK-8214710
WebRev:
http://cr.openjdk.java.net/~ihse/JDK-8214710-fix-update-copyright-year/webrev.01
This looks okay to me.
Thanks.
For the follow-on "missing copyright year updates" then maybe the
entire source should be done rather than doing it in piecemeal.
I just created JDK-8214718... Anyway, I'm not sure that's a good idea.
The script gives an indication of files that should be updated, but it's
not 100% correct and someone will need to verify that it does not
trigger for changes that should not result in a copyright year update
(like updating the copyright header). The script, as it is, tries to
filter out changesets that it should not consider, but that looked very
brittle, and I did in fact remove that check before running the script
on the build system code. Perhaps not component knowledge is needed, but
laying the burden on a single person to do this grunt work for all of
the code base is perhaps not fair either. For me, I'd like to know that
the code I'm responsible for is correct.
/Magnus
