Review request for JDK-8014834 : shell tests don't begin with #!/bin/sh (original) (raw)

Alan Bateman Alan.Bateman at oracle.com
Mon Jun 3 10:07:46 UTC 2013


On 01/06/2013 13:32, Eric McCorkle wrote:

Thanks, Kumar. Do I have a second? For the jdk repository then only one reviewer is required although in specialized or complicated areas then having a reviewer that knows the area well is important. In this case I think you are good to go.

Just on Kumar's comment then I think a lot of the shell tests start with "#!" already so I would expect that scripts that check or fix-up copyright headers should be able to deal with this already. Another point, and maybe this for follow-on work, is that many of the shell tests don't have anything so it would be good to fix them up too. We should of course keep an eye out for opportunities to replace the shell tests as our ultimate goal should be to get rid of them.

-Alan.



More information about the core-libs-dev mailing list