From: Thomas Adam <thomas@xteddy.org>
To: "Antoine Beaupré" <anarcat@debian.org>
Cc: Thomas Adam <thomas@xteddy.org>, git@vger.kernel.org
Subject: Re: [PATCH v3 7/7] remote-mediawiki: show progress while fetching namespaces
Date: Sat, 4 Nov 2017 09:57:40 +0000 [thread overview]
Message-ID: <20171104095740.hhdgcqgtyjejmqjm@laptop.local> (raw)
In-Reply-To: <87o9ok46qu.fsf@curie.anarc.at>
On Thu, Nov 02, 2017 at 07:10:17PM -0400, Antoine Beaupré wrote:
> Actually, is there a standard way to do this in git with Perl
> extensions? I know about "option verbosity N" but how should I translate
> this into Perl? Carp? Warn? Log::Any? Log4perl?
No, not really. From a quick glance at some of the existing perl code in git,
a lot of it continues to use "print STDERR" -- but then to be fair, a lot of
the perl code also reads like it has been written by C programmers...
While there's nothing wrong with using "print STDERR", it's probably wiser to
transition this to using Carp in the long run -- it would decrease the
round-trip time to debugging should there be a situation where that was
needed, and hence I would recommend using "warn" for less-severe
errors/debugging.
-- Thomas Adam
next prev parent reply other threads:[~2017-11-04 9:57 UTC|newest]
Thread overview: 78+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-29 16:08 [PATCH 0/4] WIP: git-remote-media wiki namespace support Antoine Beaupré
2017-10-29 16:08 ` [PATCH 1/4] remote-mediawiki: add " Antoine Beaupré
2017-10-29 17:24 ` Eric Sunshine
2017-10-29 18:29 ` Antoine Beaupré
2017-10-29 20:07 ` Eric Sunshine
2017-10-29 23:08 ` Kevin
2017-10-30 2:14 ` Antoine Beaupré
2017-10-30 2:51 ` [PATCH v2 0/7] " Antoine Beaupré
2017-10-30 2:51 ` [PATCH 1/7] " Antoine Beaupré
2017-10-30 2:51 ` [PATCH 2/7] remote-mediawiki: allow fetching namespaces with spaces Antoine Beaupré
2017-10-30 2:51 ` [PATCH 3/7] remote-mediawiki: show known namespace choices on failure Antoine Beaupré
2017-10-30 2:51 ` [PATCH 4/7] remote-mediawiki: skip virtual namespaces Antoine Beaupré
2017-11-01 13:52 ` Eric Sunshine
2017-11-01 16:45 ` Antoine Beaupré
2017-11-02 1:24 ` Junio C Hamano
2017-11-02 21:20 ` Antoine Beaupré
2017-11-06 0:38 ` Junio C Hamano
2017-10-30 2:51 ` [PATCH 5/7] remote-mediawiki: support fetching from (Main) namespace Antoine Beaupré
2017-11-01 19:56 ` Eric Sunshine
2017-11-02 21:19 ` Antoine Beaupré
2017-10-30 2:51 ` [PATCH 6/7] remote-mediawiki: process namespaces in order Antoine Beaupré
2017-11-01 19:59 ` Eric Sunshine
2017-10-30 2:51 ` [PATCH 7/7] remote-mediawiki: show progress while fetching namespaces Antoine Beaupré
2017-11-01 20:01 ` Eric Sunshine
2017-11-02 21:25 ` [PATCH v3 0/7] remote-mediawiki: namespace support Antoine Beaupré
2017-11-02 21:25 ` [PATCH v3 1/7] remote-mediawiki: add " Antoine Beaupré
2017-11-02 21:25 ` [PATCH v3 2/7] remote-mediawiki: allow fetching namespaces with spaces Antoine Beaupré
2017-11-02 21:25 ` [PATCH v3 3/7] remote-mediawiki: show known namespace choices on failure Antoine Beaupré
2017-11-02 21:25 ` [PATCH v3 4/7] remote-mediawiki: skip virtual namespaces Antoine Beaupré
2017-11-02 22:43 ` Eric Sunshine
2017-11-02 22:54 ` Antoine Beaupré
2017-11-02 21:25 ` [PATCH v3 5/7] remote-mediawiki: support fetching from (Main) namespace Antoine Beaupré
2017-11-02 22:48 ` Eric Sunshine
2017-11-02 21:25 ` [PATCH v3 6/7] remote-mediawiki: process namespaces in order Antoine Beaupré
2017-11-02 22:49 ` Eric Sunshine
2017-11-02 21:25 ` [PATCH v3 7/7] remote-mediawiki: show progress while fetching namespaces Antoine Beaupré
2017-11-02 22:18 ` Thomas Adam
2017-11-02 22:26 ` Antoine Beaupré
2017-11-02 22:31 ` Thomas Adam
2017-11-02 23:10 ` Antoine Beaupré
2017-11-04 9:57 ` Thomas Adam [this message]
2017-11-02 22:50 ` Eric Sunshine
2017-11-06 21:19 ` [PATCH v4 0/7] remote-mediawiki: namespace support Antoine Beaupré
2017-11-06 21:19 ` [PATCH v4 1/7] remote-mediawiki: add " Antoine Beaupré
2017-11-06 21:19 ` [PATCH v4 2/7] remote-mediawiki: allow fetching namespaces with spaces Antoine Beaupré
2017-11-07 7:08 ` Thomas Adam
2017-11-07 16:03 ` Antoine Beaupré
2017-11-06 21:19 ` [PATCH v4 3/7] remote-mediawiki: show known namespace choices on failure Antoine Beaupré
2017-11-07 10:45 ` Thomas Adam
2017-11-07 16:07 ` Antoine Beaupré
2017-11-06 21:19 ` [PATCH v4 4/7] remote-mediawiki: skip virtual namespaces Antoine Beaupré
2017-11-06 21:19 ` [PATCH v4 5/7] remote-mediawiki: support fetching from (Main) namespace Antoine Beaupré
2017-11-06 21:19 ` [PATCH v4 6/7] remote-mediawiki: process namespaces in order Antoine Beaupré
2017-11-06 21:19 ` [PATCH v4 7/7] remote-mediawiki: show progress while fetching namespaces Antoine Beaupré
2017-11-07 16:06 ` [PATCH v5 0/7] namespace support Antoine Beaupré
2017-11-07 16:06 ` [PATCH v5 1/7] remote-mediawiki: add " Antoine Beaupré
2017-11-07 16:06 ` [PATCH v5 2/7] remote-mediawiki: allow fetching namespaces with spaces Antoine Beaupré
2017-11-07 16:06 ` [PATCH v5 3/7] remote-mediawiki: show known namespace choices on failure Antoine Beaupré
2017-11-07 16:06 ` [PATCH v5 4/7] remote-mediawiki: skip virtual namespaces Antoine Beaupré
2017-11-07 16:06 ` [PATCH v5 5/7] remote-mediawiki: support fetching from (Main) namespace Antoine Beaupré
2017-11-07 16:07 ` [PATCH v5 6/7] remote-mediawiki: process namespaces in order Antoine Beaupré
2017-11-07 16:07 ` [PATCH v5 7/7] remote-mediawiki: show progress while fetching namespaces Antoine Beaupré
2017-11-08 2:07 ` [PATCH v5 0/7] namespace support Junio C Hamano
2017-10-30 10:43 ` [PATCH 1/4] remote-mediawiki: add " Matthieu Moy
2017-10-29 16:08 ` [PATCH 2/4] remote-mediawiki: allow fetching namespaces with spaces Antoine Beaupré
2017-10-29 16:08 ` [PATCH 3/4] remote-mediawiki: show known namespace choices on failure Antoine Beaupré
2017-10-29 17:34 ` Eric Sunshine
2017-10-29 18:31 ` Antoine Beaupré
2017-11-04 10:57 ` Thomas Adam
2017-10-29 16:08 ` [PATCH 4/4] remote-mediawiki: allow using (Main) as a namespace and skip special namespaces Antoine Beaupré
2017-10-29 19:49 ` Eric Sunshine
2017-10-30 2:11 ` Antoine Beaupré
2017-10-30 3:49 ` Eric Sunshine
2017-10-30 2:43 ` Antoine Beaupré
2017-10-30 3:52 ` Eric Sunshine
2017-10-30 12:17 ` Antoine Beaupré
2017-10-30 10:40 ` [PATCH 0/4] WIP: git-remote-media wiki namespace support Matthieu Moy
2017-10-30 12:20 ` Antoine Beaupré
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171104095740.hhdgcqgtyjejmqjm@laptop.local \
--to=thomas@xteddy.org \
--cc=anarcat@debian.org \
--cc=git@vger.kernel.org \
/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.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.