Community
Participate
Working Groups
Build ID: M20080911-1700 Steps To Reproduce: 1. Add all features/plugins (1st add one which is used by others (common plugin) and then add features/plugins which use it) 2. Change common plugin version. Remove it from updater list and add new one (with new version). It will be added to the end of the site.xml. 3. Try to build. It will fail. 4. Manually move common plugin configuration in site.xml to the 1st position. Everything is okay. More information: - Maybe order can be calculated dynamically? It's not that difficult it should be a DAG. - Or at least allow users to reorder plugins/features.
Old Summary: Updater project fails to build all features/plugins...
I believe this site builder is in PDE. However, I couldn't reproduce the problem. Can you clarify the steps, since you talk about adding/removing a plugin from a site.xml, but sites can only contain features.
Features are compiled in order, depth first. However, starting in 3.5M3, flattenDependencies (bug 212467) could be used here. For UI use BuildScriptGenerator#setFlattenDependencies
What's the negative impact of doing this Andrew?
Actually, it probably won't be that easy. Setting this property results in a separate compile script being generated "compile.<feature>.xml". Currently UI is explicitly calling the feature's build.xml build.jars target (FeatureExportOperation#doExport calls to runScript). It may also be calling the build.sources target. Doing this compiles everything in-order, depth first. To use the flattenDependencies, you would instead need to invoke the compile.<feature>.xml script (default target). However, this compile script only calls build.jars, not build.sources, so if you want sources you would need a second call to the feature's build.xml as before but with just the build.sources target.
Created attachment 126033 [details] org.eclipse.pde.core.patch To start, we would enable this for every operation. This would solve quite a bit of problems for people. The next step is to go over Andrew's comments to see what we can do. Andrew, do you have sometime to look this over with me? I think this is a big win if we can fix this ordering problem with which I've seen quite a few people bit in the field.
Created attachment 126079 [details] org.eclipse.pde.core.patch An updated patch that takes Andrew's suggestion. I need to do more testing before I'm confident in releasing. Andrew, would have time to do some testing also? I'm trying to think of scenarios where this would break someone.
done testing. Looks good from my end. > 20090217 Let's see how this works in the next i-build!