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

Bug 345763

Summary:	Source file is updated during the build
Product:	[RT] Jetty	Reporter:	Michael Gorovoy <mgorovoy>
Component:	build	Assignee:	Michael Gorovoy <mgorovoy>
Status:	RESOLVED FIXED	QA Contact:
Severity:	trivial
Priority:	P3	CC:	jesse.mcconnell, jetty-inbox
Version:	7.4.0
Target Milestone:	7.2.x
Hardware:	PC
OS:	Linux
Whiteboard:

Description Michael Gorovoy

2011-05-13 13:58:55 EDT

After running a full maven build, the following message occurs then trying to push changes committed earlier to a git svn cloned repository using 'git svn dcommit'.

mgorovoy@glnxlap01:~/code/git/jetty7$ git svn dcommit
Cannot dcommit with a dirty index.  Commit your changes first, or stash them with `git stash'.
 at /usr/lib/git-core/git-svn line 499

Running 'git status' indicates clean index, but gitk indicates that the following source file has been updated.

mgorovoy@glnxlap01:~/code/git/jetty7$ git svn dcommit
Cannot dcommit with a dirty index.  Commit your changes first, or stash them with `git stash'.
 at /usr/lib/git-core/git-svn line 499

Running 'git svn rebase' restores order and allows push to proceed.

This ticket has nothing to do with git, it is only to check why a source file is being touch'ed during the maven build.

Comment 1 Michael Gorovoy

2011-05-13 14:01:57 EDT

It would have been helpful if I pasted the file information into the description... Here it goes.

mgorovoy@glnxlap01:~/code/git/jetty7$ ls -l jetty-policy/src/test/resources/monitor-test-2/global-all-permission.policy 
-rw-r--r-- 1 mgorovoy mgorovoy 51 2011-05-13 13:34 jetty-policy/src/test/resources/monitor-test-2/global-all-permission.policy

Comment 2 Jesse McConnell

2011-05-13 14:06:26 EDT

nothing for us to do with this

if your on git, your on your own, nothing we are doing intrinsically that would cause this

Comment 3 Michael Gorovoy

2011-05-13 15:40:00 EDT

This file appears to be touch'ed by PolityMonitorTest.

<snip>
permFile = MavenTestingUtils.getTestResourceFile("monitor-test-2/global-all-permission.policy");
        
permFile.setLastModified(System.currentTimeMillis());
</snip>

I'll take a closer look how this could be avoided over the weekend.

-Michael

Comment 4 Michael Gorovoy

2011-05-14 00:49:14 EDT

Issue is confirmed, working on the fix.

Comment 5 Michael Gorovoy

2011-05-14 01:04:26 EDT

Committed r3146