fix perlcc test #2

rurban · 2016-03-15T21:49:13Z

t/RT96893_perlcc.t failed with a working perlcc

t/binary.t ........... ok
t/exotic_names.t ..... ok
t/names.t ............ ok
t/RT42725_deparse.t .. ok
t/RT96893_perlcc.t ... 1/1 # No tests run!
t/RT96893_perlcc.t ... Dubious, test returned 255 (wstat 65280, 0xff00)
All 1 subtests passed
t/smoke.t ............ ok

Test Summary Report
-------------------
t/RT96893_perlcc.t (Wstat: 65280 Tests: 1 Failed: 0)
  Non-zero exit status: 255

… (problem under perlcc -O3)

Fix leak when setting a fully-qualified name

Provide back-compat macro for pre-5.10 perls

Closes RT#50524

Update %DB::sub

e.g. see failures: http://www.cpantesters.org/cpan/report/744c9670-1e48-11e4-bb3c-7408e0bfc7aa http://www.cpantesters.org/cpan/report/7ce69e5a-1f08-11e4-bc9c-0165e0bfc7aa http://www.cpantesters.org/cpan/report/dc6e14f2-1f26-11e4-ad93-d8e4e0bfc7aa http://www.cpantesters.org/cpan/report/0faa4ed0-1ebe-11e4-ae61-5739e0bfc7aa

…e the funky copyright

