Bug#803669: Breaks QProcess in owncloud-client tests on mips an mipsel

[Dmitry Shachnev]

Hi Gaudenz,

On Wed, 04 Nov 2015 15:51:40 +0100, Gaudenz Steinlin wrote:
> The testcase from the upstream bug report does not work on the porter
> box with 5.5.1+dfsg-6 as well. So this fix is either not applied to this
> build or it's incomplete.
>
> This is the output from the test program:
> (sid_mipsel-dchroot)gaudenz@eder:~/testcase$ ./testcase 
> 24 Calling waitForStarted()
> 54 waitForStarted() returned true
> 55 Calling waitForFinished()
> 30086 waitForFinished() returned false
> 30086 errorString() is "Process operation timed out"
> QProcess: Destroyed while process ("/bin/true") is still running.

Are you sure you were testing with 5.5.1+dfsg-6?

This bug is definitely fixed there, here is the output of the same binary on
the same machine:

(sid_mipsel-dchroot)mitya57@eder:/home/gaudenz/testcase$ ./testcase
3 Calling waitForStarted()
9 waitForStarted() returned true
10 Calling waitForFinished()
11 waitForFinished() returned true
12 errorString() is "Unknown error"

Tried multiple times to be completely sure.

--
Dmitry Shachnev

signature.asc
Description: OpenPGP digital signature

Bug#803669: Breaks QProcess in owncloud-client tests on mips an mipsel

[Lisandro Damián Nicanor Pérez Meyer]

Version: 5.5.1+dfsg-6

[snip]
> > Tried multiple times to be completely sure.
> 
> You are absolutely right.

OK, closing the bug with the version the fix got in then :)

Thanks Gaudenz for trying again :)

-- 
Simulations are not data. In God we trust, all the others must supply data.
 Walter Opyd, "Show Me The Data" IEEE Spectrum's reader's comments,
 http://www.spectrum.ieee.org/nov04/4004

Lisandro Damián Nicanor Pérez Meyer
http://perezmeyer.com.ar/
http://perezmeyer.blogspot.com/


signature.asc
Description: This is a digitally signed message part.

Bug#803669: Breaks QProcess in owncloud-client tests on mips an mipsel

[Gaudenz Steinlin]

Hi

Dmitry Shachnev  writes:

> Hi Gaudenz,
>
> On Wed, 04 Nov 2015 15:51:40 +0100, Gaudenz Steinlin wrote:
>> The testcase from the upstream bug report does not work on the porter
>> box with 5.5.1+dfsg-6 as well. So this fix is either not applied to this
>> build or it's incomplete.
>>
>> This is the output from the test program:
>> (sid_mipsel-dchroot)gaudenz@eder:~/testcase$ ./testcase 
>> 24 Calling waitForStarted()
>> 54 waitForStarted() returned true
>> 55 Calling waitForFinished()
>> 30086 waitForFinished() returned false
>> 30086 errorString() is "Process operation timed out"
>> QProcess: Destroyed while process ("/bin/true") is still running.
>
> Are you sure you were testing with 5.5.1+dfsg-6?
>
> This bug is definitely fixed there, here is the output of the same binary on
> the same machine:
>
> (sid_mipsel-dchroot)mitya57@eder:/home/gaudenz/testcase$ ./testcase
> 3 Calling waitForStarted()
> 9 waitForStarted() returned true
> 10 Calling waitForFinished()
> 11 waitForFinished() returned true
> 12 errorString() is "Unknown error"
>
> Tried multiple times to be completely sure.

You are absolutely right. I had the shell I was testing this running in
screen and forgot that I set an LD_LIBRARY_PATH there the other day to
be able to test different Qt versions. So LD_LIBRARY_PATH was still
pointing to the old Qt version. After clearing the variable everything
is fine.

Thanks again for the quick fix and sorry for the noise.

Gaudenz


signature.asc
Description: PGP signature

