Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 332546 - [releng] OCL 3.1.0 M4 is missing lpg.runtime.java
Summary: [releng] OCL 3.1.0 M4 is missing lpg.runtime.java
Status: CLOSED FIXED
Alias: None
Product: OCL
Classification: Modeling
Component: Core (show other bugs)
Version: 3.1.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Adolfo Sanchez-Barbudo Herrera CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-14 11:51 EST by Anthony Hunter CLA
Modified: 2011-05-27 03:13 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anthony Hunter CLA 2010-12-14 11:51:23 EST
S201012131752/mdt-ocl-SDK-3.1.0M4.zip is missing the lpg.runtime.java bundle. OCL will not load without it.
Comment 1 Adolfo Sanchez-Barbudo Herrera CLA 2010-12-14 12:19:09 EST
Hi Anthony,

Unexpected, but you are right. Besides I've checked that it's also missed in the M3 zips...

More info:
- All-In-One Update Site contains the bundle (otherwise, indigo's runAggregator build would have failed since M3).
- As I have been commented in the mdt/ocl dev list since time ago, the source bundle of lpg.runtime.java is not included. Neither in the All-In-One update site.
- Strangely, the zips contain the proper features, but due to an unknown reason the Orbit's lpg.runtime.java bundle is not included

I need to look into the bucky build to figure out why while including the correct features, the lpg.runtimejava plugin is not included in the zip files :\.

I'll also try to definitely make the lpg.runtime.java sources be included in every zip.

Cheers,
Adolfo.
Comment 2 Anthony Hunter CLA 2010-12-14 13:15:46 EST
The M4 zips on the download site can to be patched by getting the bundle from the OCL update site.

I have patched an M4 that I can use for the modeling builds on modeling.eclipse.org for now.
Comment 3 Ed Willink CLA 2010-12-14 13:38:26 EST
For 3.1 we should perhaps maintain the packaging, but is this really right?

The MDT/OCL zip does not repackage EMF or UML2, so why does it repackage LPG which is in Orbit?

If another project, perhaps IMP, also repackages, perhaps a different LPG version, and is used in conjunction with MDT/OCL, then there will be a conflict.

Surely fetching from Orbit is the responsibility of a build/install system that uses MDT/OCL zips? MDT/OCL should only distribute its own contributions.

Indigo is a major build migration to Buckminster. If Buckminster did it differently perhaps it did it better. Doing it the Buckminster way is a justification for a packaging change.

Adolfo: I'm inclined to wait for further comments and perhaps cross-project-dev discussion before deciding to fix this.
Comment 4 Anthony Hunter CLA 2010-12-14 13:45:32 EST
(In reply to comment #3)
> Surely fetching from Orbit is the responsibility of a build/install system that
> uses MDT/OCL zips? MDT/OCL should only distribute its own contributions.

There is no orbit download, the bundles are distributed with the projects. That is why the lpg.runtime.java is listed in the org.eclipse.ocl feature.
Comment 5 Ed Willink CLA 2010-12-14 13:49:56 EST
I don't understand your comment.

Surely you have a build that does perhaps EMF+UML+OCL+EMFt, for which you reference a ZIP each. For convenience LPG saved you finding LPG.

I'm suggesting that this is a poor packaging practice and that you should do EMF+UML+OCL+EMFt+LPG. For instance when using Xtext, we have to track down all the Xtenm, Xpand, Mwe, Mwe2 hidden dependencies.
Comment 6 Anthony Hunter CLA 2010-12-14 14:06:51 EST
A good read : http://www.eclipse.org/orbit/overview.php
Comment 7 Ed Willink CLA 2010-12-14 16:25:16 EST
(In reply to comment #6)
> A good read : http://www.eclipse.org/orbit/overview.php

This doesn't help me. It doesn't recommend that Orbit bundles should or should not be repackaged. It merely discusses availability.

Looking for an example: In Xtext, Xtend, mwe, mwe2, I can find 3 identical versions of log4j, so I guess Orbit is ensuring that we can have identical versions.

I'll email David Williams to see whether he'd like to add "The consistent Orbit names ensure that projects can package the Orbit bundles they use in their own ZIPs and avoid the conflicts that could otherwise arise from non-identical copies." to the second paragraph.
Comment 8 Adolfo Sanchez-Barbudo Herrera CLA 2010-12-23 10:53:55 EST
Since some builds ago the lpg.runtime.java is included in the generated artifacts. The lpg.runtime.java.source bundle is also included in the zips, excepting the Update zip, in which I haven't been able to include it

I've not closed this bug since I firstly wanted to also complete the correct packaging of the Update zip. However, my help request in dev-cross-project list and buckminster forums (http://www.eclipse.org/forums/index.php?t=msg&th=201971&start=0&S=823ed1040c0319f5bf27e3db26f783b9 )  have not received any response....

Anyway, instead of closing this and open a new one.... I'll leave the bug open for tracking until it's finally solved... Anthony the last I-build contains (and M5 will) your request, all right ?

(In reply to comment #2)
> The M4 zips on the download site can to be patched by getting the bundle from
> the OCL update site.
> 
> I have patched an M4 that I can use for the modeling builds on
> modeling.eclipse.org for now.

I'm not sure what you refer...did you mean your M4 zips ?, which kind of patch ? could you explain for my information ?

Cheers,
Adolfo.
Comment 9 Anthony Hunter CLA 2010-12-23 11:04:03 EST
Our build on modeling.eclipse.org downloads its dependencies and does not download them again each build. So I have patched the downloaded mdt-ocl-SDK-3.1.0M4.zip by adding the missing lpg.runtime.java.
Comment 10 Adolfo Sanchez-Barbudo Herrera CLA 2011-03-10 12:47:28 EST
After checking that the following bugzilla fix Bug 338803, we are finally distributing lpg.runtime.java.source bundle in the our P2 repository and the corresponding mdt.ocl-Update.zip.

Resolving as fixed.
Comment 11 Ed Willink CLA 2011-05-27 03:13:44 EDT
Closing