- move variable declaration to fix warning under -Werror=declaration-after-statement, to allow compilation under MSVC (J.R.Mash, github rafl#3)

- fix licence in LICENSE and pod

…utors don't have to install it

…oken can result in a skipped test coming out wrong: e.g. http://www.cpantesters.org/cpan/report/5846c0a8-d12c-11e4-9cf9-0ad4a10b8abb http://www.cpantesters.org/cpan/report/fb6132f2-be7e-11e4-883f-3fda4ebb7ab2

…a/Develop.pm

- fix optional test of interaction with B::C that sometimes invalidly failed

disable ' as package seperator add t/names.t from ribasushi and Aristotle Pagaltzis

This is scaffolding to sanity-check our assumptions about perl: that anything we *can* eval is named by the interpreter as expected. This test passes on 5.6.1+ on both 32 and 64 bit perls.

Reveals incorrect handling of - ' (\x27) on all perls - \0 on perl > 5.14 - Unicode on perl > 5.16

but not of the stash yet. (which should not be necessary) also remove unneeded backcompat macro which are already in ppport.h

all tests pass now

We call the sub by ref, not by name. This always worked. Just test if we can call it and store the binary name.

Only for <5.14 ' is replaced by ::

5.10-5.16: B did not absolute it when converting to a PV, so we cannot preserve utf8 GvNAMEs from 5.10 - 5.16. Tests still pass. ' => :: conversion is still in core for all stash names parsed. I added a hopeful 25 version enddate for this. Needs to be adjusted if nobody will deprecate this perl4'ism. The only remaining problems are now with <5.8.9

ppport.h gv_fetchpvn_flags for 5.8.8 does not terminate the name buf at len

do not use Test::Builder with an external result

Reini Urban and others added 30 commits July 10, 2014 07:24

Fix RT#96893 - do not change the string arg in XS; use a copy instead…

24f50a7

… (problem under perlcc -O3)

add MYMETA.yml/json to .gitignore

5effcd9

add README make target

1b5160b

prepare for 0.06 release

92bfd01

fix TAP output

0290d41

fix test when B::C not installed

3aaa8d1

ignore all the usual things (including MYMETA.*)

ccbc33a

declare minimum perl version and repository, bugtracker in metadata

6810442

skip t/RT96893_perlcc.t with too old B::C versions

cc891d2

prepare for 0.07 release

988a6b1

Fix leak when setting a fully-qualified name

85cf5fa

Merge pull request rafl#1 from ilmari/leakfix

1898eaa

Fix leak when setting a fully-qualified name

whitespace only

c258c98

prepare for 0.08 release

dc23763

declare runtime prereqs

22fe73b

Switch from Newz to Newxz

bcbff2c

Provide back-compat macro for pre-5.10 perls

Copy the contents of the %DB::sub entry if it exists

3967e62

Closes RT#50524

Merge pull request rafl#2 from ilmari/db-sub

716d610

Update %DB::sub

changelog for RT#50524

4b2bb17

remove unneeded shebang

cf5ba43

better to call require at runtime

5d71dfc

convert raw tests to using Test::More

2b63751

improve wording of this skip message

2e4f8ed

whitespace

a0f015f

use strict; warnings!

b9534ef

SEE ALSO Sub::Identify (RT#76822)

fa1e952

version 0.09

0ad1ba0

convert to Dist::Zilla, including careful tailoring of pod to preserv…

54595ea

…e the funky copyright

fix author and release tests - spelling, Changes formatting

30f18ad

karenetheridge and others added 30 commits August 19, 2014 21:47

Sub-Name-0.11-TRIAL

00d1784

- move variable declaration to fix warning under -Werror=declaration-after-statement, to allow compilation under MSVC (J.R.Mash, github rafl#3)

fix weaving of licence into pod

d2e2a4b

include cPanel in the LICENSE file

5e55cc3

changelog for the release

0db938d

shorter pod lists

691b40b

refer to Sub::Util

86f08d9

Sub-Name-0.12

db3290d

- fix licence in LICENSE and pod

run ExtUtils::MakeMaker::Dist::Zilla::Develop out of inc/, so contrib…

1cd6d14

…utors don't have to install it

skip generating man pages for dev builds

dd1de3c

Makefile.PL is already excluded by the bundle

19320b3

ignore new EUMM build artifact

6860467

keep $VERSION in repo version of module

dfd7ef2

fix test: the TAP actually comes from a subprocess, so the plan is br…

ac980bb

…oken can result in a skipped test coming out wrong: e.g. http://www.cpantesters.org/cpan/report/5846c0a8-d12c-11e4-9cf9-0ad4a10b8abb http://www.cpantesters.org/cpan/report/fb6132f2-be7e-11e4-883f-3fda4ebb7ab2

my plugin bundle now always excludes inc/ExtUtils/MakeMaker/Dist/Zill…

db4dd4d

…a/Develop.pm

Sub-Name-0.13

6eb2af6

- fix optional test of interaction with B::C that sometimes invalidly failed

increment $VERSION after release

9b196bc

skip this test if Devel::CheckBin is not installed (RT#103165)

b099984

add ppport.h to the distribution, and keep it updated

3b02e09

0.15: support binary names

df63f15

disable ' as package seperator add t/names.t from ribasushi and Aristotle Pagaltzis

add infrastructure of test for exotic sub names

b7c3be9

This is scaffolding to sanity-check our assumptions about perl: that anything we *can* eval is named by the interpreter as expected. This test passes on 5.6.1+ on both 32 and 64 bit perls.

test Sub::Name for exotic sub names

9171033

Reveals incorrect handling of - ' (\x27) on all perls - \0 on perl > 5.14 - Unicode on perl > 5.16

Merge remote-tracking branch 'ap/exotic_naming' into binary

b75de4d

Set UTF8 flag of GvNAME

994c791

but not of the stash yet. (which should not be necessary) also remove unneeded backcompat macro which are already in ppport.h

And fix also the utf8 stash part

5292084

all tests pass now

enable t/binary.t for all perl versions

13de769

We call the sub by ref, not by name. This always worked. Just test if we can call it and store the binary name.

fix t/names for old ' package seperator

aa76cbd

Only for <5.14 ' is replaced by ::

set stash UTF8 flag also 5.10-5.12

b0a9a91

Fix for 5.8.x: need to copy the temp. stash gv

d57df7a

ppport.h gv_fetchpvn_flags for 5.8.8 does not terminate the name buf at len

fix t/RT96893_perlcc.t test

84ce864

do not use Test::Builder with an external result

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix perlcc test #2

fix perlcc test #2

rurban commented Mar 15, 2016

fix perlcc test #2

Are you sure you want to change the base?

fix perlcc test #2

Conversation

rurban commented Mar 15, 2016