From: Hariom verma <hariom18599@gmail.com>
To: Christian Couder <christian.couder@gmail.com>
Cc: git <git@vger.kernel.org>, Heba Waly <heba.waly@gmail.com>
Subject: Re: [RFC] ref-filter: add new atom `signature`
Date: Wed, 1 Jul 2020 17:25:06 +0530 [thread overview]
Message-ID: <CA+CkUQ9dW9E+yujwU0A7KYu6-fdK9Y4q31+Gq6-9AYoyLyQT6Q@mail.gmail.com> (raw)
In-Reply-To: <CAP8UFD2qbtoXay1AgwrdaJ8XgVkq=NqBTa=KptUTop19xzUVuQ@mail.gmail.com>
Hi,
On Wed, Jul 1, 2020 at 3:23 PM Christian Couder
<christian.couder@gmail.com> wrote:
>
> Hi,
>
> On Sun, Jun 28, 2020 at 12:05 PM Hariom verma <hariom18599@gmail.com> wrote:
> >
> > Hi,
> >
> > In ref-filter, we have a bunch of atoms which serves as a formatting
> > option for `git for-each-ref`.
>
> Ok.
>
> > Despite of having 'subject' as an argument to 'contents' [i.e
> > `%(contents:subject)`], we still have 'subject' as an atom [i.e
> > `%(subject)`].
> > Likewise for `%(contents:signature)`, we can have 'signature' as an atom too.
>
> Ok. Some things are available in different ways perhaps because of
> historical reasons and backward compatibility.
Okay.
> > Currently, `%(contents:signature)` internally uses `parse_signature()`
> > that only works for signed tag objects. I plan to expand the scope of
> > `signature` by defining it as an atom and will make it work for
> > commits too.
>
> Didn't you say already that "we can have 'signature' as an atom too",
> so isn't it already defined as an atom? Does `%(contents:signature)`
> work for commits while '%(signature)' doesn't? And what happens when
> it doesn't work?
Sorry for the ambiguity in my statement.
- Only `contents` exists as an atom, which accepts `signature` as an
argument. i.e '%(contents:signature)'
- 'signature' does not exist as an atom. I plan to add this.
- `%(contents: signature)` only works for tag objects. For commit it
prints nothing.
> > Also, thinking of adding `signer`, `key`, `fingerprint`,
> > `primarykeyfingerprint`, `trustlevel` and `grade`[print 'G' for good,
> > 'B' for bad...] as arguments to "signature".
>
> Ok, I guess all the above arguments would be new.
Yes
> > This change will also help me in using ref-filter's logic in pretty.c
> > for formatting options like "GG", "G?", etc. To know more about these
> > formatting options, you can take a look here[1].
> >
> > If curious about what approach I'm taking for using ref-filter logic
> > in pretty.c, see[2]
> >
> > Would love hear your thoughts on it. Any suggestions will also be appreciated.
>
> It would be nice to see, at least for the atoms or arguments that you
> are planning to, or have already, worked on, a list showing how each
> formatting option in pretty.c maps to one or more atoms possibly with
> arguments.
>
> For example something like:
>
> %GS: show the name of the signer for a signed commit
> maps to: %(signature:signer)
> status: yet to be implemented
>
Yeah, that would be nice.
## User Formats
%H: commit hash
maps to: "%(objectname)"
status: Implemented
%h: abbreviated commit hash
maps to: "%(objectname:short)"
status: Implemented
%T: tree hash
maps to: "%(tree)"
status: Implemented
%t: abbreviated tree hash
maps to: "%(tree:short)"
status: Implemented
%P: parent hashes
maps to: "%(parent)"
status: Implemented
%p: abbreviated parent hashes
maps to: "%(parent:short)"
status: Implemented
%an: author name
maps to: "%(authorname)"
status: Implemented
%ae: author email
maps to: "%(authoremail)"
status: Implemented
%ad: author date
maps to: "%(authordate)"
status: Implemented
%cn: committer name
maps to: "%(committername)"
status: Implemented
%ce: committer email
maps to: "%(committeremail)"
status: Implemented
%cd: committer date
maps to: "%(committerdate)"
status: Implemented
%s: subject
maps to: "%(subject)"
status: Implemented
%f: sanitized subject line, suitable for a filename
maps to: "%(subject:sanitize)"
status: Implemented
%b: body
maps to: "%(body)"
status: Implemented
%GG: raw verification message from GPG for a signed commit
maps to: %(signature)
status: Implemented
%G?: show "G" for a good (valid) signature, "B" for a bad signature...
maps to: "%(signature:grade)"
status: In progress
%GS: show the name of the signer for a signed commit
maps to: "%(signature:signer)"
status: Implemented
%GK: show the key used to sign a signed commit
maps to: "%(signature:key)"
status: Implemented
%GF: show the fingerprint of the key used to sign a signed commit
maps to: "%(signature:fingerprint)"
status: Implemented
%GP: show the fingerprint of the primary key whose subkey was used to
sign a signed commit
maps to: "%(signature:primarykeyfingerprint)"
status: Implemented
%GT: show the trust level for the key used to sign a signed commit
maps to: "%(signature:trustlevel)"
status: In progress
## Commit Formats
Note: This is only for the commit body[as pretty.c only handles commit
body]. First line that contains `commit hash` is handled by
log-tree.c, So no need to map it.
'oneline'
maps to: "%(subject)"
status: implemented
'short'
maps to: "Author: %(authorname) %(authoremail)\n\n\t%(subject)\n"
status: Implemented
'medium'
maps to: "Author: %(authorname)
%(authoremail)\nDate:\t%(authordate)\n\n%(subject)\n\n%(body)"
status: Implemeted
'full'
maps to: "Author: %(authorname) %(authoremail)\nCommit:
%(committername) %(committeremail)\n\n%(subject)\n\n%(body)"
status: Implemented
'fuller'
maps to: "Author:\t\t%(authorname)
%(authoremail)\nAuthorDate:\t%(authordate)\nCommit:\t\t%(committername)
%(committeremail)\nCommitDate:\t%(committerdate)\n\n%(subject)\n\n%(body)"
status: Implemented
'raw'
status: Not yet Implemented
Thanks,
Hariom
prev parent reply other threads:[~2020-07-01 11:55 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-28 10:05 [RFC] ref-filter: add new atom `signature` Hariom verma
2020-07-01 9:52 ` Christian Couder
2020-07-01 11:55 ` Hariom verma [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CA+CkUQ9dW9E+yujwU0A7KYu6-fdK9Y4q31+Gq6-9AYoyLyQT6Q@mail.gmail.com \
--to=hariom18599@gmail.com \
--cc=christian.couder@gmail.com \
--cc=git@vger.kernel.org \
--cc=heba.waly@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).