| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
(cherry picked from commit c23f766f6c261b334740784b1baca327e8118a87)
|
|
|
|
|
|
|
|
| |
GitHub issue tracker
The perlbug utility and perlbug@perl.org should no longer be used to submit bug reports or patches.
(cherry picked from commit 8166b4e0bc220e759aa233af54ac1e60cc510f0c)
|
|
|
|
| |
(cherry picked from commit 617d9ec40b6f4f34796deb5007320a135e402779)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
repository (#1186)
* Update repository and bugtracker URLs to GitHub in makemeta
regen META files
* Update POD and comments to reference GitHub as canonical repository
* Update Porting/corelist.pl to recognize GitHub issue tracker
* remove "A note on camel and dromedary"
* Remove redundant 'Committing your changes' section
(cherry picked from commit 47ef154c59e75e9351d27b3bd06d6ac57494193c)
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
Neil Bowers and Andreas Koenig spotted that, based on GDPR, we should
clarify to people that their name and email will appear in AUTHORS when
submitting a patch and that it can be removed when asked.
Neil contributed the phrasing.
|
|
|
|
|
|
|
|
|
| |
... as was the case of my first patch to perl: I tried to find my use
case in perlhack.pod, failed, and "hoped for the best".
With this, perlhack makes clearer that attaching a patch to an already
existing RT ticket is a perfectly valid and accepted use-case for
getting patches into perl - same as using perlbug.
|
|
|
|
| |
Following recommendation by Matthew Horsfall.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
At http://perldoc.perl.org/perlhack.html#TESTING, this was rendering as:
1
)
These select Unicode rules....
2
)
If you use the form...
For: RT #126021
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This tool runs code snippets found in t/perf/benchmarks (or similar) under
cachegrind, in order to calculate how many instruction reads, data writes,
branches, cache misses, etc. that one execution of the snippet uses. It
will run them against two or more perl executables and show how much each
test has gotten better or worse.
It is modelled on the perlbench tool, but since it measures instruction
reads etc., rather than timings, it is much more precise and reproducible.
It is also considerably faster, and is capable or running tests in
parallel. Rather than displaying a single relative percentage per
test/perl combination, it displays values for 13 different measurements,
such as instruction reads, conditional branch misses etc.
This commit also changes the format of t/perf/benchmarks slightly; it
becomes an AoH rather than a HoH (to allow checking for duplicate keys),
and the test names themselves become a :: hierarchy.
|
|
|
|
| |
For: RT # 122996
|
|
|
|
|
| |
Using git's --attach with perlbug -f actually mangled the patches,
making them harder to apply, but now we have a better way to do it.
|
|
|
|
|
|
| |
Add Martin McGrath to AUTHORS.
For: RT #120819
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
In the Super Quick Patch Guide, if a change involves multiple commits, put
them all in a single perlbug attachment.
Father Chrysostomos says it's easier like this, in a comment on
RT #119599.
It simplifies the instructions, and avoiding the need to mention mail
clients.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
that Mutt can construct the email for you.
Obviously this isn't as generally relevant as the rest of the Guide, since
patchers will use many different mail clients. But it's a significant boon
for those who do use Mutt, and a very short addition to the Guide.
Mutt is singled out simply because it has this functionality; I suspect
that most other widely used mail clients don't.
Committer: Removed trailing whitespace. Applied patch manually because other
lines in the file had been rebroken and patch no longer applied cleanly.
For: RT #119599
|
|
|
|
|
|
|
|
|
|
|
|
| |
How to use perlbug when a change is a series of commits, not a single
commit.
This is the advice RJBS gave me over IRC. Including it in the guide should
avoid him having to repeat the advice to others.
Committer: Added single quotes around one keyboard command.
For: RT #119599
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add advice so that somebody wishing to submit a second patch doesn't need
to throw away their perl check-out and start again.
Not knowing the 'git clean' step caught me out, and meant perl wouldn't
build for me. Nicholas helped me out. Adding this to the guide will
hopefully save Nicholas from having to repeat that for others (especially
since others may not be fortunate enough to have Nicholas handily seated
next to them at the point they encounter it).
(The non-building was because some things in the repository had been
re-arranged since my previous patch (several months earlier), and the
latest build was getting confused by some files left over from a
pre-re-arragned build.)
The 'git clean' step will also remove the first 0001-*.patch file, avoiding
the problem of there being two files matching that glob when attaching the
second patch.
Committer: Removed trailing whitespace.
For: RT #119599
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Super Quick Patch Guide has been improved several times. Suggest that
a patcher looks at the latest version in the checkout of blead they've
just made, in case that's been improved since whichever released version
they were reading.
This has caught me out before: I've done something sub-optimal (for me or
those reviewing my patches) by diligently following out-of-date
instructions.
Remove trailing whitespace.
For: RT #119599
|
|
|
|
|
|
|
|
|
|
|
|
| |
To pass t/porting/podcheck.t --pedantic
The line-break is inside a $(...), so the lines can be copied-and-pasted
as they are, complete with line-break and extra spaces, and still give the
same output.
Remove trailing whitespace.
For: RT #119599
|
|
|
|
|
|
|
|
|
| |
In the Super Quick Patch Guide, run the perlbug and perl from the working
copy that the patch is against, so the bug report contains relevant
version and configuration data, rather than that of whichever system perl
the reporter happens to have installed.
For: RT #119599
|
|
|
|
|
|
|
|
|
| |
Most verbatim lines with shell commands had 2 spaces before the % prompt.
A few had 1 or 4 spaces. Make them all 2.
Remove trailing whitespace.
For: RT #119599
|
|
|
|
|
|
|
|
|
| |
In accordance with the comment at the top of the file, before I make other
changes to it.
Remove trailing whitespace.
For: RT #119599
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Remove these targets and their documentation:
check.third check.utf16 check.utf8 coretest minitest.utf16 test.deparse
test.taintwarn test.third test.torture test.utf16 test.utf8
test_notty.deparse test_notty.third test_prep.third torturetest ucheck
ucheck.third ucheck.utf16 ucheck.valgrind utest utest.third utest.utf16
utest.valgrind
It's still possible to run the actions these targets "by hand", if desired.
This commit simply removes the convenience targets from the Makefile,
reducing its complexity.
|
| |
|
| |
|
|
|
|
|
| |
Original document linked to has been removed because it was out of date and
redundant.
|
|
|
|
|
|
| |
directory t/opbasic.
For RT #115838
|
| |
|
|
|
|
| |
At Aristotle’s suggestion
|
| |
|
|
|
|
| |
Intended for testing 64-bit behavious
|
|
|
|
| |
This reverts commit 6a945912f2861921b440402072b7053a1dc414a5.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|