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 760309 - updated to build against clutter-1.25.3
updated to build against clutter-1.25.3
Status: RESOLVED INVALID
Product: cluttermm
Classification: Other
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: cluttermm-maint
cluttermm-maint
Depends on:
Blocks:
 
 
Reported: 2016-01-08 09:21 UTC by Ian Martin
Modified: 2016-08-20 23:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
updates configure.ac to use c++11 and reverts version bump (1.77 KB, patch)
2016-01-08 09:21 UTC, Ian Martin
none Details | Review
updates methods.defs (54.53 KB, patch)
2016-01-08 09:23 UTC, Ian Martin
none Details | Review
this patches the methods where gmmproc misinterprets the parameter types (5.81 KB, patch)
2016-01-08 09:24 UTC, Ian Martin
none Details | Review
regenerated enums.defs file (11.34 KB, patch)
2016-01-08 09:25 UTC, Ian Martin
none Details | Review
regnerated signals.defs file (36.36 KB, patch)
2016-01-08 09:26 UTC, Ian Martin
none Details | Review
removes deprecated API blocking the build from actor.hg (1.74 KB, patch)
2016-01-08 09:27 UTC, Ian Martin
none Details | Review
Corrects the usage of BinAlignment enum in BinLayout (1.14 KB, patch)
2016-01-08 09:28 UTC, Ian Martin
none Details | Review
Build blocks due to CairoContext being used as a parameter instead of cairo_t; manually altered. (701 bytes, patch)
2016-01-08 09:29 UTC, Ian Martin
none Details | Review
adds group.hg to filelist.am (606 bytes, patch)
2016-01-08 09:29 UTC, Ian Martin
none Details | Review
actor.h now required as #include for InputDevice. (645 bytes, patch)
2016-01-08 09:30 UTC, Ian Martin
none Details | Review
Removes reference to (deprecated) Clutter::Alpha (1013 bytes, patch)
2016-01-08 09:31 UTC, Ian Martin
none Details | Review
Path parameters are now int, not float. (1.77 KB, patch)
2016-01-08 09:32 UTC, Ian Martin
none Details | Review
Removes reference to (deprecated) Clutter::Content. (673 bytes, patch)
2016-01-08 09:32 UTC, Ian Martin
none Details | Review

Description Ian Martin 2016-01-08 09:21:48 UTC
Created attachment 318452 [details] [review]
updates configure.ac to use c++11 and reverts version bump

This series of patches are the minimum to allow building cluttermm against clutter-1.25.3
Comment 1 Ian Martin 2016-01-08 09:23:37 UTC
Created attachment 318453 [details] [review]
updates methods.defs
Comment 2 Ian Martin 2016-01-08 09:24:48 UTC
Created attachment 318454 [details] [review]
this patches the methods where gmmproc misinterprets the parameter types
Comment 3 Ian Martin 2016-01-08 09:25:30 UTC
Created attachment 318455 [details] [review]
regenerated enums.defs file
Comment 4 Ian Martin 2016-01-08 09:26:02 UTC
Created attachment 318456 [details] [review]
regnerated signals.defs file
Comment 5 Ian Martin 2016-01-08 09:27:03 UTC
Created attachment 318457 [details] [review]
removes deprecated API blocking the build from actor.hg
Comment 6 Ian Martin 2016-01-08 09:28:20 UTC
Created attachment 318458 [details] [review]
Corrects the usage of BinAlignment enum in BinLayout
Comment 7 Ian Martin 2016-01-08 09:29:19 UTC
Created attachment 318459 [details] [review]
Build blocks due to CairoContext being used as a parameter instead of cairo_t; manually altered.
Comment 8 Ian Martin 2016-01-08 09:29:54 UTC
Created attachment 318460 [details] [review]
adds group.hg to filelist.am
Comment 9 Ian Martin 2016-01-08 09:30:48 UTC
Created attachment 318461 [details] [review]
actor.h now required as #include for InputDevice.
Comment 10 Ian Martin 2016-01-08 09:31:38 UTC
Created attachment 318462 [details] [review]
Removes reference to (deprecated) Clutter::Alpha
Comment 11 Ian Martin 2016-01-08 09:32:21 UTC
Created attachment 318463 [details] [review]
Path parameters are now int, not float.
Comment 12 Ian Martin 2016-01-08 09:32:57 UTC
Created attachment 318464 [details] [review]
Removes reference to (deprecated) Clutter::Content.
Comment 13 Murray Cumming 2016-01-12 20:18:45 UTC
Is clutter-2.0 not worth worrying about at the moment then?
Comment 14 Murray Cumming 2016-01-12 20:20:42 UTC
I wonder if it would be easier to patch the cluttermm-1.24 branch, which can then become cluttermm-1.26 (or master again).
Comment 15 Ian Martin 2016-01-12 21:19:17 UTC
My understanding (which may be poor) is that he clutter devs (especially Emmanuele Bassi) are working on GSK - GTK+ Scene-graph Kit, which will integrate a Clutter API into a GTK DrawingArea instead of having 2 different scene-graph APIs.  However, they've been working on it for about 4 years now, and it still doesn't appear much closer than it was when I was working on cluttermm a couple of years back.  Information on progress is sparse; there's an excellent LWN report outlining the reasons for moving past clutter from GUADEC 2014 at http://old.lwn.net/Articles/608501/