Bug#803669: Breaks QProcess in owncloud-client tests on mips an mipsel

[Gaudenz Steinlin]

Gaudenz Steinlin  writes:

> Lisandro Damián Nicanor Pérez Meyer  writes:
>
>> forwarded 803669 https://bugreports.qt.io/browse/QTBUG-49168
>> tag 803669 upstream
>> thanks
>>
>> On Monday 02 November 2015 23:07:19 Gaudenz Steinlin wrote:
>> [snip]
>>> If I read the strace output right (attached) md5sum only tahkes less
>>> than 1 second. The long 2x30s delay comes from subsequent pselect calls
>>> which may be unrelated to waitForFinished, but related to the fact the
>>> QProcess tries to kill the no longer running md5sum process.
>>
>> You are right. My wonderful teammate Dmitry has filed [qtbug] upstream.
>>
>> [qtbug] 
>>
>> The fix should be already in unstable in 5.5.1+dfsg-6, we are waiting for 
>> the 
>> mips/mipsel buildds to pick the package. After that please try again.
>
> Now that -6 is on the mirror the porter box uses I retried the build but
> I still have the same build failure. So something is still missing here.
>
> Next I'll try the test program in the upstream bug report to see if that
> is fixed.

The testcase from the upstream bug report does not work on the porter
box with 5.5.1+dfsg-6 as well. So this fix is either not applied to this
build or it's incomplete.

This is the output from the test program:
(sid_mipsel-dchroot)gaudenz@eder:~/testcase$ ./testcase 
24 Calling waitForStarted()
54 waitForStarted() returned true
55 Calling waitForFinished()
30086 waitForFinished() returned false
30086 errorString() is "Process operation timed out"
QProcess: Destroyed while process ("/bin/true") is still running.

The strace output from running the testcase is attached.

Gaudenz


testcase.trace
Description: Binary data


signature.asc
Description: PGP signature

Bug#803669: Breaks QProcess in owncloud-client tests on mips an mipsel

[Gaudenz Steinlin]

Lisandro Damián Nicanor Pérez Meyer  writes:

> forwarded 803669 https://bugreports.qt.io/browse/QTBUG-49168
> tag 803669 upstream
> thanks
>
> On Monday 02 November 2015 23:07:19 Gaudenz Steinlin wrote:
> [snip]
>> If I read the strace output right (attached) md5sum only tahkes less
>> than 1 second. The long 2x30s delay comes from subsequent pselect calls
>> which may be unrelated to waitForFinished, but related to the fact the
>> QProcess tries to kill the no longer running md5sum process.
>
> You are right. My wonderful teammate Dmitry has filed [qtbug] upstream.
>
> [qtbug] 
>
> The fix should be already in unstable in 5.5.1+dfsg-6, we are waiting for the 
> mips/mipsel buildds to pick the package. After that please try again.

Now that -6 is on the mirror the porter box uses I retried the build but
I still have the same build failure. So something is still missing here.

Next I'll try the test program in the upstream bug report to see if that
is fixed.

Gaudenz



signature.asc
Description: PGP signature

Processed: Re: Bug#803669: Breaks QProcess in owncloud-client tests on mips an mipsel

[Debian Bug Tracking System]

Processing commands for cont...@bugs.debian.org:

> forwarded 803669 https://bugreports.qt.io/browse/QTBUG-49168
Bug #803669 [src:qtbase-opensource-src] Breaks QProcess in owncloud-client 
tests on mips an mipsel
Set Bug forwarded-to-address to 'https://bugreports.qt.io/browse/QTBUG-49168'.
> tag 803669 upstream
Bug #803669 [src:qtbase-opensource-src] Breaks QProcess in owncloud-client 
tests on mips an mipsel
Added tag(s) upstream.
> thanks
Stopping processing here.

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

Bug#803669: Breaks QProcess in owncloud-client tests on mips an mipsel

[Lisandro Damián Nicanor Pérez Meyer]

forwarded 803669 https://bugreports.qt.io/browse/QTBUG-49168
tag 803669 upstream
thanks

On Monday 02 November 2015 23:07:19 Gaudenz Steinlin wrote:
[snip]
> If I read the strace output right (attached) md5sum only tahkes less
> than 1 second. The long 2x30s delay comes from subsequent pselect calls
> which may be unrelated to waitForFinished, but related to the fact the
> QProcess tries to kill the no longer running md5sum process.

You are right. My wonderful teammate Dmitry has filed [qtbug] upstream.

[qtbug] 

The fix should be already in unstable in 5.5.1+dfsg-6, we are waiting for the 
mips/mipsel buildds to pick the package. After that please try again.

-- 
11: En Word, como se hace para pasar al proximo renglon si se alcanzo
el extramo derecho de la hoja.
* Se reinicia la maquina, se abre el Guord, se va a archivo, Nuevo
renglon, Preferencias, Abajo del otro renglon, Poner el cursor al
proncipio del nuevo renglon, Elegir misma letra que el anterior,
respetar la pagina anterior, respetar el margen anterior, respetar
numeracion de pagina, comenzar a escribir, click en "Ok deseo comenzar
a escribir", click en "Esta seguro que desea comenzar a Escribir",
click en "Ultima advertencia", click en "Comience a escribir"
Damian Nadales
http://mx.grulic.org.ar/lurker/message/20080307.141449.a70fb2fc.es.html

Lisandro Damián Nicanor Pérez Meyer
http://perezmeyer.com.ar/
http://perezmeyer.blogspot.com/


signature.asc
Description: This is a digitally signed message part.

Bug#803669: Breaks QProcess in owncloud-client tests on mips an mipsel

[Lisandro Damián Nicanor Pérez Meyer]

Hi Gaudenz!

On Sunday 01 November 2015 17:32:12 Gaudenz Steinlin wrote:
[snip]
> Setting the severity of this bug to serious because it causes another
> package to fail to build from source.
> 
> This version of Qt breaks the tests of owncloud-sync on mips and mipsel.
> Test also break when compiling version 2.0.0+dfsg-1 of owncloud-client
> which previously successfully built against the newer version of Qt. So I
> suspect this is a bug in Qt and not in owncloud-client.
> 
> See here for the failing build logs:
> https://buildd.debian.org/status/fetch.php?pkg=owncloud-client=mipsel
> er=2.0.2%2Bdfsg-1=1445685652
> 
> I tried to debug this on the mipsel porter box but could not resovlve
> the problem. This is what I found out:
> 
> - The problem is also present when running the tests with Qt
>   5.5.0+dfsg-3 so setting this version. This is the first version of Qt
>   5.5 available for mipsel. This is likely a bug introduced with Qt 5.5.
> 
> - This is the problematic part of the code in owncloud-client which
>   triggers the bug (see test/testfilesystem.h in owncloud-client):
>  
> https://anonscm.debian.org/cgit/pkg-owncloud/owncloud-client.git/tree/test/
> testfilesystem.h
> 
>  26 QByteArray shellSum( const QByteArray& cmd, const QString& file )
>  27 {
>  28QProcess md5;
>  29QStringList args;
>  30args.append(file);
>  31md5.start(cmd, args);
>  32QByteArray sumShell;
>  33qDebug() << "File: "<< file;
>  34
>  35if( md5.waitForFinished()  ) {
>  36
>  37  sumShell = md5.readAll();
>  38  sumShell = sumShell.left( sumShell.indexOf(' '));
>  39}
>  40return sumShell;
>  41 }
> 
>   This is called twice during the test to compute a md5/sha1 sum with the
> command line tool to compare this against owncloud-clients internal
> implementation. The test then fails because this function returns an empty
> string instead of the correct result.
> 
> - Running the test under strace shows that the md5sum/sha1sum call succeeds
> and returns the correct string. But apparently waitForFinished just hangs
> for 30s (default timeout value) and then returns an error.

