Skip Menu |

This queue is for tickets about the App-EditorTools CPAN distribution.

Report information
The Basics
Id: 63018
Status: resolved
Priority: 0/
Queue: App-EditorTools

People
Owner: Nobody in particular
Requestors: KENTNL [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Expected breakage in upcomming changes to App::Cmd
App::EditorTools::Command::Parent::Install should either directly, or indirectly, inherit from App::Cmd::Command.

The recommended way is using App::Cmd::Setup

   use App::Cmd::Setup -command;

but { use parent 'App::Cmd::Command;'  }

should be sufficient.

--

When my branch gets merged upstream, and pushed,

perl -Ilib script/editortools
App::EditorTools::Command::Parent::Install is not a App::Cmd::Command at /home/perltest/testd/app-cmd/lib/perl5/App/Cmd.pm line 86.

https://github.com/kentfredric/app-cmd/blob/kentnl-releases/lib/App/Cmd.pm#L86

If you want some code to test your code with prior to the release,

https://github.com/kentfredric/app-cmd/tarball/4fc56f885bf7a42a82f529797df7113ca25c0880

May be helpful.

I think I'll  mention solving this error is more complicated than It looked at first glance, and I'm attempting to hack a fix, so I'll let you know how that goes =)

https://github.com/mvgrimes/app-editortools/pull/1

Patched for you, changes reasonably simple, just not very straight forward,

May eventually get a fix in App::Cmd to make it nicer ,
Thanks for the report and patch. It took too long, but I have finally patched git and released the new version to cpan. On Tue Nov 16 01:28:04 2010, KENTNL wrote: Show quoted text
> https://github.com/mvgrimes/app-editortools/pull/1 > > Patched for you, changes reasonably simple, just not very straight forward, > > May eventually get a fix in App::Cmd to make it nicer ,