[DRE-maint] Bug#632553: Bug#632553: gem2deb: dh_ruby ignores install directory

[Lucas Nussbaum]

On 05/07/11 at 12:18 -0700, Antonio Terceiro wrote:
> Hi,
> 
> Stig Sandbeck Mathisen escreveu isso aí:
> > 
> > Package: gem2deb
> > Version: 0.2.5
> > Severity: normal
> > 
> > *** Please type your report below this line ***
> > 
> > dh_ruby installs to a directory first binary package, and not to the
> > directory given as argument to --install
> > 
> > (on a multi package, this will be [...]/debian/tmp)
> > 
> > Example from build log, using the default debian/rules created by gem2deb:
> > 
> > #    dh_auto_install -O--buildsystem=ruby
> > # 	dh_ruby --install /home/ssm/Code/tmp/build-area/puppet-2.7.1/debian/tmp
> > #   Entering dh_ruby --install
> > # install -d /home/ssm/Code/tmp/build-area/puppet-2.7.1/debian/puppet-common/usr/bin
> > # install -D -m755 bin/puppetca /home/ssm/Code/tmp/build-area/puppet-2.7.1/debian/puppet-common/usr/bin/puppetca
> > # install -D -m755 bin/puppetdoc /home/ssm/Code/tmp/build-area/puppet-2.7.1/debian/puppet-common/usr/bin/puppetdoc
> > # install -D -m755 bin/filebucket /home/ssm/Code/tmp/build-area/puppet-2.7.1/debian/puppet-common/usr/bin/filebucket
> > # install -D -m755 bin/puppetd /home/ssm/Code/tmp/build-area/puppet-2.7.1/debian/puppet-common/usr/bin/puppetd
> > 
> > In the file /usr/lib/ruby/vendor_ruby/gem2deb/dh_ruby.rb, it gets argv
> > from the caller, but does not pull the installation directory from
> > there. Rather, it sets "package" to the result of "packages.first", and
> > installs to that instead.
> > 
> > debhelper's dh_auto_install will supply the install directory to dh_ruby
> > --install (se excerpt from build log). Please check if gem2deb can use
> > this to make source packages with multiple binary packages.
> 
> Perhaps gem2deb should always use debian/tmp as intallation target
> whenever there is more than one binary package in debian/control,
> except any transitional packages.

That sounds good. Maybe with displaying a verbose message explaining why
debian/ruby-foo was chosen instead of debian/tmp.

- lucas