Re: [Debian-med-packaging] Bug#876147: Bug#876147: camp frequently FTBFS on 64bit big endian: camptest-qt (Failed)

2018-04-16 Thread Flavien Bridault
Hi Adrian,

I sent you a private email twice last week but I did not receive any
answer from you so I am afraid they did not reach you. Or maybe you were
just away or not available, if that's the case sorry for spaming you...

Meanwhile I commited a fix upstream
(https://github.com/fw4spl-org/camp/pull/2) so I would just want to try
it before proposing the fix in the package.

Cheers


Le 06/04/2018 à 14:36, John Paul Adrian Glaubitz a écrit :
> On 04/06/2018 02:34 PM, Flavien Bridault wrote:
>> Okay thanks a lot for your quick answer !
>>
>> Maybe I misunderstood the excuses
>> https://qa.debian.org/excuses.php?package=camp, but the build failure on
>> s390x looks very similar at the endianess issue we have on sparc64, so
>> this should also solves that. If not I will contact s390x folks.
> We can give a try non-theless.
>
> Please send me a private email with your desired username. Encrypt the
> mail using my GPG below (you can find in the Debian GPG keyring).
>
> Use your key that was signed by the Debian Developer you mentioned.
>
> Adrian
>

-- 
*Flavien BRIDAULT*
Ingénieur de Recherche

fbrida...@ircad.fr

*IRCAD France*
1, place de l'Hôpital - 67091 Strasbourg Cedex - FRANCE

http://www.ircad.fr/ 




signature.asc
Description: OpenPGP digital signature


Re: [Debian-med-packaging] Bug#876147: Bug#876147: camp frequently FTBFS on 64bit big endian: camptest-qt (Failed)

2018-04-16 Thread John Paul Adrian Glaubitz

Hello!

On 04/16/2018 09:15 AM, Flavien Bridault wrote:

I sent you a private email twice last week but I did not receive any
answer from you so I am afraid they did not reach you. Or maybe you were
just away or not available, if that's the case sorry for spaming you...


I forgot to follow up on that, sorry. I have been very busy. But from
the logs it's also apparent that the package builds fine - including the
tests - on sparc64, so I'm not sure the sparc64 porterbox would be of
any help.

You should preferably get access to an s390x porterbox but I don't have
any that I can provide access to. You would have to contact the correct
people at Debian for that.

See: https://dsa.debian.org/doc/guest-account/


Meanwhile I commited a fix upstream
(https://github.com/fw4spl-org/camp/pull/2) so I would just want to try
it before proposing the fix in the package.

I'll test that for you on s390x.

Adrian

--
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaub...@debian.org
`. `'   Freie Universitaet Berlin - glaub...@physik.fu-berlin.de
  `-GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913



Re: [Debian-med-packaging] Bug#876147: Bug#876147: camp frequently FTBFS on 64bit big endian: camptest-qt (Failed)

2018-04-16 Thread John Paul Adrian Glaubitz

On 04/16/2018 11:16 AM, John Paul Adrian Glaubitz wrote:

Meanwhile I commited a fix upstream
(https://github.com/fw4spl-org/camp/pull/2) so I would just want to try
it before proposing the fix in the package.

I'll test that for you on s390x.


That patch doesn't apply to the Debian version of camp:

glaubitz@zelenka:~/camp/camp-0.8.1$ patch -p1 < ~/fix-s390x.patch
patching file .gitignore
patching file include/camp/qt/qtfunction.hpp
patching file include/camp/qt/qthelper.hpp
patching file include/camp/valuemapper.hpp
Hunk #2 FAILED at 40.
Hunk #3 succeeded at 96 (offset 2 lines).
Hunk #4 succeeded at 126 (offset 2 lines).
Hunk #5 succeeded at 139 (offset 2 lines).
Hunk #6 succeeded at 156 (offset 2 lines).
Hunk #7 succeeded at 183 (offset 2 lines).
Hunk #8 succeeded at 205 (offset 2 lines).
Hunk #9 succeeded at 234 (offset 2 lines).
Hunk #10 succeeded at 250 (offset 2 lines).
Hunk #11 succeeded at 279 (offset 2 lines).
Hunk #12 succeeded at 326 (offset 2 lines).
Hunk #13 succeeded at 345 (offset 2 lines).
1 out of 13 hunks FAILED -- saving rejects to file 
include/camp/valuemapper.hpp.rej
patching file test/arrayproperty.hpp
Hunk #2 FAILED at 32.
Hunk #3 succeeded at 78 (offset 2 lines).
1 out of 3 hunks FAILED -- saving rejects to file test/arrayproperty.hpp.rej
patching file test/qt/propertymapping.cpp
Hunk #2 succeeded at 34 with fuzz 2 (offset 2 lines).
Hunk #3 succeeded at 47 (offset 2 lines).
Hunk #4 succeeded at 95 (offset 2 lines).
Hunk #5 succeeded at 106 (offset 2 lines).
glaubitz@zelenka:~/camp/camp-0.8.1$

--
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaub...@debian.org
`. `'   Freie Universitaet Berlin - glaub...@physik.fu-berlin.de
  `-GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913



Re: [Debian-med-packaging] Bug#876147: Bug#876147: camp frequently FTBFS on 64bit big endian: camptest-qt (Failed)

2018-04-16 Thread Flavien Bridault
Weird, it applies well for me on the master branch of the debian
repository. Try the patch attached to this email or directly this link

https://patch-diff.githubusercontent.com/raw/fw4spl-org/camp/pull/2.diff


Le 16/04/2018 à 11:21, John Paul Adrian Glaubitz a écrit :
> On 04/16/2018 11:16 AM, John Paul Adrian Glaubitz wrote:
>>> Meanwhile I commited a fix upstream
>>> (https://github.com/fw4spl-org/camp/pull/2) so I would just want to try
>>> it before proposing the fix in the package.
>> I'll test that for you on s390x.
>
> That patch doesn't apply to the Debian version of camp:
>
> glaubitz@zelenka:~/camp/camp-0.8.1$ patch -p1 < ~/fix-s390x.patch
> patching file .gitignore
> patching file include/camp/qt/qtfunction.hpp
> patching file include/camp/qt/qthelper.hpp
> patching file include/camp/valuemapper.hpp
> Hunk #2 FAILED at 40.
> Hunk #3 succeeded at 96 (offset 2 lines).
> Hunk #4 succeeded at 126 (offset 2 lines).
> Hunk #5 succeeded at 139 (offset 2 lines).
> Hunk #6 succeeded at 156 (offset 2 lines).
> Hunk #7 succeeded at 183 (offset 2 lines).
> Hunk #8 succeeded at 205 (offset 2 lines).
> Hunk #9 succeeded at 234 (offset 2 lines).
> Hunk #10 succeeded at 250 (offset 2 lines).
> Hunk #11 succeeded at 279 (offset 2 lines).
> Hunk #12 succeeded at 326 (offset 2 lines).
> Hunk #13 succeeded at 345 (offset 2 lines).
> 1 out of 13 hunks FAILED -- saving rejects to file
> include/camp/valuemapper.hpp.rej
> patching file test/arrayproperty.hpp
> Hunk #2 FAILED at 32.
> Hunk #3 succeeded at 78 (offset 2 lines).
> 1 out of 3 hunks FAILED -- saving rejects to file
> test/arrayproperty.hpp.rej
> patching file test/qt/propertymapping.cpp
> Hunk #2 succeeded at 34 with fuzz 2 (offset 2 lines).
> Hunk #3 succeeded at 47 (offset 2 lines).
> Hunk #4 succeeded at 95 (offset 2 lines).
> Hunk #5 succeeded at 106 (offset 2 lines).
> glaubitz@zelenka:~/camp/camp-0.8.1$
>

-- 
*Flavien BRIDAULT*
Ingénieur de Recherche

fbrida...@ircad.fr

*IRCAD France*
1, place de l'Hôpital - 67091 Strasbourg Cedex - FRANCE

http://www.ircad.fr/ 

diff --git a/.gitignore b/.gitignore
index ce734c4..9a073a8 100644
--- a/.gitignore
+++ b/.gitignore
@@ -28,6 +28,7 @@ qtc-gdbmacros/
 *.o
 *.a
 moc_*
+build/
 
 *.cmake
 CMakeFiles/
diff --git a/include/camp/qt/qtfunction.hpp b/include/camp/qt/qtfunction.hpp
index 96484ab..912383b 100644
--- a/include/camp/qt/qtfunction.hpp
+++ b/include/camp/qt/qtfunction.hpp
@@ -15,10 +15,10 @@
 ** to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
 ** copies of the Software, and to permit persons to whom the Software is
 ** furnished to do so, subject to the following conditions:
-** 
+**
 ** The above copyright notice and this permission notice shall be included in
 ** all copies or substantial portions of the Software.
-** 
+**
 ** THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 ** IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 ** FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
@@ -86,7 +86,7 @@ class QtFunction : public camp::Function
 
 case 1:
 {
-QVariant arg1 = QtHelper::valueToVariant(args[0]);
+const auto arg1 = QtHelper::argumentToVariant(args[0], m_metaMethod.parameterType(0));
 m_metaMethod.invoke(object.get(), Qt::DirectConnection, ret
   , QGenericArgument(arg1.typeName(), arg1.data()));
 break;
@@ -94,8 +94,8 @@ class QtFunction : public camp::Function
 
 case 2:
 {
-QVariant arg1 = QtHelper::valueToVariant(args[0]);
-QVariant arg2 = QtHelper::valueToVariant(args[1]);
+const auto arg1 = QtHelper::argumentToVariant(args[0], m_metaMethod.parameterType(0));
+const auto arg2 = QtHelper::argumentToVariant(args[1], m_metaMethod.parameterType(1));
 m_metaMethod.invoke(object.get(), Qt::DirectConnection, ret
   , QGenericArgument(arg1.typeName(), arg1.data())
   , QGenericArgument(arg2.typeName(), arg2.data()));
@@ -104,9 +104,9 @@ class QtFunction : public camp::Function
 
 case 3:
 {
-QVariant arg1 = QtHelper::valueToVariant(args[0]);
-QVariant arg2 = QtHelper::valueToVariant(args[1]);
-QVariant arg3 = QtHelper::valueToVariant(args[2]);
+const auto arg1 = QtHelper::argumentToVariant(args[0], m_metaMethod.parameterType(0));
+const auto arg2 = QtHelper::argumentToVariant(args[1], m_metaMethod.parameterType(1));
+const auto arg3 = QtHelper::argumentToVariant(args[2], m_metaMethod.parameterType(2));
 m_metaMethod.invoke(object.get(), Qt::DirectConnection, ret
   , QGenericArgument(arg1.typeName(), arg1.data())
 

Re: [Debian-med-packaging] Bug#876147: Bug#876147: camp frequently FTBFS on 64bit big endian: camptest-qt (Failed)

2018-04-16 Thread John Paul Adrian Glaubitz

On 04/16/2018 11:58 AM, Flavien Bridault wrote:

Weird, it applies well for me on the master branch of the debian
repository. Try the patch attached to this email or directly this link

https://patch-diff.githubusercontent.com/raw/fw4spl-org/camp/pull/2.diff


Tried to apply against the current package in Debian unstable:

glaubitz@zelenka:~$ md5sum fix-s390.diff
3fae8e5c44e239e74e2ef14a667126b4  fix-s390.diff
glaubitz@zelenka:~$ cd camp/
glaubitz@zelenka:~/camp$ dget -u 
http://deb.debian.org/debian/pool/main/c/camp/camp_0.8.1-2.dsc
dget: retrieving http://deb.debian.org/debian/pool/main/c/camp/camp_0.8.1-2.dsc
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
100   321  100   3210 0   4346  0 --:--:-- --:--:-- --:--:--  4397
100  2000  100  20000 0  20750  0 --:--:-- --:--:-- --:--:-- 20750
dget: retrieving 
http://deb.debian.org/debian/pool/main/c/camp/camp_0.8.1.orig.tar.gz
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
100   327  100   3270 0   4570  0 --:--:-- --:--:-- --:--:--  4605
100  534k  100  534k0 0  4044k  0 --:--:-- --:--:-- --:--:-- 4044k
dget: retrieving 
http://deb.debian.org/debian/pool/main/c/camp/camp_0.8.1-2.debian.tar.xz
  % Total% Received % Xferd  Average Speed   TimeTime Time  Current
 Dload  Upload   Total   SpentLeft  Speed
100   331  100   3310 0   4809  0 --:--:-- --:--:-- --:--:--  4867
100  5272  100  52720 0  63979  0 --:--:-- --:--:-- --:--:-- 63979
dpkg-source: info: extracting camp in camp-0.8.1
dpkg-source: info: unpacking camp_0.8.1.orig.tar.gz
dpkg-source: info: unpacking camp_0.8.1-2.debian.tar.xz
dpkg-source: info: applying remove_licences_files.patch
dpkg-source: info: applying hide_boost_from_qt4moc.patch
glaubitz@zelenka:~/camp$ cd camp-0.8.1/
glaubitz@zelenka:~/camp/camp-0.8.1$ patch -p1 < ~/fix-s390x.patch
patching file .gitignore
patching file include/camp/qt/qtfunction.hpp
patching file include/camp/qt/qthelper.hpp
patching file include/camp/valuemapper.hpp
Hunk #2 FAILED at 40.
Hunk #3 succeeded at 96 (offset 2 lines).
Hunk #4 succeeded at 126 (offset 2 lines).
Hunk #5 succeeded at 139 (offset 2 lines).
Hunk #6 succeeded at 156 (offset 2 lines).
Hunk #7 succeeded at 183 (offset 2 lines).
Hunk #8 succeeded at 205 (offset 2 lines).
Hunk #9 succeeded at 234 (offset 2 lines).
Hunk #10 succeeded at 250 (offset 2 lines).
Hunk #11 succeeded at 279 (offset 2 lines).
Hunk #12 succeeded at 326 (offset 2 lines).
Hunk #13 succeeded at 345 (offset 2 lines).
1 out of 13 hunks FAILED -- saving rejects to file 
include/camp/valuemapper.hpp.rej
patching file test/arrayproperty.hpp
Hunk #2 FAILED at 32.
Hunk #3 succeeded at 78 (offset 2 lines).
1 out of 3 hunks FAILED -- saving rejects to file test/arrayproperty.hpp.rej
patching file test/qt/propertymapping.cpp
Hunk #2 succeeded at 34 with fuzz 2 (offset 2 lines).
Hunk #3 succeeded at 47 (offset 2 lines).
Hunk #4 succeeded at 95 (offset 2 lines).
Hunk #5 succeeded at 106 (offset 2 lines).
glaubitz@zelenka:~/camp/camp-0.8.1$

--
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaub...@debian.org
`. `'   Freie Universitaet Berlin - glaub...@physik.fu-berlin.de
  `-GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913



Re: [Debian-med-packaging] Bug#876147: Bug#876147: camp frequently FTBFS on 64bit big endian: camptest-qt (Failed)

2018-04-16 Thread Flavien Bridault
Sorry the patch is ofc meant to be applied on the raw upstream, not
after debian patches being applied. I was applying it directly into the
git repo so that's why I did not have any issue. If you want to keep
trying it this way, I attached a rebased version of the patch that
should apply well on top of other.


Le 16/04/2018 à 13:49, John Paul Adrian Glaubitz a écrit :
> On 04/16/2018 11:58 AM, Flavien Bridault wrote:
>> Weird, it applies well for me on the master branch of the debian
>> repository. Try the patch attached to this email or directly this link
>>
>> https://patch-diff.githubusercontent.com/raw/fw4spl-org/camp/pull/2.diff
>
> Tried to apply against the current package in Debian unstable:
>
> glaubitz@zelenka:~$ md5sum fix-s390.diff
> 3fae8e5c44e239e74e2ef14a667126b4  fix-s390.diff
> glaubitz@zelenka:~$ cd camp/
> glaubitz@zelenka:~/camp$ dget -u
> http://deb.debian.org/debian/pool/main/c/camp/camp_0.8.1-2.dsc
> dget: retrieving
> http://deb.debian.org/debian/pool/main/c/camp/camp_0.8.1-2.dsc
>   % Total    % Received % Xferd  Average Speed   Time    Time
> Time  Current
>  Dload  Upload   Total   Spent   
> Left  Speed
> 100   321  100   321    0 0   4346  0 --:--:-- --:--:--
> --:--:--  4397
> 100  2000  100  2000    0 0  20750  0 --:--:-- --:--:--
> --:--:-- 20750
> dget: retrieving
> http://deb.debian.org/debian/pool/main/c/camp/camp_0.8.1.orig.tar.gz
>   % Total    % Received % Xferd  Average Speed   Time    Time
> Time  Current
>  Dload  Upload   Total   Spent   
> Left  Speed
> 100   327  100   327    0 0   4570  0 --:--:-- --:--:--
> --:--:--  4605
> 100  534k  100  534k    0 0  4044k  0 --:--:-- --:--:--
> --:--:-- 4044k
> dget: retrieving
> http://deb.debian.org/debian/pool/main/c/camp/camp_0.8.1-2.debian.tar.xz
>   % Total    % Received % Xferd  Average Speed   Time    Time
> Time  Current
>  Dload  Upload   Total   Spent   
> Left  Speed
> 100   331  100   331    0 0   4809  0 --:--:-- --:--:--
> --:--:--  4867
> 100  5272  100  5272    0 0  63979  0 --:--:-- --:--:--
> --:--:-- 63979
> dpkg-source: info: extracting camp in camp-0.8.1
> dpkg-source: info: unpacking camp_0.8.1.orig.tar.gz
> dpkg-source: info: unpacking camp_0.8.1-2.debian.tar.xz
> dpkg-source: info: applying remove_licences_files.patch
> dpkg-source: info: applying hide_boost_from_qt4moc.patch
> glaubitz@zelenka:~/camp$ cd camp-0.8.1/
> glaubitz@zelenka:~/camp/camp-0.8.1$ patch -p1 < ~/fix-s390x.patch
> patching file .gitignore
> patching file include/camp/qt/qtfunction.hpp
> patching file include/camp/qt/qthelper.hpp
> patching file include/camp/valuemapper.hpp
> Hunk #2 FAILED at 40.
> Hunk #3 succeeded at 96 (offset 2 lines).
> Hunk #4 succeeded at 126 (offset 2 lines).
> Hunk #5 succeeded at 139 (offset 2 lines).
> Hunk #6 succeeded at 156 (offset 2 lines).
> Hunk #7 succeeded at 183 (offset 2 lines).
> Hunk #8 succeeded at 205 (offset 2 lines).
> Hunk #9 succeeded at 234 (offset 2 lines).
> Hunk #10 succeeded at 250 (offset 2 lines).
> Hunk #11 succeeded at 279 (offset 2 lines).
> Hunk #12 succeeded at 326 (offset 2 lines).
> Hunk #13 succeeded at 345 (offset 2 lines).
> 1 out of 13 hunks FAILED -- saving rejects to file
> include/camp/valuemapper.hpp.rej
> patching file test/arrayproperty.hpp
> Hunk #2 FAILED at 32.
> Hunk #3 succeeded at 78 (offset 2 lines).
> 1 out of 3 hunks FAILED -- saving rejects to file
> test/arrayproperty.hpp.rej
> patching file test/qt/propertymapping.cpp
> Hunk #2 succeeded at 34 with fuzz 2 (offset 2 lines).
> Hunk #3 succeeded at 47 (offset 2 lines).
> Hunk #4 succeeded at 95 (offset 2 lines).
> Hunk #5 succeeded at 106 (offset 2 lines).
> glaubitz@zelenka:~/camp/camp-0.8.1$
>

-- 
*Flavien BRIDAULT*
Ingénieur de Recherche

fbrida...@ircad.fr

*IRCAD France*
1, place de l'Hôpital - 67091 Strasbourg Cedex - FRANCE

http://www.ircad.fr/ 

diff -uNr camp-0.8.1.old/.gitignore camp-0.8.1/.gitignore
--- camp-0.8.1.old/.gitignore	2018-04-16 14:08:14.403457120 +0200
+++ camp-0.8.1/.gitignore	2018-04-16 14:08:19.987457060 +0200
@@ -28,6 +28,7 @@
 *.o
 *.a
 moc_*
+build/
 
 *.cmake
 CMakeFiles/
diff -uNr camp-0.8.1.old/include/camp/qt/qtfunction.hpp camp-0.8.1/include/camp/qt/qtfunction.hpp
--- camp-0.8.1.old/include/camp/qt/qtfunction.hpp	2018-04-16 14:08:14.411457120 +0200
+++ camp-0.8.1/include/camp/qt/qtfunction.hpp	2018-04-16 14:08:19.987457060 +0200
@@ -15,10 +15,10 @@
 ** to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
 ** copies of the Software, and to permit persons to whom the Software is
 ** furnished to do so, subject to the following conditions:
-** 
+**
 ** The above copyright notice and this permission notice shall be included in
 ** all copies or substantial portions of the Software.
-** 
+**
 ** THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 ** I

Re: [Debian-med-packaging] Bug#876147: Bug#876147: camp frequently FTBFS on 64bit big endian: camptest-qt (Failed)

2018-04-16 Thread John Paul Adrian Glaubitz

On 04/16/2018 02:21 PM, Flavien Bridault wrote:

Sorry the patch is ofc meant to be applied on the raw upstream, not
after debian patches being applied. I was applying it directly into the
git repo so that's why I did not have any issue. If you want to keep
trying it this way, I attached a rebased version of the patch that
should apply well on top of other.


Ok, this works:

(sid_s390x-dchroot)glaubitz@zelenka:~/camp/camp-0.8.1/obj-s390x-linux-gnu$ make 
test
Running tests...
/usr/bin/ctest --force-new-ctest-process
Test project /home/glaubitz/camp/camp-0.8.1/obj-s390x-linux-gnu
Start 1: camptest
1/2 Test #1: camptest .   Passed0.01 sec
Start 2: camptest-qt
2/2 Test #2: camptest-qt ..   Passed0.00 sec

100% tests passed, 0 tests failed out of 2

Total Test time (real) =   0.02 sec
(sid_s390x-dchroot)glaubitz@zelenka:~/camp/camp-0.8.1/obj-s390x-linux-gnu$

--
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaub...@debian.org
`. `'   Freie Universitaet Berlin - glaub...@physik.fu-berlin.de
  `-GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913



Re: [Debian-med-packaging] Bug#876147: Bug#876147: camp frequently FTBFS on 64bit big endian: camptest-qt (Failed)

2018-04-16 Thread Flavien Bridault
Ok great, as soon as the merge request is accepted upstream, which
should happen very soon, I'll propose the update.

Thanks a lot for testing this.

Le 16/04/2018 à 14:34, John Paul Adrian Glaubitz a écrit :
> On 04/16/2018 02:21 PM, Flavien Bridault wrote:
>> Sorry the patch is ofc meant to be applied on the raw upstream, not
>> after debian patches being applied. I was applying it directly into the
>> git repo so that's why I did not have any issue. If you want to keep
>> trying it this way, I attached a rebased version of the patch that
>> should apply well on top of other.
>
> Ok, this works:
>
> (sid_s390x-dchroot)glaubitz@zelenka:~/camp/camp-0.8.1/obj-s390x-linux-gnu$
> make test
> Running tests...
> /usr/bin/ctest --force-new-ctest-process
> Test project /home/glaubitz/camp/camp-0.8.1/obj-s390x-linux-gnu
>     Start 1: camptest
> 1/2 Test #1: camptest .   Passed    0.01 sec
>     Start 2: camptest-qt
> 2/2 Test #2: camptest-qt ..   Passed    0.00 sec
>
> 100% tests passed, 0 tests failed out of 2
>
> Total Test time (real) =   0.02 sec
> (sid_s390x-dchroot)glaubitz@zelenka:~/camp/camp-0.8.1/obj-s390x-linux-gnu$
>
>

-- 
*Flavien BRIDAULT*
Ingénieur de Recherche

fbrida...@ircad.fr

*IRCAD France*
1, place de l'Hôpital - 67091 Strasbourg Cedex - FRANCE

http://www.ircad.fr/ 




signature.asc
Description: OpenPGP digital signature


Re: [Debian-med-packaging] Bug#876147: Bug#876147: camp frequently FTBFS on 64bit big endian: camptest-qt (Failed)

2018-04-16 Thread Flavien Bridault
Hi guys,

This looks ok for me now, I updated to 0.8.2 which includes the fix
proposal. Please upload if that looks ok for you.

Cheers,


Le 16/04/2018 à 14:47, Flavien Bridault a écrit :
> Ok great, as soon as the merge request is accepted upstream, which
> should happen very soon, I'll propose the update.
>
> Thanks a lot for testing this.
>
> Le 16/04/2018 à 14:34, John Paul Adrian Glaubitz a écrit :
>> On 04/16/2018 02:21 PM, Flavien Bridault wrote:
>>> Sorry the patch is ofc meant to be applied on the raw upstream, not
>>> after debian patches being applied. I was applying it directly into the
>>> git repo so that's why I did not have any issue. If you want to keep
>>> trying it this way, I attached a rebased version of the patch that
>>> should apply well on top of other.
>> Ok, this works:
>>
>> (sid_s390x-dchroot)glaubitz@zelenka:~/camp/camp-0.8.1/obj-s390x-linux-gnu$
>> make test
>> Running tests...
>> /usr/bin/ctest --force-new-ctest-process
>> Test project /home/glaubitz/camp/camp-0.8.1/obj-s390x-linux-gnu
>>     Start 1: camptest
>> 1/2 Test #1: camptest .   Passed    0.01 sec
>>     Start 2: camptest-qt
>> 2/2 Test #2: camptest-qt ..   Passed    0.00 sec
>>
>> 100% tests passed, 0 tests failed out of 2
>>
>> Total Test time (real) =   0.02 sec
>> (sid_s390x-dchroot)glaubitz@zelenka:~/camp/camp-0.8.1/obj-s390x-linux-gnu$
>>
>>
>
>
> ___
> Debian-med-packaging mailing list
> debian-med-packag...@alioth-lists.debian.net
> https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/debian-med-packaging

-- 
*Flavien BRIDAULT*
Ingénieur de Recherche

fbrida...@ircad.fr

*IRCAD France*
1, place de l'Hôpital - 67091 Strasbourg Cedex - FRANCE

http://www.ircad.fr/ 



signature.asc
Description: OpenPGP digital signature


Re: no quadmath.h on ppc64 ?

2018-04-16 Thread Dennis Clarke

On 12/04/18 01:39 PM, Dennis Clarke wrote:

On 05/04/18 01:24 AM, John Paul Adrian Glaubitz wrote:

On 04/05/2018 12:01 AM, Dennis Clarke wrote:

So there we see "--disable-libquadmath --disable-libquadmath-support".

damn.   :-(


Please file a bug report against Debian's gcc-5, gcc-6, gcc-7 and gcc-8
packages if you think that libquadmath must be enabled.

There are usually good reasons for such defaults but sometimes they might
just be a historic artifact and you were the first person to stumble
over it.


Working on it :

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=895452

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82686


Dennis



Managed a bootstrap that didn't blow up.

At the moment the assembly output from a trivial test shows that the IBM 
extended precision long double type is the default unless one actually 
asks for the IEEE754-2008 128-bit datatype.


dclarke@nix:~/pgm/C/ieee754$ 
PATH=/usr/local/build/gcc-7.3.0_linux_4.15.0-2-powerpc64_ppc64.005/gcc:$PATH 
xgcc 
-I/usr/local/build/gcc-7.3.0_linux_4.15.0-2-powerpc64_ppc64.005/gcc/include 
-m64 -g -Wl,-rpath=/usr/local/lib -mcpu=970 -maltivec -mfull-toc 
-mregnames -mabi=ieeelongdouble -S -o ld.s ld.c

xgcc: warning: using IEEE extended precision long double
cc1: warning: using IEEE extended precision long double
dclarke@nix:~/pgm/C/ieee754$ grep "quad" ld.s | grep "0x"
.quad   0x4000921fb54442d1,0x8469898cc51701b8

That data is correct.

dclarke@nix:~/pgm/C/ieee754$ 
PATH=/usr/local/build/gcc-7.3.0_linux_4.15.0-2-powerpc64_ppc64.005/gcc:$PATH 
xgcc 
-I/usr/local/build/gcc-7.3.0_linux_4.15.0-2-powerpc64_ppc64.005/gcc/include 
-m64 -g -Wl,-rpath=/usr/local/lib -mcpu=970 -maltivec -mfull-toc 
-mregnames -mabi=ibmlongdouble -S -o ld.s ld.c

xgcc: warning: using IBM extended precision long double
cc1: warning: using IBM extended precision long double
dclarke@nix:~/pgm/C/ieee754$ grep "quad" ld.s | grep "0x"
.quad   0x400921fb54442d18,0x3ca1a62633145c06

That is a whole other beast of a different colour.

dclarke@nix:~/pgm/C/ieee754$ 
PATH=/usr/local/build/gcc-7.3.0_linux_4.15.0-2-powerpc64_ppc64.005/gcc:$PATH 
xgcc 
-I/usr/local/build/gcc-7.3.0_linux_4.15.0-2-powerpc64_ppc64.005/gcc/include 
-m64 -g -Wl,-rpath=/usr/local/lib -mcpu=970 -maltivec -mfull-toc 
-mregnames -S -o ld.s ld.c

dclarke@nix:~/pgm/C/ieee754$ grep "quad" ld.s | grep "0x"
.quad   0x400921fb54442d18,0x3ca1a62633145c06
dclarke@nix:~/pgm/C/ieee754$

So the default is the IBM type. No idea on the libquadmath yet.
I should have more info on this issue in another 16 or 20 hours.

Dennis






Re: no quadmath.h on ppc64 ?

2018-04-16 Thread Dennis Clarke


>
> Working on it :
>
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=895452
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82686


https://gcc.gnu.org/ml/gcc-testresults/2018-04/msg01351.html


nix$ cat pq.c

#define _XOPEN_SOURCE 600
/* #include  */
#include 
#include 
#include 
#include 
#include 

#define BUFLEN 128
intmain ( int argc, char*argv[])
{
int n, precision = FLT128_DIG + 2;
char buf[BUFLEN];
char *obuf;
size_t buf_len = (size_t) BUFLEN;
__float128 pi_quad = M_PIq;
__float128 my_pi_quad;

/* long double pi_ld = 3.1415926535897932384626433832795028841971L;*/
/* __float128 pi_quad = M_PIq; */

printf ( "\n 3.1415926535897932384626433832795028841971\n" );

n = quadmath_snprintf ( buf, buf_len, "%Qg", pi_quad );

if ( (size_t)n < buf_len ) {
printf ( "%s\n", buf );
}

n = quadmath_snprintf ( NULL, 0, "%+-#46.*Qe", precision, pi_quad );

if ( n > -1 ) {
obuf = calloc ( (size_t) (n+1), (size_t)sizeof(unsigned char));
if ( obuf ) {
n = quadmath_snprintf ( obuf, (n+1), "%+-#46.*Qe", 
precision, pi_quad );

printf ( "%s\n", obuf );
printf ( "also n = %i\n", n );
} else {
fprintf ( stderr, "ERROR : well that sucks.\n" );
return ( EXIT_FAILURE );
}
free ( obuf );
}
printf ( "\n\n&pi_quad addr %p is :\n", &pi_quad );

my_pi_quad = strtoflt128 ( 
"3.1415926535897932384626433832795028841971", NULL );

n = quadmath_snprintf ( NULL, 0, "%+-#46.*Qe", precision, my_pi_quad );
if ( n > -1 ) {
obuf = calloc ( (size_t) (n+1), (size_t)sizeof(unsigned char));
if ( obuf ) {
n = quadmath_snprintf ( obuf, (n+1), "%+-#46.*Qe", 
precision, my_pi_quad );

printf ( "%s\n", obuf );
printf ( "also n = %i\n", n );
} else {
fprintf ( stderr, "ERROR : well that really sucks.\n" );
return ( EXIT_FAILURE );
}
free ( obuf );
}

printf ( "\n\n&my_pi_quad addr %p is :\n", &my_pi_quad );

return ( EXIT_SUCCESS );

}

nix$ grep "FLT128_DIG" 
/usr/local/gcc7/lib/gcc/powerpc64-unknown-linux-gnu/7.3.0/include/float.h

#undef FLT128_DIG
#define FLT128_DIG  __FLT128_DIG__
nix$

nix$ /usr/local/gcc7/bin/gcc -v -m64 -g -S -o pq.s pq.c
Using built-in specs.
COLLECT_GCC=/usr/local/gcc7/bin/gcc
Target: powerpc64-unknown-linux-gnu
Configured with: ../gcc-7.3.0/configure 
--build=powerpc64-unknown-linux-gnu --target=powerpc64-unknown-linux-gnu 
--host=powerpc64-unknown-linux-gnu --prefix=/usr/local/gcc7 
--disable-nls --enable-threads=posix --enable-shared --enable-bootstrap 
--enable-libstdcxx-debug --enable-libstdcxx-time=yes 
--enable-__cxa_atexit --with-system-zlib --disable-objc-gc 
--with-long-double-128 --enable-stage1-languages=c,c++ 
--enable-stage1-checking=misc --with-as=/usr/local/bin/as 
--with-ld=/usr/local/bin/ld 
--enable-languages=c,c++,fortran,go,lto,objc,obj-c++ 
--with-pkgversion='genunix Mon Apr 16 00:04:56 GMT 2018'

Thread model: posix
gcc version 7.3.0 (genunix Mon Apr 16 00:04:56 GMT 2018)
COLLECT_GCC_OPTIONS='-v' '-m64' '-g' '-S' '-o' 'pq.s'
 /usr/local/gcc7/libexec/gcc/powerpc64-unknown-linux-gnu/7.3.0/cc1 
-quiet -v -imultiarch powerpc64-linux-gnu -D__unix__ -D__gnu_linux__ 
-D__linux__ -Dunix -D__unix -Dlinux -D__linux -Asystem=linux 
-Asystem=unix -Asystem=posix pq.c -quiet -dumpbase pq.c -m64 
-auxbase-strip pq.s -g -version -o pq.s
GNU C11 (genunix Mon Apr 16 00:04:56 GMT 2018) version 7.3.0 
(powerpc64-unknown-linux-gnu)
compiled by GNU C version 7.3.0, GMP version 6.1.2, MPFR 
version 4.0.1, MPC version 1.1.0, isl version isl-0.19-GMP


GGC heuristics: --param ggc-min-expand=100 --param ggc-min-heapsize=131072
ignoring nonexistent directory "/usr/local/include/powerpc64-linux-gnu"
ignoring nonexistent directory 
"/usr/local/gcc7/lib/gcc/powerpc64-unknown-linux-gnu/7.3.0/../../../../powerpc64-unknown-linux-gnu/include"

#include "..." search starts here:
#include <...> search starts here:
 /usr/local/gcc7/lib/gcc/powerpc64-unknown-linux-gnu/7.3.0/include
 /usr/local/include
 /usr/local/gcc7/include
 /usr/local/gcc7/lib/gcc/powerpc64-unknown-linux-gnu/7.3.0/include-fixed
 /usr/include/powerpc64-linux-gnu
 /usr/include
End of search list.
GNU C11 (genunix Mon Apr 16 00:04:56 GMT 2018) version 7.3.0 
(powerpc64-unknown-linux-gnu)
compiled by GNU C version 7.3.0, GMP version 6.1.2, MPFR 
version 4.0.1, MPC version 1.1.0, isl version isl-0.19-GMP


GGC heuristics: --param ggc-min-expand=100 --param ggc-min-heapsize=131072
Compiler executable checksum: 4128dcdbf8336d61e21fa946bc1dc074
pq.c: In function 'main':
pq.c:13:24: error: 'FLT128_DIG' undeclared (first use in this function); 
did you mean 'FLT_DIG'?

 int n, precision = FLT128_DIG + 2;
^~
FLT_DIG
pq.c:13:24: note: each undeclared identifier is reported only once for 
each f