Hi Vijay, vijayaguru wrote:
The attached patch adds a test case for issue #3471:svn up touches file w/ lock & svn:keywords property and marking it as 'XFail' until the issue is fixed.
Thanks for the patch. I reviewed it and I could see the patch does not test the actual problem. The issue says the timestamp changes for the file hence 'make' rebuilds stuff. In this case you have tested whether the keyword is expanded or not, which is a valid check for this use case, but apart from that you must also test whether an update happens for the file or not, because if someone fixes the keyword expansion thingy, and still something else causes an update of the file's timestamp, that is also not acceptable, since 1.4.x didn't behave in such a way. Hence do a check for timestamp on the file after an update.
PS: See that all lines in your patch is within 79 columns. Thank You. -- Senthil Kumaran S http://www.stylesen.org/
