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 761713 - Build failure in Continuous
Build failure in Continuous
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: network
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2016-02-08 13:39 UTC by Emmanuele Bassi (:ebassi)
Modified: 2016-02-08 20:50 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Emmanuele Bassi (:ebassi) 2016-02-08 13:39:11 UTC
This commit:

https://git.gnome.org/browse/glib-networking/commit/?id=55c61d1c2de20af86adf711e7b6e10787077420a

Introduces a build failure inside GNOME Continuous:

http://build.gnome.org/continuous/buildmaster/builds/2016/02/08/31/build/log-glib-networking.txt

../../../tls/gnutls/gtlsfiledatabase-gnutls.c:594:3: error: implicit declaration of function 'gnutls_x509_trust_list_verify_crt2' [-Werror=implicit-function-declaration]
   gerr = gnutls_x509_trust_list_verify_crt2 (self->priv->trust_list,
   ^

The function in question was introduced in gnutls 3.3, but:

 * glib-networking still requires 3.0
 * jhbuild requires 3.1
 * continuous uses whatever yocto provides, which is guess is < 3.3

This needs to either be a conditional dependency, or we need to bump up the dependency everywhere.
Comment 1 Emmanuele Bassi (:ebassi) 2016-02-08 13:46:44 UTC
I've reverted the commit in master, as per:

https://mail.gnome.org/archives/desktop-devel-list/2016-January/msg00015.html

as I could not find any responsible party on IRC.

Feel free to undo my revert as soon as we can fix this bug.
Comment 2 Emmanuele Bassi (:ebassi) 2016-02-08 13:50:28 UTC
To clarify: I'm happy to fix jhbuild and continuous to require a newer version of gnutls, but pushing the commit to use a new symbol should come after the dependency bump, to avoid breaking bisection; also, I'd like to keep master building at all times.
Comment 3 Dan Winship 2016-02-08 13:57:59 UTC
I didn't mean to require a new version, and this can be reimplemented using older API.
Comment 4 Dan Winship 2016-02-08 20:50:09 UTC
fixed