From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-5.0 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_HI,RCVD_IN_SORBS_SPAM, RP_MATCHES_RCVD shortcircuit=no autolearn=ham autolearn_force=no version=3.4.0 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by dcvr.yhbt.net (Postfix) with ESMTP id 40B7E2035F for ; Tue, 8 Nov 2016 23:58:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750953AbcKHX6l (ORCPT ); Tue, 8 Nov 2016 18:58:41 -0500 Received: from mail-yw0-f194.google.com ([209.85.161.194]:35645 "EHLO mail-yw0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750724AbcKHX6k (ORCPT ); Tue, 8 Nov 2016 18:58:40 -0500 Received: by mail-yw0-f194.google.com with SMTP id b66so4700999ywh.2 for ; Tue, 08 Nov 2016 15:58:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=cDgn15X2PW6WyvBdl/aQR4B6ge0onF50yMdOuokT9iE=; b=CPsbK7NfhOk34ENqhry97Eg+3XlpvQQacqVOrue8hInOtBRRakps/UFeNvQ/rD162Y MHyeFEwnZHgN9GYNnhB9Vh4zZfwY5JupwAfFJDnrpC99fvU2PNw7GIaKKmFiBzu9tvFa gFJACCtQTu5rsOczTAfbby6SABqFBIvlLjHeLScimrVOXQ26F8fLDKO1fwf28sXAy0rB 72xQroDdAxqRQsdMM97cFw6tCNpROObJ4gpwMTc5JBEbihHQOvAlE4DNT341Yh35d8gr 7n2Vu6rAvpAuePNrzZYbmq9DlLW2DDbXMnJj9tbbMx2qg+7WMG7jqOSgRuUHXbSgQ7aD +SoQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=cDgn15X2PW6WyvBdl/aQR4B6ge0onF50yMdOuokT9iE=; b=HQl2tYgqA3guix5aFX1cEJTVQ4fwl5LW8sgGohHykl2B/3cPDOthROpJgz6OEWBFKr JOoRT0rYHWBvsUGOXXTfxp9fJtpHqbuPchsm67gvYOEP1zxMVFw7sa1MBhR6PLQCCIBP uDg+73hBd0y1x047LJboBWRZ9/MST9Z/y7XqyPVT3ev+gNaVnA97U7V+kiEmfaod0DQa jAi1+QQhIUzknkQ7z4FiVDwtyvm29D9lQwsGFH5gQv9c5C9ujiyUaFIWWhT8dIH/wOAG JIzzHIfM6YUzU2MEqVdMQdQOiDHJI5BME6lurKhqpzDMut8+1S3HfE5hpzW4DkfssCKO anbA== X-Gm-Message-State: ABUngveD/3ndW5sN9zRPOaozuMmXBBCTqUfSUCaKfEDcdOaOv/LpKKSRMT9EQdGkSUPjYQUxUV+dDPe6s0nv2A== X-Received: by 10.129.74.65 with SMTP id x62mr14282479ywa.59.1478649519630; Tue, 08 Nov 2016 15:58:39 -0800 (PST) MIME-Version: 1.0 Received: by 10.37.33.132 with HTTP; Tue, 8 Nov 2016 15:58:19 -0800 (PST) In-Reply-To: <20161108201211.25213-14-Karthik.188@gmail.com> References: <20161108201211.25213-1-Karthik.188@gmail.com> <20161108201211.25213-14-Karthik.188@gmail.com> From: Jacob Keller Date: Tue, 8 Nov 2016 15:58:19 -0800 Message-ID: Subject: Re: [PATCH v7 13/17] ref-filter: add `:dir` and `:base` options for ref printing atoms To: Karthik Nayak Cc: Git mailing list Content-Type: text/plain; charset=UTF-8 Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org On Tue, Nov 8, 2016 at 12:12 PM, Karthik Nayak wrote: > From: Karthik Nayak > > Add the options `:dir` and `:base` to all ref printing ('%(refname)', > '%(symref)', '%(push)' and '%(upstream)') atoms. The `:dir` option gives > the directory (the part after $GIT_DIR/) of the ref without the > refname. The `:base` option gives the base directory of the given > ref (i.e. the directory following $GIT_DIR/refs/). > Nice, this seems useful. > Add tests and documentation for the same. > > Signed-off-by: Karthik Nayak > --- > Documentation/git-for-each-ref.txt | 34 +++++++++++++++++++--------------- > ref-filter.c | 29 +++++++++++++++++++++++++---- > t/t6300-for-each-ref.sh | 24 ++++++++++++++++++++++++ > 3 files changed, 68 insertions(+), 19 deletions(-) > > diff --git a/Documentation/git-for-each-ref.txt b/Documentation/git-for-each-ref.txt > index 600b703..f4ad297 100644 > --- a/Documentation/git-for-each-ref.txt > +++ b/Documentation/git-for-each-ref.txt > @@ -96,7 +96,9 @@ refname:: > slash-separated path components from the front of the refname > (e.g., `%(refname:strip=2)` turns `refs/tags/foo` into `foo`. > `` must be a positive integer. If a displayed ref has fewer > - components than ``, the command aborts with an error. > + components than ``, the command aborts with an error. For the base > + directory of the ref (i.e. foo in refs/foo/bar/boz) append > + `:base`. For the entire directory path append `:dir`. > > objecttype:: > The type of the object (`blob`, `tree`, `commit`, `tag`). > @@ -114,22 +116,23 @@ objectname:: > > upstream:: > The name of a local ref which can be considered ``upstream'' > - from the displayed ref. Respects `:short` and `:strip` in the > - same way as `refname` above. Additionally respects `:track` > - to show "[ahead N, behind M]" and `:trackshort` to show the > - terse version: ">" (ahead), "<" (behind), "<>" (ahead and > - behind), or "=" (in sync). `:track` also prints "[gone]" > - whenever unknown upstream ref is encountered. Append > - `:track,nobracket` to show tracking information without > - brackets (i.e "ahead N, behind M"). Has no effect if the ref > - does not have tracking information associated with it. > + from the displayed ref. Respects `:short`, `:strip`, `:base` > + and `:dir` in the same way as `refname` above. Additionally > + respects `:track` to show "[ahead N, behind M]" and > + `:trackshort` to show the terse version: ">" (ahead), "<" > + (behind), "<>" (ahead and behind), or "=" (in sync). `:track` > + also prints "[gone]" whenever unknown upstream ref is > + encountered. Append `:track,nobracket` to show tracking > + information without brackets (i.e "ahead N, behind M"). Has > + no effect if the ref does not have tracking information > + associated with it. > > push:: > The name of a local ref which represents the `@{push}` > location for the displayed ref. Respects `:short`, `:strip`, > - `:track`, and `:trackshort` options as `upstream` > - does. Produces an empty string if no `@{push}` ref is > - configured. > + `:track`, `:trackshort`, `:base` and `:dir` options as > + `upstream` does. Produces an empty string if no `@{push}` ref > + is configured. > At this point would it make more sense to document the extra values here in one block separately? For example, the upstream atom is getting pretty complex with all those options. Additionally, some of the options can be combined, like nobracket, but others cannot be comined so It may be worth documenting how and when those combinations work? > HEAD:: > '*' if HEAD matches current ref (the checked out branch), ' ' > @@ -169,8 +172,9 @@ if:: > > symref:: > The ref which the given symbolic ref refers to. If not a > - symbolic ref, nothing is printed. Respects the `:short` and > - `:strip` options in the same way as `refname` above. > + symbolic ref, nothing is printed. Respects the `:short`, > + `:strip`, `:base` and `:dir` options in the same way as > + `refname` above. > > In addition to the above, for commit and tag objects, the header > field names (`tree`, `parent`, `object`, `type`, and `tag`) can > diff --git a/ref-filter.c b/ref-filter.c > index 7d3d3a6..b47b900 100644 > --- a/ref-filter.c > +++ b/ref-filter.c > @@ -31,7 +31,7 @@ struct if_then_else { > }; > > struct refname_atom { > - enum { R_NORMAL, R_SHORT, R_STRIP } option; > + enum { R_BASE, R_DIR, R_NORMAL, R_SHORT, R_STRIP } option; > unsigned int strip; > }; > > @@ -93,7 +93,11 @@ static void refname_atom_parser_internal(struct refname_atom *atom, > atom->option = R_STRIP; > if (strtoul_ui(arg, 10, &atom->strip) || atom->strip <= 0) > die(_("positive value expected refname:strip=%s"), arg); > - } else > + } else if (!strcmp(arg, "dir")) > + atom->option = R_DIR; > + else if (!strcmp(arg, "base")) > + atom->option = R_BASE; > + else > die(_("unrecognized %%(%s) argument: %s"), name, arg); > } > > @@ -252,7 +256,6 @@ static void if_atom_parser(struct used_atom *atom, const char *arg) > die(_("unrecognized %%(if) argument: %s"), arg); > } > > - > static struct { > const char *name; > cmp_type cmp_type; > @@ -1096,7 +1099,25 @@ static const char *show_ref(struct refname_atom *atom, const char *refname) > return shorten_unambiguous_ref(refname, warn_ambiguous_refs); > else if (atom->option == R_STRIP) > return strip_ref_components(refname, atom->strip); > - else > + else if (atom->option == R_BASE) { > + const char *sp, *ep; > + > + if (skip_prefix(refname, "refs/", &sp)) { > + ep = strchr(sp, '/'); > + if (!ep) > + return ""; > + return xstrndup(sp, ep - sp); > + } > + return ""; > + } else if (atom->option == R_DIR) { > + const char *sp, *ep; > + > + sp = refname; > + ep = strrchr(sp, '/'); > + if (!ep) > + return ""; > + return xstrndup(sp, ep - sp); > + } else > return refname; > } > > diff --git a/t/t6300-for-each-ref.sh b/t/t6300-for-each-ref.sh > index 7ca0a12..8ff6568 100755 > --- a/t/t6300-for-each-ref.sh > +++ b/t/t6300-for-each-ref.sh > @@ -53,12 +53,18 @@ test_atom head refname refs/heads/master > test_atom head refname:short master > test_atom head refname:strip=1 heads/master > test_atom head refname:strip=2 master > +test_atom head refname:dir refs/heads > +test_atom head refname:base heads > test_atom head upstream refs/remotes/origin/master > test_atom head upstream:short origin/master > test_atom head upstream:strip=2 origin/master > +test_atom head upstream:dir refs/remotes/origin > +test_atom head upstream:base remotes > test_atom head push refs/remotes/myfork/master > test_atom head push:short myfork/master > test_atom head push:strip=1 remotes/myfork/master > +test_atom head push:dir refs/remotes/myfork > +test_atom head push:base remotes > test_atom head objecttype commit > test_atom head objectsize 171 > test_atom head objectname $(git rev-parse refs/heads/master) > @@ -600,4 +606,22 @@ test_expect_success 'Verify usage of %(symref:strip) atom' ' > test_cmp expected actual > ' > > +cat >expected < +refs/heads > +EOF > + > +test_expect_success 'Verify usage of %(symref:dir) atom' ' > + git for-each-ref --format="%(symref:dir)" refs/heads/sym > actual && > + test_cmp expected actual > +' > + > +cat >expected < +heads > +EOF > + > +test_expect_success 'Verify usage of %(symref:base) atom' ' > + git for-each-ref --format="%(symref:base)" refs/heads/sym > actual && > + test_cmp expected actual > +' > + > test_done > -- > 2.10.2 >