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 720315 - can't build on Ubuntu 14.04: add_event_listener conflicts with method of the same name
can't build on Ubuntu 14.04: add_event_listener conflicts with method of the ...
Status: RESOLVED FIXED
Product: geary
Classification: Other
Component: build
master
Other Linux
: Normal normal
: 0.6.0
Assigned To: Jim Nelson
Geary Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-12-12 11:12 UTC by Adam Dingle
Modified: 2013-12-18 19:25 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Adam Dingle 2013-12-12 11:12:19 UTC
I'm trying to build Geary from git master on Ubuntu 14.04, which has GTK 3.10.6 and WebKitGTK 2.3.2.  The build fails with this:

WebKit-3.0.gir:7763.7-7763.43: warning: Virtual method `WebKit.DOM.EventTarget.add_event_listener' conflicts with method of the same name
      <virtual-method name="add_event_listener">
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Comment 1 Thomas Moschny 2013-12-15 10:41:33 UTC
I see the same issue with the Geary 0.5.0 snapshot on current Fedor rawhide, using webkitgtk3 2.3.2 and vala 0.22.1.
Comment 2 Adam Dingle 2013-12-16 14:02:42 UTC
I'm still seeing this, so I'm now running the Ubuntu Geary binary rather than following the one from git master.

Jim and/or Charles, do you think you'll be able to look at this this week?
Comment 3 Jim Nelson 2013-12-16 19:16:33 UTC
I started work on this Friday.  I hope to have this done in the next day or two.
Comment 4 Adam Dingle 2013-12-16 19:17:11 UTC
Great to hear - thanks!
Comment 5 Jim Nelson 2013-12-17 22:41:05 UTC
The .gir warning is the least of our worries.  WebKitGTK has broken the API with this release:

https://bugs.webkit.org/show_bug.cgi?id=77835

Although it's an improvement that they've moved to GClosures for event listening, getting Geary ported to this change is a significant amount of work.  I'm digging into this right now.
Comment 6 Jim Nelson 2013-12-18 01:40:46 UTC
It turns out they were using .gir annotations to hide the old API, so with some VAPI trickery we can still use the old-style GCallback listeners.
Comment 7 Jim Nelson 2013-12-18 03:03:04 UTC
Fixed in commit:26982f9bc
Comment 8 Chris Johnston 2013-12-18 03:04:29 UTC
Is there any chance that geary could be published in the PPA for trusty with this fix?
Comment 9 Jim Nelson 2013-12-18 03:09:29 UTC
We only publish stable releases to our main PPA.  We do have a Daily Build PPA that runs nightly compiles:

https://launchpad.net/~yorba/+archive/daily-builds/+packages

However, lately we've been getting "Dependency wait" notifications for Trusty.  We usually see these at the start of a new cycle.  I'll ping Ubuntu and see if there's some way to get this resolved soon.
Comment 10 Chris Johnston 2013-12-18 04:25:48 UTC
I believe that the issue is that Geary requires libwebkitgtk-3.0-dev (< 2.1.0) but the version in trusty is 2.3.2-1ubuntu5. Geary would need to have its depends updated.
Comment 11 Jim Nelson 2013-12-18 19:25:46 UTC
Good catch -- I overlooked that.  I updated the debian/control in master.  Hopefully that solves the problem.