An important detail here would be to know if the md5sum value gets returned 
during the 30 seconds that waitForFinished and +/- how many seconds 
before/after.
 
> At this point I'm out of ideas on how to further debug this. Help by Qt
> maintainers or mips porters would be appreciated.

If you can please test with md5.waitForFinished(6) on line 35. It's not 
strange to have great delays on mips and mipsel.

Kinds regards, Lisandro.

-- 
 hmm, el enchufe hace chispas...
<-- rata ha dejado este servidor ("Leaving").
 ouch
  Visto en #lugfi, irc.freenode.net

Lisandro Damián Nicanor Pérez Meyer
http://perezmeyer.com.ar/
http://perezmeyer.blogspot.com/


signature.asc
Description: This is a digitally signed message part.

Bug#803669: Breaks QProcess in owncloud-client tests on mips an mipsel

[Gaudenz Steinlin]

Source: qtbase-opensource-src
Version: 5.5.0+dfsg-3
Severity: serious

Setting the severity of this bug to serious because it causes another package 
to fail to build from source.

This version of Qt breaks the tests of owncloud-sync on mips and mipsel. Test 
also break
when compiling version 2.0.0+dfsg-1 of owncloud-client which previously
successfully built against the newer version of Qt. So I suspect this is
a bug in Qt and not in owncloud-client.

See here for the failing build logs:
https://buildd.debian.org/status/fetch.php?pkg=owncloud-client=mipsel=2.0.2%2Bdfsg-1=1445685652

I tried to debug this on the mipsel porter box but could not resovlve
the problem. This is what I found out:

- The problem is also present when running the tests with Qt
  5.5.0+dfsg-3 so setting this version. This is the first version of Qt
  5.5 available for mipsel. This is likely a bug introduced with Qt 5.5.

- This is the problematic part of the code in owncloud-client which
  triggers the bug (see test/testfilesystem.h in owncloud-client):
  
https://anonscm.debian.org/cgit/pkg-owncloud/owncloud-client.git/tree/test/testfilesystem.h

 26 QByteArray shellSum( const QByteArray& cmd, const QString& file )
 27 {
 28QProcess md5;
 29QStringList args;
 30args.append(file);
 31md5.start(cmd, args);
 32QByteArray sumShell;
 33qDebug() << "File: "<< file;
 34 
 35if( md5.waitForFinished()  ) {
 36 
 37  sumShell = md5.readAll();
 38  sumShell = sumShell.left( sumShell.indexOf(' '));
 39}
 40return sumShell;
 41 }

  This is called twice during the test to compute a md5/sha1 sum with the 
command 
  line tool to compare this against owncloud-clients internal implementation. 
The 
  test then fails because this function returns an empty string instead of the 
  correct result.

- Running the test under strace shows that the md5sum/sha1sum call succeeds and 
returns
  the correct string. But apparently waitForFinished just hangs for 30s 
(default timeout
  value) and then returns an error.

At this point I'm out of ideas on how to further debug this. Help by Qt 
maintainers or 
mips porters would be appreciated.

Gaudenz


-- System Information:
Debian Release: stretch/sid
  APT prefers testing-proposed-updates
  APT policy: (500, 'testing-proposed-updates'), (500, 'testing'), (100, 
'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.2.0-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages libqt5core5a depends on:
ii  libc6 2.19-22
ii  libgcc1   1:5.2.1-22
ii  libglib2.0-0  2.46.1-1
ii  libicu55  55.1-5
ii  libpcre16-3   2:8.35-7.2
ii  libstdc++65.2.1-22
ii  zlib1g1:1.2.8.dfsg-2+b1

Versions of packages libqt5core5a recommends:
ii  qttranslations5-l10n  5.5.1-2

Versions of packages libqt5core5a suggests:
ii  libthai0  0.1.22-2

-- no debconf information