Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 369080

Summary: flatten structure of Mylyn Review repository
Product: z_Archived Reporter: Steffen Pingel <steffen.pingel>
Component: MylynAssignee: Steffen Pingel <steffen.pingel>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: kilian.matt, mario.bernhart, sascha.scholz
Version: 0.8   
Target Milestone: 0.9   
Hardware: PC   
OS: Windows Vista   
Whiteboard:

Description Steffen Pingel CLA 2012-01-19 09:15:56 EST
As part of bug 368844 the R4E sources will move into a separate Git repository. That means the commit history of the org.eclipse.mylyn.reviews repository will be get rewritten.

Since the number of bundles that will remain in the repository is manageable I would like to propose that we use a flat structure. This is consistent with all other Mylyn sub-projects and will simplify management across repositories.

Additionally, I would like to rename the org.eclipse.mylyn.versions.tasks bundles to org.eclipse.mylyn.reviews.versions to make it clear that these bundles are part of the Mylyn Reviews project.

We would end up with the following bundles:

pre. 
org.eclipse.mylyn.gerrit.core
org.eclipse.mylyn.gerrit.core.tests
org.eclipse.mylyn.gerrit.feature
org.eclipse.mylyn.gerrit.target
org.eclipse.mylyn.gerrit.tests
org.eclipse.mylyn.gerrit.ui
org.eclipse.mylyn.gerrit.ui.tests
org.eclipse.mylyn.reviews.core
org.eclipse.mylyn.reviews.feature
org.eclipse.mylyn.reviews.sdk.feature
org.eclipse.mylyn.reviews.site
org.eclipse.mylyn.reviews.tasks.feature
org.eclipse.mylyn.reviews.tasks.core
org.eclipse.mylyn.reviews.tasks.dsl
org.eclipse.mylyn.reviews.tasks.ui
org.eclipse.mylyn.reviews.ui
org.eclipse.mylyn.reviews.versions.tasks.core
org.eclipse.mylyn.reviews.versions.tasks.ui


Renamed bundles:

pre. 
org.eclipse.mylyn.reviews.tasks                 -> org.eclipse.mylyn.reviews.tasks.feature
org.eclipse.mylyn.versions.tasks.core           -> org.eclipse.mylyn.versions.tasks.core
org.eclipse.mylyn.versions.tasks.mapper.generic -> merged into org.eclipse.mylyn.versions.tasks.core
org.eclipse.mylyn.versions.tasks.ui             -> org.eclipse.mylyn.versions.tasks.ui
Comment 1 Sascha Scholz CLA 2012-01-20 05:12:18 EST
ok
Comment 2 Steffen Pingel CLA 2012-01-25 03:04:06 EST
pre.. 

git clone --mirror git://git.eclipse.org/gitroot/mylyn/org.eclipse.mylyn.reviews.git
cd org.eclipse.mylyn.reviews.git

# remove R4E
git filter-branch --index-filter "git rm -r -f --cached --ignore-unmatch scm r4e framework/org.eclipse.mylyn.reviews.frame.core" --prune-empty -f -- --all

# clean commits
git filter-branch -f --prune-empty --tag-name-filter cat -- --all
rm -rf refs/original/
git reflog expire --expire=now --all
git gc --aggressive --prune=now
Comment 3 Steffen Pingel CLA 2012-01-25 03:08:50 EST
git mv framework/org.eclipse.mylyn.reviews.* gerrit/org.eclipse.mylyn.gerrit.* releng/org.eclipse.mylyn.reviews.site .
git rm -rf framework/ gerrit/ versions/
Comment 4 Steffen Pingel CLA 2012-01-25 04:07:43 EST
I'll decouple the structure changes from the R4E move. I have filed bug 369630 to track updating of the org.eclipse.mylyn.reviews repository.
Comment 5 Steffen Pingel CLA 2012-01-28 11:40:49 EST
I have pushed a portion of the proposed changes to the org.eclipse.mylyn.reviews.new repository. I have removed the gerrit/ framework/ releng/ and versions/ directories and moved the contained bundles to the top-level. 

I have left the tbr/ directory untouched. Kilian, I'll leave it to you to decide whether you want to implement the remaining changes proposed in the description.