Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adapt to changes in Module-Starter-1.76 #3

Merged
merged 1 commit into from
Mar 30, 2019

Conversation

ppisar
Copy link
Contributor

@ppisar ppisar commented Mar 19, 2019

Module::Starter::Simple changed _license_record() API. Now it returns
a Software::License object or undef. This caused
Module-Starter-Plugin-CGIApp test failures:

Use of uninitialized value in string eq at /builddir/build/BUILD/Module-Starter-Plugin-CGIApp-0.44/blib/lib/Module/Starter/Plugin/CGIApp.pm line 468.
Use of uninitialized value $license_blurb in scalar chomp at /builddir/build/BUILD/Module-Starter-Plugin-CGIApp-0.44/blib/lib/Module/Starter/Plugin/CGIApp.pm line 488.
[...]
#   Failed test 'different files'
#   at t/common.pm line 114.
#          got: '4'
#     expected: '0'

This patch adapts to the chanegs and increaes a dependency on Module::Starter
to 1.76 version.

CPAN RT#128881

Module::Starter::Simple changed _license_record() API. Now it returns
a Software::License object or undef. This caused
Module-Starter-Plugin-CGIApp test failures:

    Use of uninitialized value in string eq at /builddir/build/BUILD/Module-Starter-Plugin-CGIApp-0.44/blib/lib/Module/Starter/Plugin/CGIApp.pm line 468.
    Use of uninitialized value $license_blurb in scalar chomp at /builddir/build/BUILD/Module-Starter-Plugin-CGIApp-0.44/blib/lib/Module/Starter/Plugin/CGIApp.pm line 488.
    [...]
    #   Failed test 'different files'
    #   at t/common.pm line 114.
    #          got: '4'
    #     expected: '0'

This patch adapts to the chanegs and increaes a dependency on Module::Starter
to 1.76 version.

CPAN RT#128881
@jaldhar jaldhar merged commit 73ec81f into jaldhar:master Mar 30, 2019
@jaldhar
Copy link
Owner

jaldhar commented Mar 30, 2019

Thanks. Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants