perl/Makefile.PL: warn about duplicate module list in perl/Makefile
authorJonathan Nieder <jrnieder@gmail.com>
Fri, 15 Jun 2012 18:14:46 +0000 (13:14 -0500)
committerJunio C Hamano <gitster@pobox.com>
Fri, 15 Jun 2012 20:22:52 +0000 (13:22 -0700)
Adding or removing a module requires modifying both files to support
builds with and without MakeMaker.  Add a comment to remind patch
authors and reviewers at the crucial moment.

Longer term, it would be nicer to maintain a single list, perhaps in a
separate file used by both build systems.

Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
perl/Makefile.PL

index 2c20290fc38b17718f014f19dcb1c70f3f5826c5..b54b04a619b0aa338cae37ade9350bb52f2edd05 100644 (file)
@@ -24,6 +24,10 @@ endif
 MAKE_FRAG
 }
 
+# XXX. When editing this list:
+#
+# * Please update perl/Makefile, too.
+# * Don't forget to test with NO_PERL_MAKEMAKER=YesPlease
 my %pm = (
        'Git.pm' => '$(INST_LIBDIR)/Git.pm',
        'Git/I18N.pm' => '$(INST_LIBDIR)/Git/I18N.pm',