Skip to content

Conversation

@fglock
Copy link
Owner

@fglock fglock commented Oct 29, 2025

Summary

This PR fixes the failing t/lib/commonsense.t test by adding the File::Glob module and updating the Config module to report the correct available extensions.

Changes

  • Added File::Glob.pm: Minimal but functional implementation with bsd_glob() and glob() functions, plus necessary constants
  • Updated Config.pm: Modified extensions and dynamic_ext to include Fcntl, IO, and File/Glob

Testing

  • Test now passes: ./jperl t/lib/commonsense.tok 1 - common sense
  • File::Glob module verified to work correctly with basic glob operations
  • No regressions in other tests

Details

The test checks for three core Perl modules in $Config{extensions}:

  1. Fcntl (already present)
  2. IO (already present)
  3. File/Glob (newly added)

The File::Glob module provides a lightweight implementation using Perl's built-in glob operator.

- Added File::Glob.pm with bsd_glob() and glob() functions
- Updated Config.pm to include Fcntl, IO, and File/Glob in extensions
- Test now passes by properly reporting available Perl modules
- File::Glob uses Perl's built-in glob for functionality
@fglock fglock merged commit cb5db6b into master Oct 29, 2025
2 checks passed
@fglock fglock deleted the fix-commonsense-test branch October 29, 2025 14:01
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.

2 participants