Re: [MoM] regarding rename_binary_result.patch
Hi Malihe,
On Tue, Aug 18, 2015 at 02:07:46PM +0430, Malihe Asemani wrote:
> Hey Andreas,
>
> > On Tue, Aug 18, 2015 at 12:35 PM, Andreas Tille <andreas@an3as.eu> wrote:
> > On Mon, Aug 10, 2015 at 05:25:29PM +0200, Andreas Tille wrote:
> > > Hi Malihe,
> > >
> > > I noticed another issue. In the pristine-tar branch the old tarball name
> > > remains. I'd recommend to reimport the new tarball name
> > >
> > > gbp import-orig --pristine-tar
> > <PATH>/king-probe_2.13.110909.orig.tar.xz
> > >
> > > it might be needed to delete the upstream tag but might be it works
> > > without this.
> > I noticed that you succeeded in this - fine.
> > I have two remarks to your current packaging:
> > 1. You have cut-n-pasted NEWS.Debian from plink. While I suggested to
> > sneak into plink the file NEWS.Debian is a reserved file for *news* ==
> > differences to former releases of this package and apt will display this
> > on users machines at install time. Since there are not any previous
> > releases please rename this to README.Debian which is more apropiate and
> > free form. So you can rather use
> > (king?)probe for Debian
> > =======================
> > instead of
> > king-probe (2.13.110909-1) unstable; urgency=low
> >
> Yes, I didn't know which one of README and NEWS files should I use? And
> just chose one of them!
> I'll change it to a README file. Thanks for your comment :)
>
>
> > 2. You might possibly wonder why the package might fail to build.
>
>
> yes. exactly. I think the package should create
>
> usr/lib/debian-med/bin/
>
> directory using d/dirs and then copy the prob binary file in this path
> using d/install file.
> And then it should create a link from
>
> usr/lib/debian-med/bin/probe
>
> to
>
> usr/bin/king-probe
>
> using d/links. I dont know why this method does not work!
Since the upstream Makefile has this fixed target dir (== is broken).
That's why I recommended to ignore the upstrea Makefile.
> > You
> > probably want to override_dh_auto_install and move the build result
> > manually into the correct place (or alternatively fix the makefile if
> > you prefer this - I'd go with the first suggestion)
> >
>
> first suggestion seems more correct in my mind, too. But I still dont know
> why my current method does not work correctly.
$ grep -A1 "^install: probe" Makefile.linux
install: probe
mv probe /local/bin
This can not work ...
Kind regards
Andreas.
--
http://fam-tille.de
Reply to: