git.t test failing in master

git.t test failing in master

[Konstantin Ryabitsev]

[-- Attachment #1: Type: text/plain, Size: 1011 bytes --]

Hello:

I'm working to get lore.kernel.org to the latest master, but it would
appear "make test" is not completing correctly:

t/git.t .................... 1/? Modification of a read-only value attempted at /.../blib/lib/PublicInbox/Git.pm line 31.
# Tests were run but no plan was declared and done_testing() was not seen.
t/git.t .................... Dubious, test returned 255 (wstat 65280, 0xff00)

And, at the end:

Test Summary Report
-------------------
t/git.t                  (Wstat: 65280 Tests: 34 Failed: 0)
  Non-zero exit status: 255
  Parse errors: No plan found in TAP output
Files=67, Tests=2062, 37 wallclock secs ( 0.26 usr  0.07 sys + 16.95 cusr  9.63 csys = 26.91 CPU)
Result: FAIL
Failed 1/67 test programs. 0/2062 subtests failed.
make: *** [test_dynamic] Error 255

This is probably fairly benign from the looks of it, but I'd like your
opinion on whether this is something to worry about before deploying.

Perl: v5.16.3
Git: 2.16.5
Tree: d66aa534a4a7506cfc5cfab49d1e09f8db8be3dd

-K


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[PATCH] t/git.t: avoid passing read-only value to git_unquote

[Eric Wong]

Konstantin Ryabitsev <konstantin@linuxfoundation.org> wrote:
> This is probably fairly benign from the looks of it, but I'd like your
> opinion on whether this is something to worry about before deploying.

Yeah, the actual code only hits modifiable value

> Perl: v5.16.3

This is Fedora?  Which version?  I was working on getting a
chroot setup for testing some RPM-based distros, but some of the
mirrors/tools for doing that were out-of-date.  Might try
installing in QEMU...

> Git: 2.16.5
> Tree: d66aa534a4a7506cfc5cfab49d1e09f8db8be3dd

Thanks for the report, following should fix it.

----------8<--------
Subject: [PATCH] t/git.t: avoid passing read-only value to git_unquote

Older versions of Perl (tested 5.14.2 on Debian wheezy(*),
reported by Konstantin on Perl 5.16.3) considered the result of
concatenating two string literals to be a constant value.

(*) not that other stuff works on wheezy, but t/git.t should.

Reported-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
---
 t/git.t | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/t/git.t b/t/git.t
index 50ec4fb..052e1ce 100644
--- a/t/git.t
+++ b/t/git.t
@@ -145,7 +145,8 @@ if ('alternates reloaded') {
 }
 
 use_ok 'PublicInbox::Git', qw(git_unquote);
-is("foo\nbar", git_unquote('"foo\\nbar"'.''), 'unquoted newline');
-is("Eléanor", git_unquote('"El\\303\\251anor"'.''), 'unquoted octal');
+my $s;
+is("foo\nbar", git_unquote($s = '"foo\\nbar"'), 'unquoted newline');
+is("Eléanor", git_unquote($s = '"El\\303\\251anor"'), 'unquoted octal');
 
 done_testing();
-- 
EW

Re: [PATCH] t/git.t: avoid passing read-only value to git_unquote

[Konstantin Ryabitsev]

On Fri, Jan 18, 2019 at 07:47:06PM +0000, Eric Wong wrote:
> > Perl: v5.16.3
> 
> This is Fedora?  Which version?  I was working on getting a
> chroot setup for testing some RPM-based distros, but some of the
> mirrors/tools for doing that were out-of-date.  Might try
> installing in QEMU...

No, this is RHEL/CentOS 7.

> > Git: 2.16.5
> > Tree: d66aa534a4a7506cfc5cfab49d1e09f8db8be3dd
> 
> Thanks for the report, following should fix it.

Yep, all tests pass now. Thanks!

-K