Mailing-List: contact issues-help@karaf.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@karaf.apache.org
Date: Mon, 11 Jul 2011 10:55:59 +0000 (UTC)
From: =?utf-8?Q?Jean-Baptiste_Onofr=C3=A9_=28JIRA=29?= <jira@apache.org>
To: issues@karaf.apache.org
Message-ID: <3739400.2653.1310381759889.JavaMail.tomcat@hel.zones.apache.org>
In-Reply-To: <16064364.212391292842140914.JavaMail.jira@thor>
Subject: [jira] [Updated] (KARAF-334) At startup, Karaf does not support
 urls like mvn:, obr:.
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable


     [ https://issues.apache.org/jira/browse/KARAF-334?page=3Dcom.atlassian=
.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jean-Baptiste Onofr=C3=A9 updated KARAF-334:
---------------------------------------

    Fix Version/s: 3.1.0
                   3.0.0

> At startup, Karaf does not support urls like mvn:, obr:.
> --------------------------------------------------------
>
>                 Key: KARAF-334
>                 URL: https://issues.apache.org/jira/browse/KARAF-334
>             Project: Karaf
>          Issue Type: Improvement
>          Components: runtime
>    Affects Versions: 2.1.2
>            Reporter: Stephane Chomat
>            Assignee: Guillaume Nodet
>             Fix For: 3.0.0, 3.1.0
>
>         Attachments: patch-KARAF-334, patch-KARAF-334-karaf-2.1.x, test-k=
araf-home.zip
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> At startup, Karafi does not support urls like mvn:, obr:.
> Imagine that at level 5 you install the bundle pax-url-mvn with karaf.aut=
o.start.5 =3D "file: pax-url-mvn.jar | pax-url-mvn"=20
> and then at level 10 you install mvn:. ... With karaf.auto.start.10 =3D "=
mvn :....| ...."
> At the level 6 the url handler 'mvn' is available and you can use it.
> This feature does not work with Karaf
> I propose this path :
> diff -r f0bb11c3d77c src/main/java/org/apache/karaf/main/Main.java
> --- a/src/main/java/org/apache/karaf/main/Main.java=09Mon Dec 20 11:05:11=
 2010 +0100
> +++ b/src/main/java/org/apache/karaf/main/Main.java=09Mon Dec 20 11:46:29=
 2010 +0100
> @@ -485,28 +485,20 @@
>          // the start level to which the bundles are assigned is specifie=
d by
>          // appending a ".n" to the auto-install property name, where "n"=
 is
>          // the desired start level for the list of bundles.
> -        autoInstall(PROPERTY_AUTO_INSTALL, context, sl, convertToMavenUr=
ls);
> +        autoInstall(PROPERTY_AUTO_INSTALL, context, sl, convertToMavenUr=
ls, false);
> =20
>          // The auto-start property specifies a space-delimited list of
>          // bundle URLs to be automatically installed and started into ea=
ch
>          // new profile; the start level to which the bundles are assigne=
d
>          // is specified by appending a ".n" to the auto-start property n=
ame,
>          // where "n" is the desired start level for the list of bundles.
> -        // The following code starts bundles in two passes, first it ins=
talls
> -        // them, then it starts them.
> -            List<Bundle> bundlesToStart =3D autoInstall(PROPERTY_AUTO_ST=
ART, context, sl, convertToMavenUrls);
> -        // Now loop through and start the installed bundles.
> -        for (Bundle b : bundlesToStart) {
> -            try {
> -                b.start();
> -            }
> -            catch (Exception ex) {
> -                System.err.println("Auto-properties start: " + ex);
> -            }
> -        }
> +        // The following code starts bundles in one passes, it installs
> +        // and it starts them by level.
> +        autoInstall(PROPERTY_AUTO_START, context, sl, convertToMavenUrls=
, true);
> +       =20
>      }
> =20
> -    private List<Bundle> autoInstall(String propertyPrefix, BundleContex=
t context, StartLevel sl, boolean convertToMavenUrls) {
> +    private List<Bundle> autoInstall(String propertyPrefix, BundleContex=
t context, StartLevel sl, boolean convertToMavenUrls, boolean start) {
>          Map<Integer, String> autoStart =3D new TreeMap<Integer, String>(=
);
>          List<Bundle> bundles =3D new ArrayList<Bundle>();
>          for (Iterator i =3D configProps.keySet().iterator(); i.hasNext()=
;) {
> @@ -531,6 +523,7 @@
>          }
>          for (Integer startLevel : autoStart.keySet()) {
>              StringTokenizer st =3D new StringTokenizer(autoStart.get(sta=
rtLevel), "\" ", true);
> +            List<Bundle> bundlesLevel =3D new ArrayList<Bundle>();
>              if (st.countTokens() > 0) {
>                  String location =3D null;
>                  do {
> @@ -541,6 +534,7 @@
>                              Bundle b =3D context.installBundle(parts[0],=
 new URL(parts[1]).openStream());
>                              sl.setBundleStartLevel(b, startLevel);
>                              bundles.add(b);
> +                            bundlesLevel.add(b);
>                          }
>                          catch (Exception ex) {
>                              System.err.println("Auto-properties install:=
" + ex);
> @@ -549,6 +543,15 @@
>                  }
>                  while (location !=3D null);
>              }
> +         // Now loop through and start the installed bundles.
> +            for (Bundle b : bundlesLevel) {
> +                try {
> +                    b.start();
> +                }
> +                catch (Exception ex) {
> +                    System.err.println("Auto-properties start: " + ex);
> +                }
> +            }
>          }
>          return bundles;
>      }

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira