Matthieu Moy <Matthieu.Moy@xxxxxxx> writes: > Signed-off-by: Matthieu Moy <Matthieu.Moy@xxxxxxx> > --- > contrib/mw-to-git/git-remote-mediawiki | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/contrib/mw-to-git/git-remote-mediawiki b/contrib/mw-to-git/git-remote-mediawiki > index a6ad8cf..a2da52f 100755 > --- a/contrib/mw-to-git/git-remote-mediawiki > +++ b/contrib/mw-to-git/git-remote-mediawiki > @@ -51,6 +51,9 @@ use constant EMPTY_CONTENT => "<!-- empty page -->\n"; > # used to reflect file creation or deletion in diff. > use constant NULL_SHA1 => "0000000000000000000000000000000000000000"; > > +# Used on Git's side to reflect empty edit messages on the wiki > +use constant EMPTY_MESSAGE => '*Empty MediaWiki Message*'; > + Is there a reason why this sentinel value cannot be a real empty string? Not that I am complaining, as I find it highly unlikely for a message stored in mw to be exactly "*Empty MediaWiki Message*", but just wondering. > my $remotename = $ARGV[0]; > my $url = $ARGV[1]; > > @@ -935,7 +938,7 @@ sub mw_import_revids { > > my %commit; > $commit{author} = $rev->{user} || 'Anonymous'; > - $commit{comment} = $rev->{comment} || '*Empty MediaWiki Message*'; > + $commit{comment} = $rev->{comment} || EMPTY_MESSAGE; > $commit{title} = mediawiki_smudge_filename($page_title); > $commit{mw_revision} = $rev->{revid}; > $commit{content} = mediawiki_smudge($rev->{'*'}); > @@ -1050,6 +1053,10 @@ sub mw_push_file { > my $oldrevid = shift; > my $newrevid; > > + if ($summary eq EMPTY_MESSAGE) { > + $summary = ''; > + } > + > my $new_sha1 = $diff_info_split[3]; > my $old_sha1 = $diff_info_split[2]; > my $page_created = ($old_sha1 eq NULL_SHA1); -- 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