[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: dh-make-perl revamped (not really)



Hi Paul,

-=| Paul Fenwick, Fri, Dec 12, 2008 at 05:07:39PM +1100 |=-
> Damyan Ivanov wrote:
> 
> > * it is now a module (DhMakePerl) plus a script (dh-make-perl), which 
> >   contains POD and uses the module for the real work
> > * several additional modules were spun-off
> > * the build system uses Module::Build (perl Build.PL)
> > * there is even a test suite (./Build test)
> 
> You are awesome!  Totally and utterly awesome.  Thank-you.  ;)

Thank you for the kind words :)  I am not sure I deserve them fully 
yet, as even if the beggining has been laid, DhMakePerl is still in 
great need of slicing.

> However if I can be so bold as to make a small suggestion, would you
> consider calling the module App::DhMakePerl or Debian::DhMakePerl?
> App::DhMakePerl in particular would follow existing naming conventions for
> modules which primarily contain application code.

That was the initial idea. I dropped it for if for a trivial/lazy/pragmatic 
reason -- saving seven keystrokes ( shift a p p shift : : )

I forsee there will me several classes under DhMakePerl namespace, so 
the savings seem non-trivial.

I thought about shortening 'DhMakePerl' too, but abandoned the idea 
because "dh make perl" is engraved very deep im my mind :)

As Brian said, there is no real chance of clashes in the namespace. As 
for "polluting" it, I think this is not very relevant in this case. 
Thing is, I don't see any real value in including dh-make-perl in 
CPAN. The program is only useful on a Debian system, and since it is 
primarily developed by Debian developers, distributing it as a .deb 
makes more sense to both its developers and its users.

This is how *I* see it at least.

Ah, I also saw that Padre has no App:: so I thought it can't be that 
bad ommission.

> > though, so your testing (and fixing) is much appreciated. Please use 
> > the version from trunk/ for your new ITPs.
> 
> For the less educated amongst us, what's an ITP?  ;)

Sorry for the slang :) Thanks, gregor, for explaining.

> Unfortunately I'm still on my crazy travel schedule, which seems to have me
> in a different city each week until February; so while I've checked out the
> new code, and I'm itching to get my hands dirty with the new architecture, I
>  fear I may continue to lurk quietly over the festive season.

Even occasional advices are wery welcome. Safe travelling!

-- 
dam            JabberID: dam@jabber.minus273.org

Attachment: signature.asc
Description: Digital signature


Reply to: