After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 673275 - clutter dependency not tight enough?
clutter dependency not tight enough?
Status: RESOLVED FIXED
Product: sushi
Classification: Core
Component: general
0.4.x
Other Linux
: Normal normal
: ---
Assigned To: Sushi maintainer(s)
Sushi maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-04-01 02:52 UTC by Michael Biebl
Modified: 2012-04-02 01:28 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Michael Biebl 2012-04-01 02:52:11 UTC
Version: 0.4.0.

I currently use clutter 1.8.4 (and cogl 1.8.2), which according to configure.ac should be sufficient.
While sushi 0.4.0 compiled successfully, it simply show anything when I tried to run it. As 0.2.1 worked fine, I did a git bisect, and found the faulty commit:

commit 6603f5b68f69825f8a65c4d3ec3a4d027265aaee
Author: Cosimo Cecchi <cosimoc@gnome.org>
Date:   Mon Mar 19 15:36:34 2012 -0400

    main-window: use a ClutterBoxLayout for the title actor

After reverting this commit, sushi 0.4.0 worked again.

Does this require a more recent version of clutter? In that case, should the minimum version be bumped (it is currently at 1.0.1).
Comment 1 Michael Biebl 2012-04-01 02:53:24 UTC
(In reply to comment #0)
> While sushi 0.4.0 compiled successfully, it simply show anything when I tried
                                                     ^
                                                     didn't show anything
Comment 2 Cosimo Cecchi 2012-04-01 17:50:29 UTC
Yes good catch; this line effectively makes sushi depend on Clutter 1.10, since ClutterActor wasn't a directly instantiatable type before.

+        let hidden = new Clutter.Actor();

Can you bump the minimum required version of Clutter to 1.10.0 on git master? Otherwise I'll get to it before the next release; thanks!
Comment 3 Michael Biebl 2012-04-02 01:28:08 UTC
Pushed