bugGnash - The GNU Flash player - Bugs: bug #20847, boost-thread not checked by...

 
 

bug #20847: boost-thread not checked by configure script

Submitted by:  Klaus Rechert <krechert>
Submitted on:  Tue 21 Aug 2007 12:48:02 PM UTC  
 
Category: buildSeverity: 3 - Normal
Release: NoneStatus: Fixed
Privacy: PublicAssigned to: Rob Savoye <rsavoye>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Thu 24 Jan 2008 10:52:37 AM UTC, comment #16:

assume fixed

Sandro Santilli <strk>
Project Member
Mon 07 Jan 2008 08:47:59 PM UTC, comment #15:

Any news on this ?
Can anyone test ?

Sandro Santilli <strk>
Project Member
Fri 05 Oct 2007 06:40:10 PM UTC, comment #14:

I rewrote most of boost.m4 to work better. Due to the complex naming scheme used by the Boost project, finding both the directory and files names is convoluted. Now we have a list of headers and libraries that all must be found individually before configure is happy. Even for the --with-boost-* options, the existence of these files is confirmed before deciding the configuration process was successful. I will probably have to now improve some of the errors messages, but you should have much better luck now with cvs HEAD.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Thu 04 Oct 2007 02:48:40 PM UTC, comment #13:

When trying to build gnash CVS head:
without libboost-thread-dev installed
and with libboost-thread installed
./configure pass
and make failed with the following message
In file included from extension.cpp:42:
log.h:39:35: error: boost/thread/thread.hpp: No such file or directory
log.h:40:34: error: boost/thread/mutex.hpp: No such file or directory

Johan Euphrosine <proppy>
Thu 04 Oct 2007 02:33:31 PM UTC, comment #12:

The bug is still here, if you:

apt-get remove libboost-thread-dev
and had
apt-get install libboost-thread

There is no configure error.

Johan Euphrosine <proppy>
Thu 04 Oct 2007 02:19:49 PM UTC, comment #11:

It's seems to be fixed in cvs HEAD:

In a bulk feisty chroot environment:
I get the following error:

ERROR: No BOOST development package installed!
Install it from http://www.boost.org
or .deb users: apt-get install libboost-dev libboost-thread-dev
or .rpm users: yum install boost-devel

If I apt-get install only libboost-dev, I still get the following error:

ERROR: No BOOST development package installed!
Install it from http://www.boost.org
or .deb users: apt-get install libboost-dev libboost-thread-dev
or .rpm users: yum install boost-devel

After apt-get install libboost-thread-dev, the error message go away.

Johan Euphrosine <proppy>
Mon 24 Sep 2007 10:44:42 AM UTC, comment #10:

Any news on this ? can it be closed ?

Sandro Santilli <strk>
Project Member
Wed 05 Sep 2007 09:30:38 AM UTC, comment #9:

next week I'll be back on this box and will check if it works...

Anonymous
Mon 03 Sep 2007 01:44:55 PM UTC, comment #8:

I'm still curious why it fails, as the current boost test works for me on all systems. G++ 4.0.0 was a truly ugly release, and I'd like to know if a compiler upgrade would fix this. This extra test is unnecessary, and causes problems when cross compiling.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Mon 03 Sep 2007 10:35:18 AM UTC, comment #7:

@Rob: g++-4.0.0, Darwin.

The problem is not caused by a particular compiler version. The tests generated by autoconf call first "g++ $CXXFLAGS" and then "gcc-E $CPPFLAGS".

Unless $CPPFLAGS are set properly (so that they include the checked boost path, which is now in $CXXFLAGS) the preprocessor check "checking boost/thread.hpp presence...", which does a "#include <boost/thread.h>" will fail.

Regards,

Anonymous
Mon 03 Sep 2007 06:12:59 AM UTC, comment #6:

Using AC_CHECK_HEADER ensures the header is actually usable
with the CXXFLAGS temporarly set to the value it will get
while building libraries using boost.

I'd reccommend keeping the check in, at least when not cross-compiling.

Sandro Santilli <strk>
Project Member
Sun 02 Sep 2007 10:45:07 PM UTC, comment #5:

thread.hpp is already looked for, so we shouldn't need the AC_CHECK_HEADERS(boost/tread.hpp) Which compiler version are you running ? You should never see that WARNING.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Sat 01 Sep 2007 09:39:44 PM UTC, comment #4:

You need to set CPPFLAGS=CXXFLAGS, otherwise half of the test will fail:

checking for the Boost Version... 1_34_1
checking for Boost libraries... -L/Users/holger/gnash/build//lib -lboost_date_time-gcc40-sd -lboost_thread-gcc40-mt-sd
checking boost/thread.hpp usability... yes
checking boost/thread.hpp presence... no
configure: WARNING: boost/thread.hpp: accepted by the compiler, rejected by the preprocessor!
configure: WARNING: boost/thread.hpp: proceeding with the compiler's result
checking for boost/thread.hpp... yes

Anonymous
Thu 30 Aug 2007 12:20:46 AM UTC, comment #3:

I'd like to hear from Klaus first.

Also, I'm pretty sure I tested after I fixed this (at the time I set to Ready for Test). ... will need to retest again myself ...

Sandro Santilli <strk>
Project Member
Thu 30 Aug 2007 12:08:28 AM UTC, comment #2:

Fixed this bug so it set CXXFLAGS when checking. I think you can resolve it now.

Markus Gothe <nihilus>
Project Member
Wed 22 Aug 2007 08:58:43 AM UTC, comment #1:

It's the header not being checked, the libs seems to be checked.
I guess we want to use AC_CHECK_HEADERS here..

Sandro Santilli <strk>
Project Member
Tue 21 Aug 2007 12:48:02 PM UTC, original submission:

./GC.h:30:29: error: boost/thread.hpp: No such file or directory

CVS HEAD from 21.08.2007 on ubuntu stable

Klaus Rechert <krechert>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by strk
  • -unavailable- added by proppy (Posted a comment)
  • -unavailable- added by rsavoye (Posted a comment)
  • -unavailable- added by nihilus (Posted a comment)
  • -unavailable- added by strk (Updated the item)
  • -unavailable- added by krechert (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 12 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 24 Jan 2008 10:52:37 AM UTCstrkStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Mon 07 Jan 2008 08:48:00 PM UTCstrkCarbon-Copy-=>Added bwy
    Fri 05 Oct 2007 06:40:10 PM UTCrsavoyeStatusIn Progress=>Ready For Test
    Thu 04 Oct 2007 04:14:53 PM UTCstrkStatusNeed Info=>In Progress
      Assigned tostrk=>rsavoye
    Sun 02 Sep 2007 10:45:07 PM UTCrsavoyeSeverity5 - Blocker=>3 - Normal
      StatusReady For Test=>Need Info
    Wed 22 Aug 2007 10:52:57 AM UTCstrkStatusNone=>Ready For Test
      Assigned torsavoye=>strk
    Wed 22 Aug 2007 08:58:43 AM UTCstrkAssigned toNone=>rsavoye
    Tue 21 Aug 2007 12:53:02 PM UTCstrkSeverity3 - Normal=>5 - Blocker

    Back to the top


    Powered by Savane 3.1-cleanup1