unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Magali <magalilemes00@gmail.com>
To: "Gábor Boskovits" <boskovits@gmail.com>,
	zimoun <zimon.toutoune@gmail.com>,
	"Guix Devel" <guix-devel@gnu.org>
Subject: [Outreachy] Feedback on 'guix git log' subcommand
Date: Thu, 28 Jan 2021 00:53:27 -0300	[thread overview]
Message-ID: <a8834240-0d7f-94db-fd68-10a51837f1f4@gmail.com> (raw)

Hi, Guix!

I would like to hear tips and suggestions regarding what's been done so
far with the 'guix git log' subcommand, part of my Outreachy internship.
On a side note, I've been blogging about it at
<https://magalilemes.gitlab.io/blog>.

The code can be found on the wip-guix-log branch and also at
<https://gitlab.com/magalilemes/guix>. So in order to retrieve it either:

In the Guix Git repository:

$ git checkout wip-guix-log

or

$ git clone git@gitlab.com:magalilemes/guix.git
$ cd guix

Below are a few examples of the options currently available:

./pre-inst-env guix git log --format=medium

./pre-inst-env guix git log --oneline

./pre-inst-env guix git log --channel-cache-path

./pre-inst-env guix git log --channel-cache-path=guix

The '--format=FORMAT' option allows FORMAT to be oneline, medium or full.
The '--channel-cache-path' without any argument shows a list of all
channels and their checkout path. If you provide a channel as an
argument, then only that channel path will be shown.
Notice, also, that when the subcommand displays commits, all commits
from all channels are shown. That's done by appending the commits,
retrieved with the commit-closure procedure, from each channel. Any
suggestions on how to sort all of these commits?

There's also the '--pretty' option, whose argument is a string. For
example, it could be invoked like ./pre-inst-env guix git log
--pretty="Subject: %s". This option works using regular expressions and
there are only five placeholders. Is it worth adding more?

Another thing is that the command is a bit slower than 'git log' itself.
Thoughts on how that could be improved?


Cheers,
Magali




             reply	other threads:[~2021-01-28  3:53 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-28  3:53 Magali [this message]
2021-01-28 23:07 ` [Outreachy] Feedback on 'guix git log' subcommand zimoun
2021-01-30  1:30   ` Magali
2021-02-01  7:27     ` zimoun
2021-02-01 15:18   ` Ludovic Courtès
2021-01-29  0:04 ` [Outreachy] 'guix git log' slowness? zimoun

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: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=a8834240-0d7f-94db-fd68-10a51837f1f4@gmail.com \
    --to=magalilemes00@gmail.com \
    --cc=boskovits@gmail.com \
    --cc=guix-devel@gnu.org \
    --cc=zimon.toutoune@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://git.savannah.gnu.org/cgit/guix.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).