Excerpts from Junio C Hamano's message of Sun Mar 20 21:02:13 -0400 2011: Hi Junio, > > This patch uses changes the first check in the test to rm the symlink > > and then create the new symlink using only -s. > > s/uses //; I think. Yes, I caught that right after send. A follow up corrected it, but I'll resubmit taking your feedback below into account. > It is curious why "rm symlink.bin" is not inside the > test-have-prereq part, iow, immediately before the problematic "ln > -s", where it logically belongs to, though. What does the updated > code do on systems without symbolic link support? I think you're right about this. They should be logically grouped. I'll send an update shortly. Thanks -Ben -- Ben Walton Systems Programmer - CHASS University of Toronto C:416.407.5610 | W:416.978.4302 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html