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

Re: RFS: arp-scan (updated package, new maintainer)



William,

On Mon, 2011-06-27 at 09:01 -0500, William Vera wrote:
> On Mon, Jun 27, 2011 at 3:29 AM, Kilian Krause <kilian@debian.org> wrote:
> > William,
> >
> > On Mon, 2011-06-27 at 03:05 -0500, William Vera wrote:
> >
> > I guess you should try to do what it reads. Ask
> > arp-scan@nta-monitor.com and see what they tell about why this is
> > happening - and how to fix it properly.
> 
> Exactly, I already sent a email

Good. ;-)


> >
> > I seem to have put it confusingly. You dropped a patch that was in
> > 1.6-1. Where has that patch gone? Why is it dropped and no longer
> > needed?
> 
> Was not a patch on intentionally, was a self-generated file, the
> package did not use quilt until I implemented it, the file
> autogenerated showed differences between "diff" because of a dirty
> environment.

Ah, I see.


> > Which part of:
> >          Keep the original name of the upstream changelog. This will be
> >          accomplished by installing the upstream changelog as
> > changelog, and
> >          making a symlink from that to the original name of the
> > changelog
> >          file. This can be useful if the upstream changelog has an
> > unusual
> >          name, or if other documentation in the package refers to the
> >          changelog file.
> >
> > are you referring to? ChangeLog is not a unusual name. Thus IMHO the -k
> > can still be dropped.
> >
> 
> Droped
> 
> >
> >> > 6. dpkg-gencontrol: warning: package arp-scan: unused substitution
> >> > variable ${perl:Depends}
> >>
> >> I'm a little confused, how I can clean this warning?
> >
> > Feed the perl file that resides out of place to dh_perl on the command
> > line or drop the ${perl:Depends} if it's not needed.
> >
> 
> I don't understood, I don't have ${perl:Depends} in my control file,
> or which file are you talking about?

Right. Weird. My pbuilder does throw that though. I think it can be
safely ignored then.


> >
> >
> > Thanks!
> >
> 
> Thanks to you for your review

Np ;-)

-- 
Best regards,
Kilian

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: