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 784242 - gnome-contacts fails to build with vala 0.36
gnome-contacts fails to build with vala 0.36
Status: RESOLVED FIXED
Product: gnome-contacts
Classification: Core
Component: general
3.22.x
Other Linux
: Normal critical
: ---
Assigned To: GNOME Contacts maintainer(s)
GNOME Contacts maintainer(s)
Depends on: 784240
Blocks:
 
 
Reported: 2017-06-27 10:48 UTC by Jeremy Bicha
Modified: 2017-07-17 23:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
build log (274.66 KB, text/plain)
2017-06-27 10:48 UTC, Jeremy Bicha
  Details
build: Bump folks requirement (744 bytes, patch)
2017-07-05 11:24 UTC, Bastien Nocera
none Details | Review

Description Jeremy Bicha 2017-06-27 10:48:22 UTC
Created attachment 354554 [details]
build log

gnome-contacts 3.22.1 fails to build from source on Ubuntu 17.10 Alpha with vala 0.36. (vala 0.36 was released alongside GNOME 3.22)

I cherry-picked https://git.gnome.org/browse/gnome-contacts/commit/?id=2ba6e38 but that wasn't sufficient. I'm attaching a build log made with that patch applied.
Comment 1 Niels De Graef 2017-06-27 10:57:59 UTC
Indeed, seems like it's due to Folks' VAPI's not being handled well by valac 0.36 (mentioned by bug 784240).

I'm keeping this bug open in case we might want to look for an intermediate solution ourselves (e.g. temporarily holding a patched version of the vapi's ourselves).
Comment 2 Jeremy Bicha 2017-06-27 20:11:06 UTC
I was able to build gnome-contacts successfully after patching folks with the folks commit
https://git.gnome.org/browse/folks/commit/?id=ddc7e71c5d0d113a1b46e2a89986f5a9c3ab8374

Then I needed to patch gnome-contacts with these two commits:
https://git.gnome.org/browse/gnome-contacts/commit/?id=2ba6e38
https://git.gnome.org/browse/gnome-contacts/commit/?id=7b9d178
Comment 3 Bastien Nocera 2017-07-05 11:24:01 UTC
Created attachment 354914 [details] [review]
build: Bump folks requirement

So we don't hit https://bugzilla.gnome.org/show_bug.cgi?id=784240
Comment 4 Bastien Nocera 2017-07-05 13:04:54 UTC
gnome-contacts master built fine against the new folks release.
Comment 5 Niels De Graef 2017-07-17 23:38:54 UTC
Thanks! Committed as 7a9001b056b4fb1d00375e7b2adeda9b7cf93c90