Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 459584 - "dirt" left after "copy up" of "Eclipse.app".
Summary: "dirt" left after "copy up" of "Eclipse.app".
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Releng (show other bugs)
Version: 4.5   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.5 M6   Edit
Assignee: David Williams CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 420078
  Show dependency tree
 
Reported: 2015-02-10 12:12 EST by David Williams CLA
Modified: 2015-02-12 06:23 EST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Williams CLA 2015-02-10 12:12:52 EST
One consequence of the fix for bug 457071 is that now there is (once again) some "dirt" left in working tree: 

This now shows up in "untracked files" from a 'git status'
eclipse.platform.releng.tychoeclipsebuilder/rcp.config/rootfiles/

and what is under that (the real dirt) is 
macosx.cocoa.x86_64/Eclipse.app/Contents/MacOS/eclipse

Not sure what the fix is here, unless we figure out how to avoid the "copy up".
Comment 1 David Williams CLA 2015-02-11 12:59:04 EST
Is it a legitimate fix ... or, a partial fix? ... to have a .gitignore entry that say says to ignore rcp.config/rootfiles/?

That would at least make it "cleanable" with git clean -Xd
Comment 2 David Williams CLA 2015-02-11 13:02:33 EST
Markus, Tom, ... 

Do you know enough about "git" and "dirt" to have an opinion about how helpful ".gitignore" would be? 

I'll "give it a try" as it couldn't hurt, but thought I'd ask. 

(And, if "is better that nothing" seems the same "trick" could be used in a few other places?).
Comment 3 Markus Keller CLA 2015-02-11 15:16:03 EST
Yes, .gitignore is exactly meant for such situations. I.e. for files that are generated during a build but should not be committed.

.gitignore entries are best created with a tool such as EGit. The pattern format is slightly different from .cvsignore, so you'd have to read http://git-scm.com/docs/gitignore to avoid surprises with manually created patterns.

http://git.eclipse.org/c/platform/eclipse.platform.releng.aggregator.git/commit/?id=756a0be8559cd9f1d03f1eb352651424aa189676 matches not only
/eclipse.platform.releng.tychoeclipsebuilder/rcp.config/rootfiles/*
but also
/eclipse.platform.releng.tychoeclipsebuilder/rcp.config/source/rootfiles/*
=> probably not what you meant.

EGit's "Team > Ignore" on the folder would generate a .gitignore with contents:
/rootfiles/
Comment 4 David Williams CLA 2015-02-12 03:03:02 EST
Thanks for the tip, Markus, on more precession. 

I checked tonight's nightly, and "git status" did not show it as "dirt", even though it is, still, a form of dirt. 

Also confirmed that git clean -x -d -f did remove 'rootfiles' directory (as well as /target/ (and others). 

So, all in all, in the Nightly build, left following modules as showing "modified". 

	modified:   eclipse.jdt.core (untracked content)
	modified:   eclipse.jdt.debug (untracked content)
	modified:   eclipse.jdt.ui (untracked content)
	modified:   eclipse.platform.common (untracked content)
	modified:   eclipse.platform.runtime (untracked content)
	modified:   eclipse.platform.ua (untracked content)
	modified:   eclipse.platform.ui (untracked content)
	modified:   rt.equinox.framework (untracked content)

I would think the same .gitignore settings could handle a lot of that? 
Again, if nothing else, making it easier to clean with  git clean. 
Pretty sure 'mvn clean' does not know about "git clean" ... it just cleans what it built?
Comment 5 David Williams CLA 2015-02-12 03:04:02 EST
Counting as "done" (though, someday would like to find a way to get rid of the "partial dirt" ... or, that is, never create it in the first place.