Bug 259763 - [product] allow launcher bundles to be excluded
Summary: [product] allow launcher bundles to be excluded
Status: RESOLVED FIXED
Alias: None
Product: PDE
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.4.1   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: 3.5 M7   Edit
Assignee: Chris Aniszczyk CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
: 255362 255363 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-12-29 15:16 EST by Scott Lewis CLA
Modified: 2009-04-21 04:53 EDT (History)
6 users (show)

See Also:


Attachments
org.eclipse.pde.patch (11.66 KB, patch)
2009-03-16 18:56 EDT, Chris Aniszczyk CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Scott Lewis CLA 2008-12-29 15:16:47 EST
Currently, the o.e.equinox.launcher and the an associated platform-specific fragment is required to do a .product-based build.

In some use cases (e.g. creating an servletbridge-based equinox server) the launcher bundles are not needed, and it would be very useful to be able to exclude the launcher bundles as part of .product-based PDE build.
Comment 1 Andrew Niefer CLA 2009-01-05 16:30:18 EST
This is possible now, exclude all launcher bits (exe, jar & fragment) by setting includeLaunchers=false.

See also bug 255362, bug 255363
Comment 2 Scott Lewis CLA 2009-01-05 17:28:57 EST
I'm reopening this, because the request was intended to include user interface in PDE build to allow this exclusion from PDE Product Export.  

I changed it to PDE UI component to reflect this...if this is a misclassification then feel free to change appropriately.

Comment 3 Scott Lewis CLA 2009-01-06 09:59:15 EST
*** Bug 255362 has been marked as a duplicate of this bug. ***
Comment 4 Benjamin Cabé CLA 2009-01-06 10:22:43 EST
reassigning to pde/ui inbox
Comment 5 Curtis Windatt CLA 2009-01-06 10:54:35 EST
Chris, what do you think about this?  Shouldn't be hard to implement, though it
is yet another option on the product export dialog.
Comment 6 Chris Aniszczyk CLA 2009-01-06 11:04:34 EST
(In reply to comment #5)
> Chris, what do you think about this?  Shouldn't be hard to implement, though it
> is yet another option on the product export dialog.

I'm thinking an option on the .product file is better. Somewhere on the launching page, you can specify via a checkbox that you want a launcher. This would than enable the launcher section for you to fill out. This would be enabled by default.

Does this sound as an acceptable workflow for you Scott?
Comment 7 Scott Lewis CLA 2009-01-06 11:32:51 EST
(In reply to comment #6)
> (In reply to comment #5)
> > Chris, what do you think about this?  Shouldn't be hard to implement, though it
> > is yet another option on the product export dialog.
> 
> I'm thinking an option on the .product file is better. Somewhere on the
> launching page, you can specify via a checkbox that you want a launcher. This
> would than enable the launcher section for you to fill out. This would be
> enabled by default.
> 
> Does this sound as an acceptable workflow for you Scott?
> 

I suppose, but it does seem to me more like an option for the export dialog.  Maybe that's because I've been using the export dialog a lot, but it feels more like something done during a build/export rather than a launch within Eclipse (as it seems more about packaging...at least in the case of build of servers from product specs).

Comment 8 Andrew Niefer CLA 2009-01-06 11:51:48 EST
I think this is better done in the .product file as this file is becoming the focal point of a lot of p2 metadata generation.   We are trying to separate the metadata generation to be more standalone and less reliant on pde.build, which means keeping eveything in the .product file as opposed to having product metadata depending on the build setup.

(Though this means more work for me instead of just leaving it to UI to add an option in the export :) )

This will need to be sent back to build and to p2 once the option is in the .product file.

Note that there were a couple of bugs here that said similar things, there are 3 cases:
1) include the exe and the launcher bundle and fragments
2) include the launcher bundle and fragments but not the exe
3) don't include any launcher pieces.
Comment 9 Scott Lewis CLA 2009-01-06 11:58:34 EST
(In reply to comment #8)
> I think this is better done in the .product file as this file is becoming the
> focal point of a lot of p2 metadata generation.   We are trying to separate the
> metadata generation to be more standalone and less reliant on pde.build, which
> means keeping eveything in the .product file as opposed to having product
> metadata depending on the build setup.
> 
> (Though this means more work for me instead of just leaving it to UI to add an
> option in the export :) )
> 
> This will need to be sent back to build and to p2 once the option is in the
> .product file.


I completely understand the desire to put this option in the .product file (so that it's stored as metadata rather than in the build setup), but isn't this orthogonal to where/whether the UI allows the user to set the option?  (i.e. couldn't it be in product export even if stored in .product file?

Comment 10 Chris Aniszczyk CLA 2009-01-06 12:00:52 EST
(In reply to comment #8)
> Note that there were a couple of bugs here that said similar things, there are
> 3 cases:
> 1) include the exe and the launcher bundle and fragments
> 2) include the launcher bundle and fragments but not the exe

Who needs this case? And how would we instruct PDE Build to do this?

> 3) don't include any launcher pieces.

Comment 11 Jeff McAffer CLA 2009-01-06 12:54:44 EST
Not sure about case #2.  From a high level PDE user point of view there are three cases
a) normal run a product/application use the exe and launcher stuff (standard RCP)
b) use the exe and launcher stuff but not be doing a product/app (Equinox-based system)
c) no exe/launcher and no product/app (WAR case)

#2 is technically possible an some may want it.  I'm not sure how popular it is.
Comment 12 Chris Aniszczyk CLA 2009-03-10 13:44:04 EDT
Let's do this in M7 after the properties changes go in.
Comment 13 Chris Aniszczyk CLA 2009-03-16 18:56:13 EDT
Created attachment 129015 [details]
org.eclipse.pde.patch

Adds a new button to say whether a product is launchable or not.

The current checkbox wording is "The product is launchable"

I'm not a big fan of this yet so I'm taking suggestions for better wording.
Comment 14 Chris Aniszczyk CLA 2009-03-16 18:58:44 EDT
oh lovely, just realized it's "includeLaunchers" not excludeLaunchers... new patch coming.
Comment 15 Scott Lewis CLA 2009-03-16 19:02:04 EDT
(In reply to comment #13)
> Created an attachment (id=129015) [details]
> org.eclipse.pde.patch
> 
> Adds a new button to say whether a product is launchable or not.
> 
> The current checkbox wording is "The product is launchable"
> 
> I'm not a big fan of this yet so I'm taking suggestions for better wording.
> 

My $0.02:  "The product will be launchable"...default true.
Comment 16 Chris Aniszczyk CLA 2009-03-16 19:10:04 EDT
done.

> 20090316

Thanks for the feedback.
Comment 17 Markus Kuppe CLA 2009-03-17 03:10:08 EDT
(In reply to comment #13)
> The current checkbox wording is "The product is launchable"
> 
> I'm not a big fan of this yet so I'm taking suggestions for better wording.

"Include native launcher with product"
Comment 18 Chris Aniszczyk CLA 2009-04-21 04:53:42 EDT
*** Bug 255363 has been marked as a duplicate of this bug. ***