Having said that, the indications are that there won't be a 2.0 clutter release unless GSK proves too difficult; there's a pretty compelling argument for making sure it isn't too difficult, so I believe we'd be best to forget 2.0 until it actually happens.

Patching to a 1.24 branch makes sense.
Comment 16 Ian Martin 2016-02-09 02:26:45 UTC
Hi,
Anything I can do to get this accepted?  I'm currently working my way through the tutorial updating it, but I'd at least like to have a buildable and working cluttermm branch in git.

I'm also thinking it makes sense to merge the tutorial into the documentation built with cluttermm; it ties the documentation to a version if nothing else.  It shouldn't be too difficult given other projects have managed it( the clutter cookbook is build as part of clutter).

Ian.
Comment 17 Murray Cumming 2016-02-09 08:19:59 UTC
(In reply to Ian Martin from comment #16)
> Hi,
> Anything I can do to get this accepted?

Sorry, I had understood that you would instead make a patch against the cluttermm-1.24 branch, which presumably should need far less work.

We might want to make make that master again, leaving any cluttermm-2.0 work in a branch, in case it's ever useful. But that's a separate issue.
Comment 18 Ian Martin 2016-02-09 13:34:41 UTC
OK, sorry, my bad.  I was working off the gnome repo which doesn't contain that branch.  I'll change to github.
Comment 19 Murray Cumming 2016-02-09 13:41:00 UTC
What github repo. GNOME's git is mirrored on github, but we don't use it for anything.

The cluttermm-1.24 branch is here: https://git.gnome.org/browse/cluttermm/log/?h=cluttermm-1-24
Comment 20 Ian Martin 2016-02-09 19:33:40 UTC
I got it now.  Apologies, that was a 2am response.  I've figured out my problem since - jhbuild modulesets point to the 1.18 branch.
Comment 21 Ian Martin 2016-02-12 20:27:24 UTC
I've opened two new bugs.
https://bugzilla.gnome.org/show_bug.cgi?id=761847 : updates to cluttermm.
https://bugzilla.gnome.org/show_bug.cgi?id=761971 : tutorial updates.
Comment 22 Murray Cumming 2016-08-20 23:11:23 UTC
cluttermm-1.0 work is now back on git master, as we now know that clutter-2.0 won't happen.

None of these bugs seem to make sense now. But please open a new bug report if you have any changes. It would be nice to really remove use of deprecated Clutter API and add the necessary replacement API.