[
https://issues.apache.org/jira/browse/FELIX-1055?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12702083#action_12702083
]
Ali Naddaf commented on FELIX-1055:
-----------------------------------
Hi Clement.
java.net.URi is part of the CDC-1.1/Foundation 1.1; if you look at the
Compendium specs, under OSGi Defined Execution Environment, (section 999.3),
you will find that class included in CDC (but not in OSGi-Minumum). Please
correct me if I am wrong. I also checked my J9 Found1.1 java runtime
environment and java.net.URI class was indeed there.
> Making FileInstall bundle CDC-1.1/Foundation 1.1 compatible
> -----------------------------------------------------------
>
> Key: FELIX-1055
> URL: https://issues.apache.org/jira/browse/FELIX-1055
> Project: Felix
> Issue Type: Improvement
> Components: File Install
> Environment: All
> Reporter: Ali Naddaf
> Priority: Minor
> Original Estimate: 0h
> Remaining Estimate: 0h
>
> A minor change will make File Install bundle compatible with the
> CDC-1.1/Foundation-1.1 compatible:
> Index: src/main/java/org/apache/felix/fileinstall/FileInstall.java
> ===================================================================
> --- src/main/java/org/apache/felix/fileinstall/FileInstall.java (revision
> 761715)
> +++ src/main/java/org/apache/felix/fileinstall/FileInstall.java (working copy)
> @@ -68,7 +68,7 @@
> Object o = context.getProperty(key);
> if (o == null)
> {
> - o = System.getenv(key.toUpperCase().replaceAll(".", "_"));
> + o = System.getProperty(key.toUpperCase().replace('.', '_'));
> if (o == null)
> {
> return;
> ----------------------------------------------------------------------------------------------------------
> The only needed change is in the FileInstall class:
> 1. replacing String.replaceAll(String, String) with String.replace(char,
> char) (since replaceAll is not part of that profile and we are only making
> one char replacement). If a more robust replaceAll functionality is required,
> please let me know and I provide a simple implementation.
> 2. Using System.getProperty(..) instead of System.getenv(...)
> Thanks,
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
