Replace ReportVersions::Tiny with Test::ReportPrereqs #2
Conversation
`ReportVersions::Tiny` is deprecated; `Test::ReportPrereqs` is the recommended alternative. This change replaces the deprecated plugin with the recommended plugin in `dist.ini`. Although this replacement had been made in version 0.0.5, the change in `dist.ini` had been missed which was causing users of the bundle to still use `ReportVersions::Tiny`. It could, however, be possible that the issue is actually caused by the 0.0.4 version being considered by CPAN to be the version to use rathern than the 0.0.5 version. Nevertheless, this change, coupled with a new release, would help many downstream users as well as close RT issue #103820.
|
👍 says the But note that the version requirement of '0' is not helpful to make reproducible builds. |
Agreed. I used a version requirement of '0' to match the other version requirements in the plugin bundle, so thought that would be the best way to go. Any particular version which would be appropriate in this case (other than the most recent one)? |
|
The most recent one (0.025) is appropriate because the plugin generates code that is bundled in the distribution using this plugin via this bundle. It is important that this generated code has all the known fixes. |
|
Applied. Pushed. Thanks. |
|
For your information I just pushed a serie of 10 commits to completely modernize the bundle implementation: most manual prereqs are removed, and replaced by automatic injection by BundleDeps. |
ReportVersions::Tinyis deprecated;Test::ReportPrereqsis therecommended alternative. This change replaces the deprecated plugin with
the recommended plugin in
dist.ini. Although this replacement had beenmade in version 0.0.5, the change in
dist.inihad been missed which wascausing users of the bundle to still use
ReportVersions::Tiny. It could,however, be possible that the issue is actually caused by the 0.0.4 version
being considered by CPAN to be the version to use rathern than the 0.0.5
version. Nevertheless, this change, coupled with a new release, would help
many downstream users as well as close RT issue #103820.
This pull request is submitted in the hope that it is useful. If you have any questions or comments, please don't hesitate to let me know. If any changes are required, I can update the pull request and resubmit.