RFR: 7175331 - Remove FDS files from embedded JRE images
David Holmes
david.holmes at oracle.com
Sat Jun 9 03:50:04 PDT 2012
On 9/06/2012 9:25 AM, Daniel D. Daugherty wrote:
> On 6/7/12 7:01 PM, David Holmes wrote:
>> This is a simple fix that only applies to 7u and is targeted for 7u6.
>>
>> We need to update the embedded image target in Release-embedded.gmk to
>> remove the FDS related files that now get created. This obviously only
>> impacts embedded builds.
>>
>> http://cr.openjdk.java.net/~dholmes/7175331/webrev/
>>
>> Also tidied up some extraneous -f options on rm ($(RM) == rm -f)
>
> line 168: -type f -name "*.debuginfo" -o -name "*.diz" -print
> I think this part needs to be:
>
> -type f \( -name "*.debuginfo" -o -name "*.diz" \) -print
>
> so that the '-o' option binds correctly.
Thanks Dan you are right. It's not likely that there would be a *.diz
directory but better to get this right. I've updated the webrev for
completeness:
http://cr.openjdk.java.net/~dholmes/7175331/webrev.01/
but will assume your reviews stands, as does Kelly's. I have to request
approval to push anyway.
Thanks,
David
> Dan
>
More information about the jdk7u-dev
mailing list