diff options
author | Jim Cromie <jim.cromie@gmail.com> | 2009-12-31 00:09:03 -0500 |
---|---|---|
committer | Rafael Garcia-Suarez <rgs@consttype.org> | 2010-01-03 15:12:43 +0100 |
commit | ac1cbfb08dc1370fbe2937f6bdd95a86fa70bce4 (patch) | |
tree | 1d573f96588d08dbd694e0e5751b2e0718ece31f /pod/perlrepository.pod | |
parent | 39987fc8b0e8ebad4623529024d583219a7b856c (diff) | |
download | perl-ac1cbfb08dc1370fbe2937f6bdd95a86fa70bce4.tar.gz |
advice regarding make regen changes
Diffstat (limited to 'pod/perlrepository.pod')
-rw-r--r-- | pod/perlrepository.pod | 17 |
1 files changed, 13 insertions, 4 deletions
diff --git a/pod/perlrepository.pod b/pod/perlrepository.pod index eb1737c8f1..46fbf66718 100644 --- a/pod/perlrepository.pod +++ b/pod/perlrepository.pod @@ -399,6 +399,17 @@ $install_root/lib. If you are unsure about the proper location of a file that may have gotten copied while building the source distribution, consult the C<MANIFEST>. +As a special case, several files are regenerated by 'make regen' if +your patch alters C<embed.fnc>. These are needed for compilation, but +are included in the distribution so that you can build perl without +needing another perl to generate the files. You must test with these +regenerated files, but it is preferred that you instead note that +'make regen is needed' in both the email and the commit message, and +submit your patch without them. If you're submitting a series of +patches, it might be best to submit the regenerated changes +immediately after the source-changes that caused them, so as to have +as little effect as possible on the bisectability of your patchset. + =for XXX What should we recommend about binary files now? Do we need anything? @@ -418,7 +429,6 @@ important to write a good commit message. Your commit message should start with a description of the problem that the patch corrects or new functionality that the patch adds. - As a general rule of thumb, your commit message should let a programmer with a reasonable familiarity with the Perl core quickly understand what you were trying to do, how you were trying to do it and why the change @@ -428,7 +438,8 @@ matters to Perl. =item What -Your commit message should describe what part of the Perl core you're changing and what you expect your patch to do. +Your commit message should describe what part of the Perl core you're +changing and what you expect your patch to do. =item Why @@ -448,8 +459,6 @@ month or next year. =back - - =item Comments, Comments, Comments Be sure to adequately comment your code. While commenting every line |