PadWalker peek_our misses file-scope our variables on JVM
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
Start with the existing PadWalker regression coverage for peek_our(0) inside a subroutine and trace how the JVM backend handles that lookup; the issue does not name the relevant files. Add project-owned top-level coverage for both JVM and interpreter backends, checking the expected variable names and references against system Perl. Run the regression tests and the Test::Expander upstream suite; done means the exports load and the suite proceeds.
由索引模型根据 Issue 内容生成。
描述
Summary
After adding the bundled Scalar::Readonly support for #1664, jcpan -t Test::Expander 2.7.1 gets past the missing XS implementation but all 26 test programs fail while loading Test::Expander::Constants. The module populates @EXPORT_OK from PadWalker::peek_our(0) at file scope; on the JVM that lookup does not return its package variables, so imports such as $COMMENT, %OPTION_PARSER, and %REST_CONSTANTS_TO_EXPORT are reported as not exported.
Reproduction
use PadWalker qw(peek_our);
our $probe = 17;
my $vars = peek_our(0);
print join(',', sort keys %$vars), "\n";
System Perl reports $probe; PerlOnJava's JVM reports an empty key list. The existing PadWalker regression covers peek_our(0) inside a subroutine, but not at file scope.
The affected upstream run used nice -n 19 timeout 1800 ./jcpan -t Test::Expander. It failed 26/26 programs before their tests could run, with “is not exported by Test::Expander::Constants” errors. This is a follow-up to #1664: the missing Scalar::Readonly XS support is now available, and this issue tracks the separate PadWalker::peek_our behavior needed by this consumer.
Expected behavior
PadWalker::peek_our(0) should expose package our variables at file scope with Perl-compatible names and references, so Test::Expander::Constants can build its export list and the upstream suite can proceed.
Regression coverage
Add project-owned coverage for top-level peek_our(0) on both JVM and interpreter backends, validate the oracle with system Perl, then rerun the Test::Expander upstream suite.
- 主要语言
- Perl
- 星标
- 64
- 派生
- 7
- 平均合并
- 5 小时 25 分钟
- 30 天内合并 PR
- 178
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
fglock/PerlOnJava 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 68/100
fglock/PerlOnJava#1651 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 72/100
fglock/PerlOnJava#1579 ·
维护者通常 1 天内回复
-
area:cpan-port area:unicode bug
难度 2/5 1-3 小时 新手友好度 88/100
fglock/PerlOnJava#1341 ·
维护者通常 1 天内回复
-
area:runtime bug
难度 4/5 3-5 天 新手友好度 35/100
fglock/PerlOnJava#1700 ·
维护者通常 1 天内回复
-
难度 5/5 一周以上 新手友好度 12/100
fglock/PerlOnJava#1696 ·
维护者通常 1 天内回复
查看 fglock/PerlOnJava 的全部 Issue
相似的 Issue
-
难度 1/5 1 小时以内 新手友好度 85/100
维护者通常 1 天内回复
-
bug help wanted
难度 2/5 1 小时以内 新手友好度 70/100
bioepic-data/bervo#145 · 1 条评论 ·
维护者通常 1 天内回复
-
Type: Bug
难度 2/5 1-3 小时 新手友好度 72/100
inverse-inc/packetfence#9413 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
-
Help-Wanted Needs-Triage Package-Update
难度 2/5 1-3 小时 新手友好度 72/100
microsoft/winget-pkgs#448814 · 1 条评论 ·
维护者通常 1 天内回复