RFR 7191662: JCE providers should be located via ServiceLoader,

[Mandy Chung]

On 05/22/2015 08:09 AM, Alan Bateman wrote:
>
> On 22/05/2015 13:55, Chris Hegarty wrote:
>> :
>> I think it could be done either way.
>
> Valerie - have you considered not pushing the services configuration 
> files with this change? With the change then the java.security 
> configuration is still class names, not provider names, so the 
> fallback should just work. This is what we've done in a few other 
> areas (like JNDI for example).

I wasn't aware of the other areas that move to service provider but 
remain being loaded with the fallback Class.forName.

I would prefer java.security should convert to use the provider names as 
an example and also exercise the code path using service providers.  If 
this causes much work to workaround it temporarily, I won't object the 
security providers are not truly service providers (no META-INF/services 
and java.security lists class name instead)

Another option to workaround this:

we only need to merge the service config files for generating the 
image.  Can we have do the concatenation of 
jdk/modules/*/META-INF/services file and output to supports/image_gensrc 
before the images target and have the image builder to exclude all 
jdk/modules/*/META-INF/services files and take the supports/image_gensrc 
instead?

This will remove the process-provider logic from Gensrc-*.gmk files.  
Would this be a better alternative?

Mandy