Skip to content

Fix File::ShareDir::Install test failures#758

Merged
fglock merged 4 commits into
masterfrom
fix-file-sharedir-install
May 19, 2026
Merged

Fix File::ShareDir::Install test failures#758
fglock merged 4 commits into
masterfrom
fix-file-sharedir-install

Conversation

@fglock
Copy link
Copy Markdown
Owner

@fglock fglock commented May 19, 2026

Fix File::ShareDir::Install test failures by:

  • Use $(INST_LIB) and $(INST_ARCHLIB) variables instead of hardcoded blib/lib and blib/arch paths
  • Make config:: target depend on pm_to_blib so postamble rules are executed
  • Add command to copy share files from blib to install directory during make install
  • Fix _build_share_file_mapping to filter dot directories based on dotdirs setting using File::Find preprocess callback

This resolves test failures in t/10_makefile.t and t/11_dotfile.t.

fglock added 4 commits May 19, 2026 10:09
- Use $(INST_LIB) and $(INST_ARCHLIB) variables instead of hardcoded blib/lib and blib/arch paths
- Make config:: target depend on pm_to_blib so postamble rules are executed
- Add command to copy share files from blib to install directory during make install
- Fix _build_share_file_mapping to filter dot directories based on dotdirs setting using File::Find preprocess callback
- Add stub methods (oneliner, split_command, quote_literal) to make File::ShareDir::Install's postamble work correctly
- Fix PREFIX handling to set INSTALLSITELIB using $(SITEPREFIX) as a variable in the Makefile
- Add SITEPREFIX variable to the Makefile when provided
- Fix install command to properly copy share files from $(INST_LIB)/auto/share to $(INSTALLSITELIB)/auto/share
- All File::ShareDir::Install tests now pass (39/39 subtests)
- Change to perl5/t/ directory for tests in perl5/t/ so they can find ./test.pl via require
- Fixes false positive regressions in mro/next_NEXT.t and mro/next_NEXT_utf8.t
- Both tests now pass (4/4 each)
- Change to perl5_t/t/ directory for tests in perl5_t/t/ so they can find ./test.pl via require
- Fixes false positive regressions in mro/next_NEXT.t and mro/next_NEXT_utf8.t
- Both tests now pass (4/4 each)
@fglock fglock merged commit 1681041 into master May 19, 2026
2 checks passed
@fglock fglock deleted the fix-file-sharedir-install branch May 19, 2026 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant