cziegeler commented on a change in pull request #1: FELIX-6232 - adding option 
to install parallel version of bundle
URL: https://github.com/apache/felix-dev/pull/1#discussion_r386392472
 
 

 ##########
 File path: 
webconsole/src/main/java/org/apache/felix/webconsole/internal/core/BundlesServlet.java
 ##########
 @@ -1617,14 +1622,15 @@ private FileItem getParameter( Map params, String name 
)
     }
 
 
-    private void installBundle( String location, File bundleFile, int 
startLevel, boolean start, boolean refreshPackages )
+    private void installBundle( String location, File bundleFile, int 
startLevel, boolean start, boolean refreshPackages, boolean parallelVersion)
             throws IOException
     {
         if ( bundleFile != null )
         {
 
             // try to get the bundle name, fail if none
             String symbolicName = getSymbolicName( bundleFile );
 
 Review comment:
   I suggest to get bsn and version in one method, otherwise you open the file 
twice

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to