2011/6/9 Sverre Rabbelier <srabbelier@xxxxxxxxx>: > Heya, > > On Thu, Jun 9, 2011 at 15:15, Jeremie Nikaes > <jeremie.nikaes@xxxxxxxxxxxxxxx> wrote: >> +sub mw_capabilities { >> + print STDOUT "push\n"; > >> +sub mw_push { >> + print STDERR "Push not yet implemented\n"; >> +} > > If it's not supported then you shouldn't advertise it above. Is there > any particular reason you implemented it this way? E.g., did you > encounter a breakage in remote-helpers if push is not implemented? Well, if you dont give the capability to push, you get the following message error: failed to push some refs to 'mediawiki::http://mediawikiurl' Which is less explicit than 'Push not yet implemented'. Regards, -- Jérémie Nikaes -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html