Hi Sam, Sam Vilain wrote: > On Sat, 2010-09-04 at 22:15 -0500, Jonathan Nieder wrote: >> It works like this: >> >> frontend: >> feature report-fd=3 >> commit refs/heads/master >> ... revision 1 ... >> >> importer: >> abc7856cba76bca87a65bca76bca8bca98bca78bca76 > > This is probably quite a late comment, but I don't think that > 'report-fd=3' is a good idea in a protocol like this. Yes, I guess "feature report-fd=3" is a protocol layering violation. Unfortunately import-marks and export-marks have already set a precedent. How about a "feature report-fd" (with no argument) that checks if the report-fd was set and errors out if not? Well-behaved streams could use that and rely on the fd to be set on the command line, while poorly-behaved streams could still use "feature report-fd=whatever" to get the effect of --report-fd=whatever and avoid breaking UI consistency. -- 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