Processed: Re: Bug#869363: gradle-helper-plugin: Incorrect usage of doit_in_builddir in gradle.pm (Debhelper Buildsystem)

2017-07-23 Thread Debian Bug Tracking System 
Processing commands for cont...@bugs.debian.org:

> reassign 869363 gradle-debian-helper 1.5.1
Bug #869363 [gradle-helper-plugin] gradle-helper-plugin: Incorrect usage of 
doit_in_builddir in gradle.pm (Debhelper Buildsystem)
Warning: Unknown package 'gradle-helper-plugin'
Bug reassigned from package 'gradle-helper-plugin' to 'gradle-debian-helper'.
No longer marked as found in versions 1.5.1.
Ignoring request to alter fixed versions of bug #869363 to the same values 
previously set
Bug #869363 [gradle-debian-helper] gradle-helper-plugin: Incorrect usage of 
doit_in_builddir in gradle.pm (Debhelper Buildsystem)
Marked as found in versions gradle-debian-helper/1.5.1.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
869363: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=869363
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems

__
This is the maintainer address of Debian's Java team
. 
Please use
debian-j...@lists.debian.org for discussions and questions.

Re: Bug#869363: gradle-helper-plugin: Incorrect usage of doit_in_builddir in gradle.pm (Debhelper Buildsystem)

2017-07-23 Thread Adrian Bunk 
reassign 869363 gradle-debian-helper 1.5.1
thanks

On Sat, Jul 22, 2017 at 07:14:11PM +0200, Niels Thykier wrote:
> Package: gradle-helper-plugin
> Version: 1.5.1
> Severity: important
> 
> Hi,
> 
> There is bug in gradle.pm that is hidden by bug in Dh_lib.  I intend
> to fix the latter soon, which will cause issues in gradle.pm once that
> happens.  This bug is here:
> 
> 
> > sub clean {
> > my $this=shift;
> > 
> > $this->doit_in_builddir("find . -wholename .*build/tmp | xargs echo | 
> > sed -e 's^build/tmp^build^g' | xargs rm -Rf");
> 
> ^^
> > $this->doit_in_builddir("find . -wholename .*build/debian | xargs echo 
> > | sed -e 's^build/tmp^build^g' | xargs rm -Rf");
> 
> ^
> > $this->doit_in_builddir("rm", "-Rf", "$this->{cwd}/.gradle", 
> > "$this->{cwd}/buildSrc/.gradle", ".m2");
> > }
> 
> 
> The "doit_in_builddir" method calls "print_and_doit", which in turn
> calls "doit".  The API of doit intends for it to *not* fork a shell,
> but the above only works in a shell call.
> 
> I will introduce a "complex_doit_in_builddir" in debhelper/10.7, which
> you can use instead.  You should be able to support both the new call
> plus work with older versions of debhelper by relying on "can" a la:
> 
> """
>   if ($this->can('complex_doit_in_builddir')) {
> $this->complex_doit_in_builddir(...);
> $this->complex_doit_in_builddir(...);
>   } else {
> $this->doit_in_builddir(...);
> $this->doit_in_builddir(...);
>   }
> """
> 
> Thanks,
> ~Niels


__
This is the maintainer address of Debian's Java team
. 
Please use
debian-j...@lists.debian.org for discussions and questions.