Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ExtUtils::MakeMaker 7.28+ MM_Win32.t fails t/porting/exec-bit #16009

Closed
p5pRT opened this issue Jun 9, 2017 · 6 comments
Closed

ExtUtils::MakeMaker 7.28+ MM_Win32.t fails t/porting/exec-bit #16009

p5pRT opened this issue Jun 9, 2017 · 6 comments

Comments

@p5pRT
Copy link

p5pRT commented Jun 9, 2017

Migrated from rt.perl.org#131545 (status was 'resolved')

Searchable as RT131545$

@p5pRT
Copy link
Author

p5pRT commented Jun 9, 2017

From al.dunsmuir@sympatico.ca

Created by al.dunsmuir@sympatico.ca

This is a bug report for perl from al.dunsmuir@​sympatico.ca,
generated with the help of perlbug 1.40 running under perl 5.24.1.

-----------------------------------------------------------------
After fixing build for 5.26.0 on AIX 5.3 under ticket #131415
  Perl 5.24+ Configure loses hints script changes
  https://rt-archive.perl.org/perl5/Ticket/Display.html?id=131415

the ExtUtils​::MakeMaker selftest for the release included in 5.26.0
failed with​:

Failed 1 test out of 2446, 99.96% okay. ../cpan/ExtUtils-MakeMaker/t/02-xsdynamic.t
With multiple failures of the form​:
# make[1]​: *** No rule to make target 'Test.exp', needed by 'dynamic'. Stop.
# make[1]​: *** No rule to make target 'Test.exp', needed by 'static'. Stop.

This appeared to be previously reported under ticket #128902
  MakeMaker 02-xsdynamic.t failure in AIX 5.3 with blead > be93048
  https://rt-archive.perl.org/perl5/Ticket/Display.html?id=128902

I updated the ExtUtils​::MakeMaker package in my 5.26.0 build to
release 7.28, and while it corrected that test, a later test failed​:

t/porting/exec-bit .............................................
# Failed test 82 - tarball will chmod +x ../cpan/ExtUtils-MakeMaker/t/MM_Win32.t
at porting/exec-bit.t line 60 # Remove the exec bit or add
'../cpan/ExtUtils-MakeMaker/t/MM_Win32.t' to Porting/exec-bit.txt
FAILED at test 82

I added a chmod command to reset the exec bit on MM_Win32.t and the
test passed.

I have verified that the exec bit is also set in the gz file for the
current 7.29_01 test release, so am reporting it so can be fixed
in the next iteration.

Perl Info

Flags:
    category=core
    severity=low


@p5pRT
Copy link
Author

p5pRT commented Jun 10, 2017

From @iabyn

On Fri, Jun 09, 2017 at 08​:40​:51AM -0700, Al Dunsmuir wrote​:

After fixing build for 5.26.0 on AIX 5.3 under ticket #131415
Perl 5.24+ Configure loses hints script changes
https://rt-archive.perl.org/perl5/Ticket/Display.html?id=131415

the ExtUtils​::MakeMaker selftest for the release included in 5.26.0
failed with​:

Failed 1 test out of 2446, 99.96% okay. ../cpan/ExtUtils-MakeMaker/t/02-xsdynamic.t
With multiple failures of the form​:
# make[1]​: *** No rule to make target 'Test.exp', needed by 'dynamic'. Stop.
# make[1]​: *** No rule to make target 'Test.exp', needed by 'static'. Stop.

This appeared to be previously reported under ticket #128902
MakeMaker 02-xsdynamic.t failure in AIX 5.3 with blead > be93048
https://rt-archive.perl.org/perl5/Ticket/Display.html?id=128902

I updated the ExtUtils​::MakeMaker package in my 5.26.0 build to
release 7.28, and while it corrected that test, a later test failed​:

t/porting/exec-bit .............................................
# Failed test 82 - tarball will chmod +x ../cpan/ExtUtils-MakeMaker/t/MM_Win32.t
at porting/exec-bit.t line 60 # Remove the exec bit or add
'../cpan/ExtUtils-MakeMaker/t/MM_Win32.t' to Porting/exec-bit.txt
FAILED at test 82

I added a chmod command to reset the exec bit on MM_Win32.t and the
test passed.

I have verified that the exec bit is also set in the gz file for the
current 7.29_01 test release, so am reporting it so can be fixed
in the next iteration.

So if I've understood all the above correctly, you're saying​:

1) perl needs to bundle a newer release of ExtUtils​::MakeMaker to work
correctly on some platforms.

2) The current development release of EU​::MM
has an execute bit incorrectly set on one of its files.

So the only thing that needs to be done at the perl end is to update
the bundled EU​::MM when a suitable version is available, and make sure
the exec bit isn't set.

--
"Strange women lying in ponds distributing swords is no basis for a system
of government. Supreme executive power derives from a mandate from the
masses, not from some farcical aquatic ceremony."
  -- Dennis, "Monty Python and the Holy Grail"

@p5pRT
Copy link
Author

p5pRT commented Jun 10, 2017

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Jun 11, 2017

From al.dunsmuir@sympatico.ca

On Sat, 10 Jun 2017 02​:16​:23 -0700, davem wrote​:

I have verified that the exec bit is also set in the gz file for the
current 7.29_01 test release, so am reporting it so can be
fixed
in the next iteration.

So if I've understood all the above correctly, you're saying​:

1) perl needs to bundle a newer release of ExtUtils​::MakeMaker to work
correctly on some platforms.

2) The current development release of EU​::MM
has an execute bit incorrectly set on one of its files.

So the only thing that needs to be done at the perl end is to update
the bundled EU​::MM when a suitable version is available, and make sure
the exec bit isn't set.

Dave,

Yes.

I've updated the bundled version in my build to the latest stable 7.28,
but expect that by the time 5.26.1 comes around there will be something
newer than the 7.29 test release.

I figured that giving the author the "heads up" about the exec bit would
save whoever does the integration testing at least 1 build iteration. 8^)

--
Al Dunsmuir

@p5pRT
Copy link
Author

p5pRT commented Jun 12, 2017

From al.dunsmuir@sympatico.ca

On Sun, 11 Jun 2017 16​:13​:51 -0700, al.dunsmuir@​sympatico.ca wrote​:

I have verified that the exec bit is also set in the gz file for the
current 7.29_01 test release, so am reporting it so can be
fixed in the next iteration.

I have verified that this is corrected in release 7.30, dated June 12.
--
Al Dunsmuir

@p5pRT p5pRT closed this as completed Jun 14, 2017
@p5pRT
Copy link
Author

p5pRT commented Jun 14, 2017

@iabyn - Status changed from 'open' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant