Return an error instead to help with the libification effort. Signed-off-by: Jonathan Nieder <jrnie...@gmail.com> --- That’s the end of the series. Thanks for reading.
I was a bit torn about whether to present this as a request for comment or a patch series ready for application. On one hand it fixes a bug; on the other hand, I have very little confidence that it works well in the presence of arbitrary rev-list options. Thoughts and testing would be very welcome. Good night, Jonathan bundle.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/bundle.c b/bundle.c index 7aff369..7dd3f65 100644 --- a/bundle.c +++ b/bundle.c @@ -349,7 +349,7 @@ static int bundle_list_refs(int bundle_fd, struct rev_info *revs) free(ref); } if (!ref_count) - die("Refusing to create empty bundle."); + return error("Refusing to create empty bundle."); return 0; } -- 1.7.1.198.g8d802 -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org