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.

Sign up for GitHub

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

Jump to bottom

Duplicate entries in Makefile.pl for VERSION_FROM created by 'dancer2 gen -a my::app' #1675

Closed
barkYboY opened this issue Feb 18, 2023 · 1 comment
Closed

Duplicate entries in Makefile.pl for VERSION_FROM created by 'dancer2 gen -a my::app' #1675

barkYboY opened this issue Feb 18, 2023 · 1 comment
Labels
Bug

Comments

@barkYboY
Copy link

'dancer2 gen -a my::app' puts 2 entries (lib/my/app.pm lib/my/app.pm) in VERSION_FROM field in Makefile.PL
this causes 'perl Makefile.pl' to fail with
Could not open 'lib/my/app.pm lib/my/app.pm': No such file or directory at /usr/share/perl/5.34/ExtUtils/MM_Unix.pm line 2967.

removing one of the entries manually cures the error, but doesn't fix it for next project generated.
@cromedome cromedome added the Bug label Feb 20, 2023
@cromedome cromedome mentioned this issue Feb 20, 2023
Closed
@cromedome
Copy link
Contributor

I've created a PR to address this. Thanks for the bug report!

cromedome added a commit that referenced this issue Oct 9, 2023
@cromedome
v1.0.0
ab2a50a 
    [ BUG FIXES ]
    * GH #1663: Allow overriding of prefix in add_route (GeekRuthie)
    * GH #1675: Stringify VERSION_FROM correctly in Makefile.PL (Jason
      A. Crome)
    * GH #1677: Don't deserialize multipart form data on post (Emil
      Perhinschi)
    * GH #1694: Update JS assets in Dancer2 app skel (Jason A. Crome)

    [ ENHANCEMENTS ]
    * PR #1682: Bump minimum version of Perl to 5.14 (Jason A. Crome)

    [ DOCUMENTATION ]
    * GH #1580: Document the purpose of the .dancer file (Jason A. Crome)
    * GH #1669: Show correct usage of Dancer2::Core::Error (GeekRuthie)
    * GH #1674: Fix POD for input_handle() (mauke)
    * GH #1414: Add documentation resources to the doc map (Jason A.
      Crome, Yanick Champoux)
    * PR #1684: Remove shumphrey from core developers (Steven Humphrey)
    * GH #1685: Document the versioning scheme and Dancer2 release
      process (Jason A. Crome)
    * PR #1688: Fixed various bugs/issues in Dancer2 Pod (Jason A. Crome)
    * PR #1691: Update the contribution guidelines (Jason A. Crome)
    * PR #1692: Change README extension .mkdn -> .md (Jason A. Crome)

    [ DEPRECATED ]
    * GH #1645: Deprecated Dancer2::Test (Jason A. Crome)
    * GH #1646: Deprecated keyword: push_header (Jason A. Crome)
    * GH #1647: Deprecated keyword: header (Jason A. Crome)
    * GH #1648: Deprecated keyword: headers (Jason A. Crome)
    * GH #1649: Deprecated keyword: context (Jason A. Crome)
    * GH #1650: Deprecated: splat/capture named placeholders (Jason A.
      Crome)
    * GH #1651: Deprecated core Request instance method:
      request->dispatch_path (Jason A. Crome)
    * GH #1652: Deprecated keyword in plugins: plugin_setting (Jason A.
      Crome)
    * GH #1653: Deprecated keyword in plugins: dancer_app (Jason A. Crome)
    * GH #1654: Deprecated keyword in plugins: request (Jason A. Crome)
    * GH #1655: Deprecated keyword in plugins: var (Jason A. Crome)
    * GH #1656: Deprecated keyword in plugins: hook (Jason A. Crome)

    [ MISC ]
    * GH #1659: Rename `master` branch to be `main` (Yanick Champoux)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Stringify VERSION_FROM correctly in Makefile.PL PerlDancer/Dancer2
2 participants
@cromedome @barkYboY