Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 332098 - Metadata generator should not create mkdir/rmdir actions for reconciler bundle
Summary: Metadata generator should not create mkdir/rmdir actions for reconciler bundle
Status: RESOLVED FIXED
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: p2 (show other bugs)
Version: 3.7   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: 3.6.2   Edit
Assignee: DJ Houghton CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 332165
  Show dependency tree
 
Reported: 2010-12-07 18:43 EST by DJ Houghton CLA
Modified: 2010-12-08 14:28 EST (History)
2 users (show)

See Also:


Attachments
patch (2.23 KB, patch)
2010-12-08 11:35 EST, DJ Houghton CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description DJ Houghton CLA 2010-12-07 18:43:22 EST
In the old metadata generator code it is creating mkdir and rmdir configure/unconfigure actions for the reconciler bundle. 

We currently have clients who are still using the metadata generator (instead of the publisher) and this metadata will cause problems in shared installed scenarios. 

We should change the metadata generator code in R3.6.x to not do this. I'm sure not if we need to make the code change in HEAD since we plan on removing the generator soon. (in the 3.7 release I believe?)
Comment 1 DJ Houghton CLA 2010-12-08 11:35:12 EST
Created attachment 184795 [details]
patch

Patch to remove the rmdir command from the unconfigure action for the reconciler bundle.
We've left the mkdir as-is.
Comment 2 John Arthorne CLA 2010-12-08 13:03:59 EST
To give more background, in the metadata generator these actions were hard-coded as instructions on the p2.reconciler.dropins bundle. In the publisher, this hard-coding was removed entirely, and instead the mkdir action is done at the product level (whether dropins should exist, and where it should go, is a product-level concern). The rmdir action for dropins was removed entirely when we switched to the publisher.
Comment 3 DJ Houghton CLA 2010-12-08 14:27:55 EST
Released to R36x branch.