git-check-attr previously always returned success. With this patch it returns success when the requested attribute is found for all supplied paths. This lets you check in a script whether a file has an attribute: $ git-check-attr attribute -- file && echo "file has attribute" Signed-off-by: Andy Parkins <andyparkins@xxxxxxxxx> --- This is handy in itself, but I added it to support the tests in the next patch. builtin-check-attr.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/builtin-check-attr.c b/builtin-check-attr.c index 47b0721..b51c2ae 100644 --- a/builtin-check-attr.c +++ b/builtin-check-attr.c @@ -9,6 +9,7 @@ int cmd_check_attr(int argc, const char **argv, const char *prefix) { struct git_attr_check *check; int cnt, i, doubledash; + int allset = 1; doubledash = -1; for (i = 1; doubledash < 0 && i < argc; i++) { @@ -43,7 +44,9 @@ int cmd_check_attr(int argc, const char **argv, const char *prefix) (check[j].isset < 0) ? "unspecified" : (check[j].isset == 0) ? "unset" : "set"); + if( check[j].isset <= 0 ) + allset = 0; } } - return 0; + return !allset; } -- 1.5.1.1.821.g88bdb - 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