Re: [PATCH v4 3/7] remote-mediawiki: show known namespace choices on failure
- From
Thomas Adam <thomas@xteddy.org>
- Date
- Nov 7, 2017, 10:45 UTC
- Message-ID
- <20171107104526.ywnapdat57vi66xy@laptop>
- In-Reply-To
- <20171106211953.27910-4-anarcat@debian.org>
On Mon, Nov 06, 2017 at 04:19:49PM -0500, Antoine Beaupré wrote:
Show 24 quoted lines
> If we fail to find a requested namespace, we should tell the user
> which ones we know about, since those were already fetched. This
> allows users to fetch all namespaces by specifying a dummy namespace,
> failing, then copying the list of namespaces in the config.
>
> Eventually, we should have a flag that allows fetching all namespaces
> automatically.
>
> Reviewed-by: Antoine Beaupré <anarcat@debian.org>
> Signed-off-by: Antoine Beaupré <anarcat@debian.org>
> ---
> contrib/mw-to-git/git-remote-mediawiki.perl | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/contrib/mw-to-git/git-remote-mediawiki.perl b/contrib/mw-to-git/git-remote-mediawiki.perl
> index a1d783789..6364d4e91 100755
> --- a/contrib/mw-to-git/git-remote-mediawiki.perl
> +++ b/contrib/mw-to-git/git-remote-mediawiki.perl
> @@ -1334,7 +1334,8 @@ sub get_mw_namespace_id {
> my $id;
>
> if (!defined $ns) {
> - print {*STDERR} "No such namespace ${name} on MediaWiki.\n";
> + my @namespaces = map { s/ /_/g; $_; } sort keys %namespaces_id;Oops. This was my typo from my original suggestion. The hash is '%namespace_id', not '%namespaces_id'. However, how did this slip through testing? I'm assuming you blindly copied this from my example, which although quick to do, is only being caught because of my sharp eyes...
-- Thomas Adam