Bugzilla – Full Text Bug Listing
| Summary: | Support Eclipse installations in target files (location type="Profile") | ||||||
|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Meng Xin Zhu <kane.zhu> | ||||
| Component: | Tycho | Assignee: | Christoph Laeubrich <laeubi> | ||||
| Status: | RESOLVED FIXED | QA Contact: | |||||
| Severity: | enhancement | ||||||
| Priority: | P3 | CC: | kane.mx, laeubi, mikhail.kalkov, sbouchet, stephan.leichtvogt, t-oberlies | ||||
| Version: | unspecified | ||||||
| Target Milestone: | --- | ||||||
| Hardware: | PC | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Bug Depends on: | 550601 | ||||||
| Bug Blocks: | |||||||
| Attachments: |
|
||||||
|
Description
Meng Xin Zhu
Created attachment 206508 [details]
patch
There is some limitations of eclipse installation as target platform, it's out of scope of this patch.
1. The installation does not have binary root IUs by default. So it's no way to build application or product that need launcher binary IUs.
2. The dir bundles can't be downloaded due to the default rule of artifacts.xml is mapping to jar file.
Meng, I am impressed about the scope of the change that your patch addresses. Unfortunately, I don't think that the solution is done in the right way: From what I understand, the locations of type "Profile" are fundamentally different from locations of type "InstallableUnit". Forcing TargetDefinitionFile.InstallationLocation under the TargetDefinition.InstallableUnitLocation interface requires a couple of hacks because the concepts just don't match. Also I have my doubts that it works correctly - did you ever try to mix "Profile" and "InstallableUnit" location types? Another possible functional issue seems to be the handling of the <includeBundles> attribute. I thought that that attribute is supposed to work with any location type. In your patch, it only seems to work with location type "Profile". In any case, if you want to provide support for the <includeBundles> attribute, please do this in a separate patch. @Meng Xin Zhu Are you still interested in this feature? I'm currently adding support for this via Bug 538144 but I'm still a little bit uncertain what a "Eclipse Installation" should supply, you are talking about binaries and UIs but when I add such an installation to my target it only explains itself as: > An installation (such as an Eclipse SDK) in the local file system. > The installed plug-ins will be added to the target definition. So it seems we should scan the bundles only (not features nor units or alike)? This is available in Tycho 2.0 now, please open a new bug if something is missing for you in the current implementation. |