[Perl/perl5] 64ed4f: t/29a_upcopy.t - under parallel builds allow more ...
Branch: refs/heads/yves/tie_file_alarm_time_increase Home: https://github.com/Perl/perl5 Commit: 64ed4ff70478e1fe96ad9a5e1505eb4c10bc3abe https://github.com/Perl/perl5/commit/64ed4ff70478e1fe96ad9a5e1505eb4c10bc3abe Author: Yves Orton Date: 2023-01-31 (Tue, 31 Jan 2023) Changed paths: M dist/Tie-File/lib/Tie/File.pm M dist/Tie-File/t/24_cache_loop.t M dist/Tie-File/t/29_downcopy.t M dist/Tie-File/t/29a_upcopy.t Log Message: --- t/29a_upcopy.t - under parallel builds allow more time for test Karl has reported that he has issues with t/29a_upcopy.t under parallel builds. I can not see any file based race conditions, but I can see code using alarm around a test I can easily imagine would be too short for a loaded box running many tests in parallel. This patch allows the test to use 20 seconds instead of 10 if TEST_JOBS or HARNESS_OPTIONS are defined in the environment. Hopefully this fixes tests on Karls box. In a previous commit Dave M raised this from 5 to 10 seconds, so lets double it again and see if Karls errors go away. In an abundance of caution I also adjusted the other two cases of using alarm() in this file to use the same logic and produce similar style error messages.
[Perl/perl5] fe46a8: embed.pl - the 's', 'S', 'i' and 'I' flags are mut...
Branch: refs/heads/yves/tidy_embed_fnc Home: https://github.com/Perl/perl5 Commit: fe46a8577afc4347d9f079368d86273affdc6fc5 https://github.com/Perl/perl5/commit/fe46a8577afc4347d9f079368d86273affdc6fc5 Author: Yves Orton Date: 2023-01-31 (Tue, 31 Jan 2023) Changed paths: M embed.fnc M embed.h M inline.h M proto.h M regen/embed.pl Log Message: --- embed.pl - the 's', 'S', 'i' and 'I' flags are mutually exclusive We had a bug where we processed the first one in the flags definition. Sorting the flags or rearranging them changes the output, which shouldn't happen. This also fixes the handling and specification of PerlEnv_putenv(), which was marked "si" when it should have been marked "i". This required changing its implementation from a Perl_ prefix to a S_ prefix and regenerating. I have run embed.pl in a loop with a local patch to shuffle the flags to see if there were any other order dependencies. No output files changed so I assume with this patch we are free of such bugs. Commit: 4d23a55748e0f583705f1e518f0674fba969a572 https://github.com/Perl/perl5/commit/4d23a55748e0f583705f1e518f0674fba969a572 Author: Yves Orton Date: 2023-01-31 (Tue, 31 Jan 2023) Changed paths: M embed.fnc M regen/tidy_embed.pl Log Message: --- embed.fnc - sort entries alphabetically by function name. This is actually a library sort (lc with underbars removed), followed by a lexicographical sort. Comment lines are sticky to the line that follows them. Somehow the original version of this patch was missed in my earlier work on tidy_embed.pl, I think I messed up a rebase somehow. I noticed it was missing when I realized that new entries werent being sorted into place correctly. While this patch creates a fair bit of churn in the file right now, long term it will make it easier to use. Also note that the *output* files have not changed, which validates that the patch did not break anything. Commit: 4601075444f0cabcd9c70768b14ce1535e161c45 https://github.com/Perl/perl5/commit/4601075444f0cabcd9c70768b14ce1535e161c45 Author: Yves Orton Date: 2023-01-31 (Tue, 31 Jan 2023) Changed paths: M embed.fnc M regen/HeaderParser.pm Log Message: --- embed.pl - sort and dedupe flags in embef.fnc as part of tidy This ensures we use a canonical string for each possible flag variant, which makes it easier to search for flags with a given flag signature. It also exposed a mutex bug in flag handling which caused PerlEnv_putenv to be improperly marked as static, when it is in fact static inline. To validate there arent any issues like this remaining i set it up so the flags were shuffled during processing and ran embed.pl in a loop for a while and none of the output files changed, so I assume there are no further such issues. Compare: https://github.com/Perl/perl5/compare/fe46a8577afc%5E...4601075444f0
[Perl/perl5] 2efdea: embed.pl - the 's', 'S', 'i' and 'I' flags are mut...
Branch: refs/heads/yves/fix_embed_flag_static_type_mutex_detection Home: https://github.com/Perl/perl5 Commit: 2efdea06b858aab36551696dcf89b4a341023518 https://github.com/Perl/perl5/commit/2efdea06b858aab36551696dcf89b4a341023518 Author: Yves Orton Date: 2023-01-31 (Tue, 31 Jan 2023) Changed paths: M embed.fnc M embed.h M inline.h M proto.h M regen/embed.pl Log Message: --- embed.pl - the 's', 'S', 'i' and 'I' flags are mutually exclusive We had a bug where we processed the first one in the flags definition. Sorting the flags or rearranging them changes the output, which shouldn't happen. This also fixes the handling and specification of PerlEnv_putenv(), which was marked "si" when it should have been marked "i". This required changing its implementation from a Perl_ prefix to a S_ prefix and regenerating. I have run embed.pl in a loop with a local patch to shuffle the flags to see if there were any other order dependencies. No output files changed so I assume with this patch we are free of such bugs.
[Perl/perl5] 046c8c: embed.pl - the 's', 'S', 'i' and 'I' flags are mut...
Branch: refs/heads/yves/fix_embed_flag_static_type_mutex_detection Home: https://github.com/Perl/perl5 Commit: 046c8c399dbf739cd47a8b88313f818670e599d4 https://github.com/Perl/perl5/commit/046c8c399dbf739cd47a8b88313f818670e599d4 Author: Yves Orton Date: 2023-01-31 (Tue, 31 Jan 2023) Changed paths: M embed.fnc M embed.h M inline.h M proto.h M regen/embed.pl Log Message: --- embed.pl - the 's', 'S', 'i' and 'I' flags are mutually exclusive We had a bug where we processed the first one in the flags definition. Sorting the flags or rearranging them changes the output, which shouldn't happen.
[Perl/perl5]
Branch: refs/heads/yves/fix_ff_cleanup_race Home: https://github.com/Perl/perl5
[Perl/perl5] d404f7: File-Find/t - rework tempdir creation and cleanup
Branch: refs/heads/blead Home: https://github.com/Perl/perl5 Commit: d404f73baa29d5697ed2cac768de015217d5e819 https://github.com/Perl/perl5/commit/d404f73baa29d5697ed2cac768de015217d5e819 Author: Yves Orton Date: 2023-01-31 (Tue, 31 Jan 2023) Changed paths: M ext/File-Find/t/find.t M ext/File-Find/t/taint.t Log Message: --- File-Find/t - rework tempdir creation and cleanup Fixes #20734 - in a previous patch I missed the early cleanup call and the fact that it could result in a race condition. This hopefully resolves the problem. These tests files are pretty crufty. It would be nice to see them split apart so that the "sanity" checks which expect to be run in t/ are executed in a separate test files from the checks which build a tree to traverse for testing. A perfect task for a new contributor.
[Perl/perl5] e9f20e: UNIVERSAL.pm - Use lexical variables in Synopsis
Branch: refs/heads/blead Home: https://github.com/Perl/perl5 Commit: e9f20e0860ca7ba8a9430281f0c0e3dc4ba912cf https://github.com/Perl/perl5/commit/e9f20e0860ca7ba8a9430281f0c0e3dc4ba912cf Author: Elvin Aslanov Date: 2023-01-31 (Tue, 31 Jan 2023) Changed paths: M lib/UNIVERSAL.pm Log Message: --- UNIVERSAL.pm - Use lexical variables in Synopsis Add `my`, and name variables differently to avoid confusion.
[Perl/perl5] 42a4d6: Update autodie to CPAN version 2.36
Branch: refs/heads/blead Home: https://github.com/Perl/perl5 Commit: 42a4d650822086a91f0da204e86e1bc35c2ed66e https://github.com/Perl/perl5/commit/42a4d650822086a91f0da204e86e1bc35c2ed66e Author: Todd Rinaldo Date: 2023-01-30 (Mon, 30 Jan 2023) Changed paths: M Porting/Maintainers.pl M cpan/autodie/lib/Fatal.pm M cpan/autodie/lib/autodie.pm M cpan/autodie/lib/autodie/Scope/Guard.pm M cpan/autodie/lib/autodie/Scope/GuardStack.pm M cpan/autodie/lib/autodie/Util.pm M cpan/autodie/lib/autodie/exception.pm M cpan/autodie/lib/autodie/exception/system.pm M cpan/autodie/lib/autodie/hints.pm M cpan/autodie/lib/autodie/skip.pm M cpan/autodie/t/exception_class.t M cpan/autodie/t/lib/pujHa/ghach.pm M cpan/autodie/t/lib/pujHa/ghach/Dotlh.pm Log Message: --- Update autodie to CPAN version 2.36 [DELTA] 2.36 2023-01-30 16:48:23+00:00 UTC * Remove the use of ' as a package separator. * Fix spelling errors in POD.
[Perl/perl5]
Branch: refs/heads/yves/va1der_configure_fix Home: https://github.com/Perl/perl5
[Perl/perl5] 85d2c8: Configure - fix handling of quoted gcc output
Branch: refs/heads/blead Home: https://github.com/Perl/perl5 Commit: 85d2c8e80f418c456c28213a08f692e33833b8c4 https://github.com/Perl/perl5/commit/85d2c8e80f418c456c28213a08f692e33833b8c4 Author: Kurt Fitzner Date: 2023-01-31 (Tue, 31 Jan 2023) Changed paths: M AUTHORS M Configure Log Message: --- Configure - fix handling of quoted gcc output This patch was submitted in GH issue #20606. When gcc output contains quoted elements we fail to handle it properly. This tweaks the sed command to do so. Fixes #20606.