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 787210 - Allow skipping compilation of examples
Allow skipping compilation of examples
Status: RESOLVED FIXED
Product: libgdata
Classification: Platform
Component: General
0.17.x
Other Linux
: Normal normal
: ---
Assigned To: libgdata-maint
libgdata-maint
Depends on:
Blocks:
 
 
Reported: 2017-09-03 12:58 UTC by Gilles Dartiguelongue
Modified: 2018-02-15 21:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
disable demos build configure switch (1.90 KB, patch)
2017-09-03 12:58 UTC, Gilles Dartiguelongue
needs-work Details | Review

Description Gilles Dartiguelongue 2017-09-03 12:58:29 UTC
Created attachment 359018 [details] [review]
disable demos build configure switch

In Gentoo we want to properly control build dependencies as a rolling release distribution could not work without it. In libgdata, gtk+ presence is detected but not controllable in order to build examples. The attached patch allows disabling examples build while keeping the current default.
Comment 1 Philip Withnall 2017-09-08 15:07:19 UTC
Review of attachment 359018 [details] [review]:

I’d prefer an --enable-gtk+ option which defines an ENABLE_GTK variable to replace the existing HAVE_GTK condition in Makefile.am. There’s no point in disabling compilation of all the non-GTK+ demos.
Comment 2 Philip Withnall 2018-02-14 16:17:58 UTC
Fixed in master with an approach similar to what I suggested in comment #1.

commit 40e3eefae8e2d03d549d0f26862d0e2e0883d6c9 (HEAD -> master, origin/master, origin/HEAD)
Author: Philip Withnall <withnall@endlessm.com>
Date:   Wed Feb 14 16:15:15 2018 +0000

    build: Add --enable-gtk option to conditionalise GTK+ examples
    
    Only build the GTK+ examples if enabled explicitly with --enable-gtk, or
    if --enable-gtk is not specified and GTK+ is available.
    
    Signed-off-by: Philip Withnall <withnall@endlessm.com>
    
    https://bugzilla.gnome.org/show_bug.cgi?id=787210

 Makefile.am  |  3 ++-
 configure.ac | 12 ++++++++++--
 2 files changed, 12 insertions(+), 3 deletions(-)
Comment 3 Gilles Dartiguelongue 2018-02-15 21:17:34 UTC
Yeah my patch was a bit biased as package maintainers don't need demos, examples and whatnot to build by default as it won't be used in the package. We generally ship demos or examples source files only in doc/examples.