Skip Menu |

This queue is for tickets about the MooX-Types-MooseLike CPAN distribution.

Report information
The Basics
Id: 103400
Status: rejected
Priority: 0/
Queue: MooX-Types-MooseLike

People
Owner: Nobody in particular
Requestors: mendoza [...] pvv.ntnu.no
Cc:
AdminCc:

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



Subject: Missing runtime dependency for Moo
When creating a debian package out of this module, we include the runtime deps, but that means someone else using this module will not have the required Moo modules that are required runtime if installing this.
On 2015-04-08 16:28:18, NICOMEN wrote: Show quoted text
> When creating a debian package out of this module, we include the > runtime deps, but that means someone else using this module will not > have the required Moo modules that are required runtime if installing > this.
Can you be more specific please? What dependencies are you adding, and what error did you get before you added them?
to. 09. april 2015 01.45.44 skrev ETHER: Show quoted text
> On 2015-04-08 16:28:18, NICOMEN wrote:
> > When creating a debian package out of this module, we include the > > runtime deps, but that means someone else using this module will not > > have the required Moo modules that are required runtime if installing > > this.
> > > Can you be more specific please? What dependencies are you adding, > and what error did you get before you added them?
This module needs Moo. There is no runtime dependency for Moo in the Makefile.PL, only a test dependency. In our build systems (which is not really relevant), I have to add manually a runtime dependency for Moo, so that when someone installs the packaged Moox::Types::MooseLike package they also install the dependency of Moo, since it depends on it to run ;-)
On Thu Apr 09 05:28:35 2015, NICOMEN wrote: Show quoted text
> to. 09. april 2015 01.45.44 skrev ETHER:
> > On 2015-04-08 16:28:18, NICOMEN wrote:
> > > When creating a debian package out of this module, we include the > > > runtime deps, but that means someone else using this module will > > > not > > > have the required Moo modules that are required runtime if > > > installing > > > this.
> > > > > > Can you be more specific please? What dependencies are you adding, > > and what error did you get before you added them?
> > This module needs Moo. There is no runtime dependency for Moo in the > Makefile.PL, only a test dependency. > > In our build systems (which is not really relevant), I have to add > manually a runtime dependency for Moo, so that when someone installs > the packaged Moox::Types::MooseLike package they also install the > dependency of Moo, since it depends on it to run ;-)
The module does not depend on Moo to run. I realize this can be surprising but it's true. So far I resisted adding Moo as a runtime dependency since it's not, but maybe I should just to avoid this confusion?
On Thu Apr 09 09:02:23 2015, MATEU wrote: Show quoted text
> On Thu Apr 09 05:28:35 2015, NICOMEN wrote:
> > to. 09. april 2015 01.45.44 skrev ETHER:
> > > On 2015-04-08 16:28:18, NICOMEN wrote:
> > > > When creating a debian package out of this module, we include the > > > > runtime deps, but that means someone else using this module will > > > > not > > > > have the required Moo modules that are required runtime if > > > > installing > > > > this.
> > > > > > > > > Can you be more specific please? What dependencies are you adding, > > > and what error did you get before you added them?
> > > > This module needs Moo. There is no runtime dependency for Moo in the > > Makefile.PL, only a test dependency. > > > > In our build systems (which is not really relevant), I have to add > > manually a runtime dependency for Moo, so that when someone installs > > the packaged Moox::Types::MooseLike package they also install the > > dependency of Moo, since it depends on it to run ;-)
> > The module does not depend on Moo to run. I realize this can be > surprising but it's true. So far I resisted adding Moo as a runtime > dependency since it's not, but maybe I should just to avoid this > confusion?