Re: [DRE-maint] Bug#632553: gem2deb: dh_ruby ignores install directory
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
Reply to: