Bug#807493: altree: Produces empty binary package

2015-12-09 Thread Dominic Hargreaves
Source: altree
Version: 1.3.1-3
Severity: grave
Justification: package not usable
Tags: sid
X-Debbugs-Cc: debian-p...@lists.debian.org

The last upload, fixing #807423, unfortunately started creating empty
binary packages on most (but not all) architectures. See for example

https://packages.debian.org/sid/amd64/altree/filelist
https://packages.debian.org/sid/armel/altree/filelist
https://packages.debian.org/sid/armhf/altree/filelist
https://packages.debian.org/sid/i386/altree/filelist

I can't see from a quick glance what is going on here; CC-ing
debian-perl in case anyone can help.

Cheers,
Dominic.



Bug#807493: altree: Produces empty binary package

2015-12-09 Thread Dominic Hargreaves
On Wed, Dec 09, 2015 at 04:56:11PM +0100, Vincent Danjean wrote:
>   Hi,
> 
> Le 09/12/2015 16:31, Dominic Hargreaves a écrit :
> > Source: altree
> > Version: 1.3.1-3
> > Severity: grave
> > Justification: package not usable
> > Tags: sid
> > X-Debbugs-Cc: debian-p...@lists.debian.org
> > 
> > The last upload, fixing #807423, unfortunately started creating empty
> > binary packages on most (but not all) architectures. See for example
> > 
> > https://packages.debian.org/sid/amd64/altree/filelist
> > https://packages.debian.org/sid/armel/altree/filelist
> > https://packages.debian.org/sid/armhf/altree/filelist
> > https://packages.debian.org/sid/i386/altree/filelist
> > 
> > I can't see from a quick glance what is going on here; CC-ing
> > debian-perl in case anyone can help.
> 
>   It is an error from me. altree files are now installed in debian/tmp
> instead of directly debian/altree by debian/rules, so a debian/altree.install
> file is required.
>   I will fix this in a few hours if nobody do it before. And I will
> reincorporate Andreas modifications that were all good.

Great, thanks for the swift response!

Cheers,
Dominic.



Bug#807493: altree: Produces empty binary package

2015-12-09 Thread Vincent Danjean
  Hi,

Le 09/12/2015 16:31, Dominic Hargreaves a écrit :
> Source: altree
> Version: 1.3.1-3
> Severity: grave
> Justification: package not usable
> Tags: sid
> X-Debbugs-Cc: debian-p...@lists.debian.org
> 
> The last upload, fixing #807423, unfortunately started creating empty
> binary packages on most (but not all) architectures. See for example
> 
> https://packages.debian.org/sid/amd64/altree/filelist
> https://packages.debian.org/sid/armel/altree/filelist
> https://packages.debian.org/sid/armhf/altree/filelist
> https://packages.debian.org/sid/i386/altree/filelist
> 
> I can't see from a quick glance what is going on here; CC-ing
> debian-perl in case anyone can help.

  It is an error from me. altree files are now installed in debian/tmp
instead of directly debian/altree by debian/rules, so a debian/altree.install
file is required.
  I will fix this in a few hours if nobody do it before. And I will
reincorporate Andreas modifications that were all good.

  Regards,
Vincent

> Cheers,
> Dominic.
> 


-- 
Vincent Danjean   GPG key ID 0xD17897FA vdanj...@debian.org
GPG key fingerprint: 621E 3509 654D D77C 43F5  CA4A F6AE F2AF D178 97FA
Unofficial pkgs: http://moais.imag.fr/membres/vincent.danjean/deb.html
APT repo:  deb http://people.debian.org/~vdanjean/debian unstable main



Bug#807493: altree: Produces empty binary package

2015-12-09 Thread Sebastiaan Couwenberg
On 09-12-15 16:31, Dominic Hargreaves wrote:
> The last upload, fixing #807423, unfortunately started creating empty
> binary packages on most (but not all) architectures. See for example
> 
> https://packages.debian.org/sid/amd64/altree/filelist
> https://packages.debian.org/sid/armel/altree/filelist
> https://packages.debian.org/sid/armhf/altree/filelist
> https://packages.debian.org/sid/i386/altree/filelist
> 
> I can't see from a quick glance what is going on here; CC-ing
> debian-perl in case anyone can help.

This change seems to be the cause:

-override_dh_auto_install:
-   $(MAKE) PREFIX=debian/altree/usr install
-

Because there is no install file, the files in debian/tmp are not
installed in any package, only the documentation gets installed via the
docs files.

Adding an altree.install file to install the files in debian/tmp/usr is
probably sufficient.

Kind Regards,

Bas

-- 
 GPG Key ID: 4096R/6750F10AE88D4AF1
Fingerprint: 8182 DE41 7056 408D 6146  50D1 6750 F10A E88D 4AF1