[Debian-med-packaging] Bug#625120: clonalframe: FTBFS: src/move_hidden.cpp:169:62: error: taking address of temporary [-fpermissive]

[Andreas Tille]

Hi,

sorry for the late reply to this bug.  I can reproduce the problem on my
side but I'm not finally sure that this is really a problem of clonalframe
or whether it is a bad coincidence with libgsl0-dev.  The line in question
where the problem occures is:

 src/move_hidden.cpp:423:59: error: taking address of temporary [-fpermissive]                                                                                                                                 

 423:            Util::normalize(&(gsl_matrix_row(e,i+1).vector));

So I suspect that there are temporary variables used where they should
not but these are not declared in Move_hidden::makee().  My c++
knowledge ist too limited to track down the problem in a reasonable time
frame and thus I CC debian-mentors and upstream (Xavier please find the
full log of this bug below or at http://bugs.debian.org/625120).

Any help is welcome

         Andreas.


On Mon, May 02, 2011 at 02:30:18PM +0200, Lucas Nussbaum wrote:
> Source: clonalframe
> Version: 1.2-1
> Severity: serious
> Tags: wheezy sid
> User: debian-qa at lists.debian.org
> Usertags: qa-ftbfs-20110502 qa-ftbfs
> Justification: FTBFS on amd64
> 
> Hi,
> 
> During a rebuild of all packages in sid, your package failed to build on
> amd64.
> 
> Relevant part:
> > g++ -c -pipe -W -Wall -O3 -static -DHAVE_INLINE -DGSL_RANGE_CHECK_OFF -Isrc -O2 -Wall -W -D_REENTRANT -DQT_NO_DEBUG -DQT_GUI_LIB -DQT_CORE_LIB -DQT_SHARED -I/usr/share/qt4/mkspecs/linux-g++ -I. -I/usr/include/qt4/QtCore -I/usr/include/qt4/QtGui -I/usr/include/qt4 -Ibuild -o build/move_hidden.o src/move_hidden.cpp
> > src/move_hidden.cpp:25:5: warning: unused parameter 'p' [-Wunused-parameter]
> > src/move_hidden.cpp: In member function 'void wb::Move_hidden::forwardfast(wb::Param*, wb::Tree*)':
> > src/move_hidden.cpp:127:33: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
> > src/move_hidden.cpp:133:41: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
> > src/move_hidden.cpp:137:45: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
> > src/move_hidden.cpp:132:21: warning: unused variable 'top' [-Wunused-variable]
> > src/move_hidden.cpp:132:27: warning: unused variable 'left' [-Wunused-variable]
> > src/move_hidden.cpp:132:34: warning: unused variable 'right' [-Wunused-variable]
> > src/move_hidden.cpp: In member function 'gsl_matrix* wb::Move_hidden::forward(wb::Param*)':
> > src/move_hidden.cpp:169:62: error: taking address of temporary [-fpermissive]
> > src/move_hidden.cpp: In member function 'void wb::Move_hidden::backward(wb::Param*, gsl_matrix*, wb::Tree*)':
> > src/move_hidden.cpp:202:17: warning: unused variable 'd' [-Wunused-variable]
> > src/move_hidden.cpp: In member function 'void wb::Move_hidden::makee(wb::Param*, wb::Tree*)':
> > src/move_hidden.cpp:423:59: error: taking address of temporary [-fpermissive]
> > make[2]: *** [build/move_hidden.o] Error 1
> 
> The full build log is available from:
>    http://people.debian.org/~lucas/logs/2011/05/02/clonalframe_1.2-1_lsid64.buildlog
> 
> A list of current common problems and possible solutions is available at 
> http://wiki.debian.org/qa.debian.org/FTBFS . You're welcome to contribute!
> 
> About the archive rebuild: The rebuild was done on about 50 AMD64 nodes
> of the Grid'5000 platform, using a clean chroot.  Internet was not
> accessible from the build systems.
> 
> -- 
> | Lucas Nussbaum
> | lucas at lucas-nussbaum.net   http://www.lucas-nussbaum.net/ |
> | jabber: lucas at nussbaum.fr             GPG: 1024D/023B3F4F |
> 
> 
> 
> _______________________________________________
> Debian-med-packaging mailing list
> Debian-med-packaging at lists.alioth.debian.org
> http://lists.alioth.debian.org/mailman/listinfo/debian-med-packaging
> 

-- 
http://fam-tille.de