Re: [PATCH 1/4] remote-mediawiki: add namespace support

2017-10-30 Thread Matthieu Moy
Antoine Beaupré writes: > @@ -319,6 +341,10 @@ sub get_mw_pages { > $user_defined = 1; > get_mw_tracked_categories(\%pages); > } > +if (@tracked_namespaces) { > +$user_defined = 1; > +get_mw_tracked_namespaces(\%pages); > +} > if (!$

Re: [PATCH 1/4] remote-mediawiki: add namespace support

2017-10-29 Thread Antoine Beaupré
On 2017-10-29 23:08:00, Kevin wrote: > So I shared the patch some time ago (~2 years). Surprisingly its just > now getting attention. I guess some renewed interest in using mediawiki > with git. I think what's happening is that someone (ie. me :p) figured it was about frigging time to actually sen

Re: [PATCH 1/4] remote-mediawiki: add namespace support

2017-10-29 Thread Kevin
So I shared the patch some time ago (~2 years). Surprisingly its just now getting attention. I guess some renewed interest in using mediawiki with git. Myself, however, am no longer using mediawiki. Nor am I completely clear on what the reasons were for using some variable or another a couple of ye

Re: [PATCH 1/4] remote-mediawiki: add namespace support

2017-10-29 Thread Eric Sunshine
On Sun, Oct 29, 2017 at 2:29 PM, Antoine Beaupré wrote: > On 2017-10-29 13:24:03, Eric Sunshine wrote: >> On Sun, Oct 29, 2017 at 12:08 PM, Antoine Beaupré wrote: >> So, the idea is that if the input has form "something:number", then >> you want to look up "something" as a namespace name. Anythin

Re: [PATCH 1/4] remote-mediawiki: add namespace support

2017-10-29 Thread Antoine Beaupré
On 2017-10-29 13:24:03, Eric Sunshine wrote: > On Sun, Oct 29, 2017 at 12:08 PM, Antoine Beaupré wrote: >> From: Kevin >> >> this introduces a new remote.origin.namespaces argument that is a > > s/this/This/ ack. >> space-separated list of namespaces. the list of pages extract is then > > s/the

Re: [PATCH 1/4] remote-mediawiki: add namespace support

2017-10-29 Thread Eric Sunshine
On Sun, Oct 29, 2017 at 12:08 PM, Antoine Beaupré wrote: > From: Kevin > > this introduces a new remote.origin.namespaces argument that is a s/this/This/ > space-separated list of namespaces. the list of pages extract is then s/the/The/ > taken from all the specified namespaces. > > Reviewed-

[PATCH 1/4] remote-mediawiki: add namespace support

2017-10-29 Thread Antoine Beaupré
From: Kevin this introduces a new remote.origin.namespaces argument that is a space-separated list of namespaces. the list of pages extract is then taken from all the specified namespaces. Reviewed-by: Antoine Beaupré Signed-off-by: Antoine Beaupré --- contrib/mw-to-git/git-remote-mediawiki.p