Apologies - it appears I missed on : > case. Will fix in next iteration...
jon.
On Sun, Sep 26, 2010 at 2:18 AM, Jon Seymour <jon.seymour@xxxxxxxxx> wrote:
> This change ensures that git rev-parse --flags complies with its documentation,
> namely:
>
> Â "Do not output non-flag parameters".
>
> Previously:
> Â $ git rev-parse --flags HEAD
> Â <sha1 hash of HEAD>
> Â $
>
> Now:
> Â $ git rev-parse --flags HEAD
> Â $
>
> Signed-off-by: Jon Seymour <jon.seymour@xxxxxxxxx>
> ---
> ÂDocumentation/git-rev-parse.txt | Â 24 +++++++++++++-----------
> Âbuiltin/rev-parse.c       |  Â6 +++++-
> Ât/t1510-rev-parse-flags.sh   Â|  24 +++++++++++++++++++++---
> Â3 files changed, 39 insertions(+), 15 deletions(-)
>
> diff --git a/Documentation/git-rev-parse.txt b/Documentation/git-rev-parse.txt
> index f5e6637..f26fc7b 100644
> --- a/Documentation/git-rev-parse.txt
> +++ b/Documentation/git-rev-parse.txt
> @@ -49,20 +49,22 @@ OPTIONS
> Â Â Â Â'git rev-list' command.
>
> Â--flags::
> - Â Â Â Do not output non-flag parameters which are not also revisions.
> - Â Â Â +
> - Â Â Â If specified, this option causes 'git rev-parse' to stop
> - Â Â Â interpreting remaining arguments as options for its own
> - Â Â Â consumption. As such, this option should be specified
> - Â Â Â after all other options that 'git rev-parse' is expected
> - Â Â Â to interpret.
> + Â Â Â Do not output non-flag parameters.
> ++
> +If specified, this option causes 'git rev-parse' to stop
> +interpreting remaining arguments as options for its own
> +consumption. As such, this option should be specified
> +after all other options that 'git rev-parse' is expected
> +to interpret.
> ++
> +If `--flags` is specified, `--no-revs` is implied.
>
> Â--no-flags::
> Â Â Â ÂDo not output flag parameters.
> - Â Â Â +
> - Â Â Â If both `--flags` and `--no-flags` are specified, the first
> - Â Â Â option specified wins and the other option is treated like
> - Â Â Â a non-option argument.
> ++
> +If both `--flags` and `--no-flags` are specified, the first
> +option specified wins and the other option is treated like
> +a non-option argument.
>
> Â--default <arg>::
> Â Â Â ÂIf there is no parameter given by the user, use `<arg>`
> diff --git a/builtin/rev-parse.c b/builtin/rev-parse.c
> index 2ad269a..0655424 100644
> --- a/builtin/rev-parse.c
> +++ b/builtin/rev-parse.c
> @@ -521,7 +521,11 @@ int cmd_rev_parse(int argc, const char **argv, const char *prefix)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcontinue;
> Â Â Â Â Â Â Â Â Â Â Â Â}
> Â Â Â Â Â Â Â Â Â Â Â Âif (!strcmp(arg, "--flags")) {
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â filter &= ~DO_NONFLAGS;
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (!(filter & DO_FLAGS)) {
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â /* prevent --flags being interpreted if --no-flags has been seen */
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â continue;
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â }
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â filter &= ~(DO_NONFLAGS|DO_REVS);
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcontinue;
> Â Â Â Â Â Â Â Â Â Â Â Â}
> Â Â Â Â Â Â Â Â Â Â Â Âif (!strcmp(arg, "--no-flags")) {
> diff --git a/t/t1510-rev-parse-flags.sh b/t/t1510-rev-parse-flags.sh
> index ef0b4ad..7df2081 100755
> --- a/t/t1510-rev-parse-flags.sh
> +++ b/t/t1510-rev-parse-flags.sh
> @@ -29,10 +29,10 @@ test_expect_success 'git rev-parse --no-revs --flags HEAD -> ""' \
> Â Â Â Âtest_cmp expected actual
> Â'
>
> -test_expect_success 'git rev-parse --symbolic --flags HEAD -> "HEAD"' \
> +test_expect_success 'git rev-parse --flags HEAD -> ""' \
> Â'
> - Â Â Â echo HEAD > expected &&
> - Â Â Â git rev-parse --symbolic --flags HEAD >actual &&
> + Â Â Â : > expected &&
> + Â Â Â git rev-parse --flags HEAD >actual &&
> Â Â Â Âtest_cmp expected actual
> Â'
>
> @@ -106,4 +106,22 @@ test_expect_success 'git rev-parse --symbolic --no-flags --flags HEAD -> "HEAD"'
> Â Â Â Âtest_cmp expected actual
> Â'
>
> +test_expect_success 'git rev-parse --no-revs file -> "file"' \
> +'
> + Â Â Â echo foo >file &&
> + Â Â Â echo file >expected &&
> + Â Â Â git rev-parse --no-revs file >actual &&
> + Â Â Â test_cmp expected actual
> +'
> +
> +test_expect_success 'git rev-parse --no-revs -- not-a-file -> "-- not-a-file"' \
> +'
> + Â Â Â cat >expected <<-EOF &&
> +--
> +not-a-file
> + Â Â Â EOF
> + Â Â Â git rev-parse --no-revs -- not-a-file >actual &&
> + Â Â Â test_cmp expected actual
> +'
> +
> Âtest_done
> --
> 1.7.3.4.g73371.dirty
>
>
--
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