bugGnash - The GNU Flash player - Bugs: bug #43198, Build process is stopped in...

 
 

bug #43198: Build process is stopped in ASHandlers.cpp

Submitter:  Frederic Bezies <fredbezies>
Submitted:  Thu 11 Sep 2014 03:32:09 PM UTC
   
 
Category:  build Severity:  3 - Normal
Release:  None Status:  Fixed
Privacy:  Public Assigned to:  bjacques
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Sun 01 Feb 2015 09:48:07 PM UTC, comment #10: 
Anonymous
Sun 01 Feb 2015 07:48:18 PM UTC, comment #9: 

Should be fixed in 4a9aa74.

Bastiaan Jacques <bjacques>
Group Member
Sun 01 Feb 2015 05:40:40 PM UTC, comment #8: 

fails: boost-devel-1.57.0-2.fc21.x86_64 + gcc-c++-4.9.2-1.fc21.x86_64

Bastiaan Jacques <bjacques>
Group Member
Sun 01 Feb 2015 04:16:26 PM UTC, comment #7: 

works: boost-1.55.0-8.fc21.x86_64 + gcc-c++-4.9.2-1.fc21.x86_64

So maybe it's just boost >= 1.56?

Bastiaan Jacques <bjacques>
Group Member
Sun 01 Feb 2015 03:22:50 PM UTC, comment #6: 

It seems to be a problem with boost > 1.54.0:

on openSUSE:
working: boost 1.54.0 + gcc 4.8.3+r212056 (openSUSE_13.2)
failing: boost 1.56.0 + gcc 4.8.3+r218481 (Factory)

See:
https://pmbs.links2linux.de/package/show/Essentials/gnash

Anonymous
Wed 15 Oct 2014 06:12:34 PM UTC, comment #5: 

I'm actually surprised abot the problem you initially have:
cannot convert 'const boost::optional<gnash::SWFRect>' to 'bool'

Converting boost::optional to bool is one of the main goals for that class, so could it be a problem with GCC 4.9.1-2 and boost (which version ?) ?

Sandro Santilli <strk>
Group Member
Wed 15 Oct 2014 04:46:55 PM UTC, comment #4: 

Okay, I did manage to get it to build with the change, but I'm using gcc 4.9.1-2. If anyone else has the same problem, they can try the workaround.

It's odd that the comparison would cause a failure, though. But I'm not C savvy enough to try and fix it without much trial and error.

Tim Van den Langenbergh <tmtvl>
Wed 15 Oct 2014 04:35:24 PM UTC, comment #3: 

My failure was actually due to the change, so I reverted it as of
commit d821b4c9abbbabf03491a2cb0a02ebeeda4d5360

Sandro Santilli <strk>
Group Member
Wed 15 Oct 2014 04:26:37 PM UTC, comment #2: 

I've pushed the hasBounds() change with commit ace2e81f36f6c4d266e0c63a1c95cc23d3d45567

But build still fails for me later with GCC 4.8.2:


In file included from /usr/include/boost/optional/optional.hpp:36:0,
                 from /usr/include/boost/optional.hpp:15,
                 from /usr/include/boost/format/internals.hpp:20,
                 from /usr/include/boost/format.hpp:38,
                 from /usr/include/boost/math/policies/error_handling.hpp:31,
                 from /usr/include/boost/math/special_functions/gamma.hpp:21,
                 from /usr/include/boost/math/special_functions/detail/bessel_jy.hpp:14,
                 from /usr/include/boost/math/special_functions/bessel.hpp:18,
                 from /usr/include/boost/math/special_functions/airy.hpp:10,
                 from /usr/include/boost/math/special_functions.hpp:15,
                 from /usr/include/boost/random/generate_canonical.hpp:22,
                 from /usr/include/boost/random.hpp:52,
                 from /usr/src/gnash/gnash-head/libcore/vm/ASHandlers.cpp:29:
/usr/include/boost/utility/compare_pointees.hpp: In instantiation of 'bool boost::equal_pointees(const OptionalPointee&, const OptionalPointee&) [with OptionalPointee = boost::optional<gnash::SWFRect>]':
/usr/include/boost/optional/optional.hpp:894:41:   required from 'bool boost::operator==(boost::none_t, const boost::optional<T>&) [with T = gnash::SWFRect; boost::none_t = int boost::detail::none_helper::*]'
/usr/include/boost/optional/optional.hpp:904:15:   required from 'bool boost::operator!=(boost::none_t, const boost::optional<T>&) [with T = gnash::SWFRect; boost::none_t = int boost::detail::none_helper::*]'
/usr/src/gnash/gnash-head/libcore/DragState.h:65:22:   required from here
/usr/include/boost/utility/compare_pointees.hpp:32:52: error: no match for 'operator==' (operand types are 'const gnash::SWFRect' and 'const gnash::SWFRect')
   return (!x) != (!y) ? false : ( !x ? true : (*x) == (*y) ) ;
                                                    ^

Sandro Santilli <strk>
Group Member
Wed 15 Oct 2014 03:51:59 PM UTC, comment #1: 

A possible workaround could be to edit libcore/DragState.h:64:
bool hasBounds() const {
    -return (_bounds);
    +return (NULL != _bounds);
}

As this returns a boolean instead of an object.

Tim Van den Langenbergh <tmtvl>
Thu 11 Sep 2014 03:32:09 PM UTC, original submission:  

I was trying to build gnash from git sources after bug #42574 was declared as fixed, but building process crashed.

Here is the log I got right before crash :

" CXXLD    libgnashparser.la
make[4]: Leaving directory '/home/fred/gnash-trunk-git/src/gnash/libcore/parser'
Making all in vm
make[4]: Entering directory '/home/fred/gnash-trunk-git/src/gnash/libcore/vm'
  CXX      ASHandlers.lo
In file included from ASHandlers.cpp:46:0:
../../libcore/DragState.h: In member function 'bool gnash::DragState::hasBounds() const':
../../libcore/DragState.h:65:24: error: cannot convert 'const boost::optional<gnash::SWFRect>' to 'bool' in return
         return (_bounds);
                        ^
Makefile:757: recipe for target 'ASHandlers.lo' failed
make[4]: * [ASHandlers.lo] Error 1
make[4]: Leaving directory '/home/fred/gnash-trunk-git/src/gnash/libcore/vm'
Makefile:2381: recipe for target 'all-recursive' failed
make[3]: * [all-recursive] Error 1
make[3]: Leaving directory '/home/fred/gnash-trunk-git/src/gnash/libcore'
Makefile:1246: recipe for target 'all' failed
make[2]: * [all] Error 2"

Boost version : 1.56
Gcc : 4.9.1

Frederic Bezies <fredbezies>

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by bjacques (Posted a comment)
  • -email is unavailable- added by strk (Posted a comment)
  • -email is unavailable- added by tmtvl (Posted a comment)
  • -email is unavailable- added by kozross (Interested in resolution)
  • -email is unavailable- added by fredbezies (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-02-01 bjacques StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2015-02-01 bjacques StatusNone Ready For Test
    2015-02-01 bjacques Assigned toNone bjacques
    2014-10-15 strk Severity5 - Blocker 3 - Normal
    2014-10-15 strk CategoryNone build
        Severity3 - Normal 5 - Blocker
    2014-09-11 kozross Carbon-Copy- Added kozross

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code