Bug#852930: Fixed by skiping two tests

2017-02-05 Thread Andreas Tille
Hi Markus,

On Sun, Feb 05, 2017 at 06:02:36PM +0100, Markus Koschany wrote:
> I can take care of the upload. If I don't hear any objections I will opt
> for maven.test.skip=true because that's also easier to review for the
> release team but I don't mind using your patch either.

I do not consider my solution in any aspect special - so feel free to
use whatever you consider sensible as long as the bug will be fixed.

Kind regards

 Andreas.

-- 
http://fam-tille.de

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


Bug#852930: Fixed by skiping two tests

2017-02-05 Thread tony mancill
On Sun, Feb 05, 2017 at 06:02:36PM +0100, Markus Koschany wrote:
> On 05.02.2017 15:11, Andreas Tille wrote:
> > tags 852930 pending
> > thanks
> > 
> > Hi,
> > 
> > I've commited 6f42250872ee5ca8cc93a74c8285643c27aa0592 to Git which
> > simply skips two tests that do not seem to finish for whatever reason.
> > Please verify this solution and either upload or ask me to do a NMU to
> > fix this bug.
> > 
> > If I do not hear from you I'll upload after three days to make sure the
> > rdepends of this package will not be removed from testing.
> 
> Hi Andreas,
> 
> thank you for the patch. I can confirm this would resolve the issue.
> Another option is to completely disable the tests in maven.properties by
> using maven.test.skip=true. We already use
> maven.test.failure.ignore=true. I don't think this is a big change in
> this case.
> 
> I can take care of the upload. If I don't hear any objections I will opt
> for maven.test.skip=true because that's also easier to review for the
> release team but I don't mind using your patch either.

No objections to either approach, thank you both for taking care of this.

Cheers,
tony

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


Bug#852930: Fixed by skiping two tests

2017-02-05 Thread Markus Koschany
On 05.02.2017 15:11, Andreas Tille wrote:
> tags 852930 pending
> thanks
> 
> Hi,
> 
> I've commited 6f42250872ee5ca8cc93a74c8285643c27aa0592 to Git which
> simply skips two tests that do not seem to finish for whatever reason.
> Please verify this solution and either upload or ask me to do a NMU to
> fix this bug.
> 
> If I do not hear from you I'll upload after three days to make sure the
> rdepends of this package will not be removed from testing.

Hi Andreas,

thank you for the patch. I can confirm this would resolve the issue.
Another option is to completely disable the tests in maven.properties by
using maven.test.skip=true. We already use
maven.test.failure.ignore=true. I don't think this is a big change in
this case.

I can take care of the upload. If I don't hear any objections I will opt
for maven.test.skip=true because that's also easier to review for the
release team but I don't mind using your patch either.

Alles Gute

Markus




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

Bug#852930: Fixed by skiping two tests

2017-02-05 Thread Andreas Tille
tags 852930 pending
thanks

Hi,

I've commited 6f42250872ee5ca8cc93a74c8285643c27aa0592 to Git which
simply skips two tests that do not seem to finish for whatever reason.
Please verify this solution and either upload or ask me to do a NMU to
fix this bug.

If I do not hear from you I'll upload after three days to make sure the
rdepends of this package will not be removed from testing.

Kind regards

 Andreas.

-- 
http://fam-tille.de

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