git/copy.c, branch v2.6.2

copy.c: make copy_fd() report its status silently

2015-05-19T21:48:54Z

When copy_fd() function encounters errors, it emits error messages itself, which makes it impossible for callers to take responsibility for reporting errors, especially when they want to ignore certain errors. Move the error reporting to its callers in preparation. - copy_file() and copy_file_with_time() by indirection get their own calls to error(). - hold_lock_file_for_append(), when told to die on error, used to exit(128) relying on the error message from copy_fd(), but now it does its own die() instead. Note that the callers that do not pass LOCK_DIE_ON_ERROR need to be adjusted for this change, but fortunately there is none ;-) - filter_buffer_or_fd() has its own error() already, in addition to the message from copy_fd(), so this will change the output but arguably in a better way. Signed-off-by: Junio C Hamano

copy_fd(): do not close the input file descriptor

2014-08-28T17:25:14Z

The caller, not this function, opened the file descriptor; it is selfish for the callee to close it when it is done reading from it. The caller may want an option to rewind and re-read the contents after it returns. Simplify the loop to copy the input in full to the output; its body essentially is what a call to write_in_full() helper does. Signed-off-by: Steffen Prohaska Helped-by: Jeff King Signed-off-by: Junio C Hamano

preserve mtime of local clone

2009-09-13T08:32:26Z

A local clone without hardlinks copies all objects, including dangling ones, to the new repository. Since the mtimes are renewed, those dangling objects cannot be pruned by "git gc --prune", even if they would have been old enough for pruning in the original repository. Instead, preserve mtime during copy. "git gc --prune" will then work in the clone just like it did in the original. Signed-off-by: Clemens Buchacher Signed-off-by: Junio C Hamano

copy.c: copy_fd - correctly report write errors

2008-04-23T04:21:08Z

Previously, the errno could have been lost due to an intervening close() call. This patch also contains minor cosmetic changes. Signed-off-by: Ariel Badichi Signed-off-by: Junio C Hamano

Use a single implementation and API for copy_file()

2008-02-25T21:06:49Z

Originally by Kristian Hï¿œgsberg; I fixed the conversion of rerere, which had a different API. Signed-off-by: Daniel Barkalow Signed-off-by: Junio C Hamano

War on whitespace

2007-06-07T07:04:01Z

This uses "git-apply --whitespace=strip" to fix whitespace errors that have crept in to our source files over time. There are a few files that need to have trailing whitespaces (most notably, test vectors). The results still passes the test, and build result in Documentation/ area is unchanged. Signed-off-by: Junio C Hamano

Ensure return value from xread() is always stored into an ssize_t

2007-05-16T04:16:03Z

This patch fixes all calls to xread() where the return value is not stored into an ssize_t. The patch should not have any effect whatsoever, other than putting better/more appropriate type names on variables. Signed-off-by: Johan Herland Signed-off-by: Junio C Hamano

copy_fd: close ifd on error

2005-12-27T18:49:25Z

In copy_fd when write fails we ought to close input file descriptor. Signed-off-by: Sam Ravnborg Signed-off-by: Junio C Hamano

xread/xwrite: do not worry about EINTR at calling sites.

2005-12-20T02:28:16Z

We had errno==EINTR check after read(2)/write(2) sprinkled all over the places, always doing continue. Consolidate them into xread()/xwrite() wrapper routines. Credits for suggestion goes to HPA -- bugs are mine. Signed-off-by: Junio C Hamano

copy.c::copy_fd() - do not leak file descriptor on error return.

2005-11-05T19:02:56Z

Signed-off-by: Junio C Hamano