RFR(XS): CODETOOLS-7902075: jtreg doesn't build if jh.jar doesn't contain signatures
Volker Simonis
volker.simonis at gmail.com
Tue Dec 5 08:19:15 UTC 2017
Hi Jon,
thanks a lot for the detailed information.
That all sound very reasonable, so we will start to use JT Harness 5 as well.
Regards,
Volker
On Tue, Dec 5, 2017 at 12:19 AM, Jonathan Gibbons
<jonathan.gibbons at oracle.com> wrote:
> Hi Volker,
>
> We're in transition.
>
> I've done all the work to enable JT Harness 5 to be the default.
>
> Oracle is still using 4.6, but we're going through some internal
> administrative trivia to change that. I'm hoping that that will
> happen this week, and once it does, I'll then change the default
> and add an appropriate new tag.
>
> Internally within Oracle, most folk use a jtreg build of the
> most recently tagged version, although we do have some CI
> systems using jtreg tip, for the purpose of testing jtreg itself.
>
> My expectation is that once JT Harness 5 is the default,
> we will remove support for JT Harness 4.6, and in particular,
> remove the need/ability to include JavaHelp. But I don't expect
> that to happen until sometime next year.
>
> Once the default has been changed, I'll updated the jtreg
> build page [1].
>
> -- Jon
>
> [1] http://openjdk.java.net/jtreg/build.html
>
>
>
> On 12/04/2017 01:38 AM, Volker Simonis wrote:
>>
>> Hi Jonathan,
>>
>> thanks a lot for the quick reaction.
>>
>> So what is currently the recommended version of JTHanress, 4 or 5 ?
>> And what is Oracle using for running the JTreg tests (e.g. in the
>> HotSpot per-integration testing formerly known as JPRT, now probably
>> Mach5) ?
>>
>> We want to be as close as possible to the Oracle setup, but the JTreg
>> build instructions page [1] still mention JTHarness 4.6. A site which
>> mentions the default test setups (equivalent to the "Supported Build
>> Platforms" [2] page) would be extremely useful.
>>
>> Thank you and best regards,
>> Volker
>>
>> [1] http://openjdk.java.net/jtreg/build.html
>> [2] https://wiki.openjdk.java.net/display/Build/Supported+Build+Platforms
>>
>> On Fri, Dec 1, 2017 at 7:52 PM, Jonathan Gibbons
>> <jonathan.gibbons at oracle.com> wrote:
>>>
>>> Volker,
>>>
>>> I can push the fix for you, but it may already be redundant.
>>>
>>> If you use JT Harness 5, then you don't need JavaHelp at all.
>>> Just leave JAVAHELP_JAR unset.
>>>
>>> I've been pushing changes to jtreg this week to make it easier
>>> to build jtreg, and although I haven't yet flipped the switch on making
>>> the default be to use JT Harness 5, I have verified that you can
>>> build and run jtreg, using JT Harness 5, without JavaHelp and without
>>> JCov.
>>>
>>> -- Jon
>>>
>>>
>>> On 12/01/2017 08:29 AM, Volker Simonis wrote:
>>>>
>>>> Hi,
>>>>
>>>> can I please have a sponsor for the following simple fix of the jtreg
>>>> build:
>>>>
>>>> http://cr.openjdk.java.net/~simonis/webrevs/2017/CODETOOLS-7902075/
>>>> https://bugs.openjdk.java.net/browse/CODETOOLS-7902075
>>>>
>>>> Change "7901920: jh.jar is signed with a weak algorithm and its
>>>> certificate is expired" introduced a logic to remove outdated
>>>> signatures from jh.jar.
>>>>
>>>> However, there's no default location for downloading jh.jar and
>>>> probably most available versions (e.g. the one from Maven:
>>>>
>>>>
>>>> http://central.maven.org/maven2/javax/help/javahelp/2.0.05/javahelp-2.0.05.jar)
>>>> if not all of them don't contain signatures at all. This breaks the
>>>> build because removing the signatures from the jar file with zip will
>>>> fail with the following error:
>>>>
>>>> zip warning: name not matched: META-INF/*.SF
>>>> zip warning: name not matched: META-INF/*.RSA
>>>> zip warning: name not matched: META-INF/*.DSA
>>>> zip error: Nothing to do!
>>>> (/net/usr.work/openjdk/tools/jtreg/jtreg/build/images/jtreg/lib/jh.jar
>>>>
>>>> It would be great if the sponsor could also tag this fix with
>>>> jtreg4.2-b11 after pushing it because the last tagged version (i.e.
>>>> jtreg4.2-b10) doesn't build out of the box without JCov (which should
>>>> be optional and which was fixed by "7902072: Add support for imported
>>>> license files").
>>>>
>>>> Thank you and best regards,
>>>> Volker
>>>
>>>
>
More information about the code-tools-dev
mailing list