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 770542 - Build fixes
Build fixes
Status: RESOLVED FIXED
Product: gnome-online-accounts
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: GNOME Online Accounts maintainer(s)
GNOME Online Accounts maintainer(s)
Depends on:
Blocks: 783204
 
 
Reported: 2016-08-29 10:37 UTC by Bastien Nocera
Modified: 2017-05-29 16:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
build: Fix running autogen.sh outside $srcdir (687 bytes, patch)
2016-08-29 10:37 UTC, Bastien Nocera
committed Details | Review
build: Disable Ubuntu Online Accounts support in tp-widgets (1.20 KB, patch)
2016-08-29 10:37 UTC, Bastien Nocera
committed Details | Review

Description Bastien Nocera 2016-08-29 10:37:10 UTC
Still not enough to get it to work in jhbuild though, the PKG_CONFIG_PATH
is not set to an absolute path.
Comment 1 Bastien Nocera 2016-08-29 10:37:15 UTC
Created attachment 334353 [details] [review]
build: Fix running autogen.sh outside $srcdir

And use an absolute path, instead of a path relative to the sources.
Comment 2 Bastien Nocera 2016-08-29 10:37:21 UTC
Created attachment 334354 [details] [review]
build: Disable Ubuntu Online Accounts support in tp-widgets

Our telepathy-account-widgets submodule doesn't need support for the
Ubuntu Online Accounts, so disable them instead of erroring out.
Comment 4 Debarshi Ray 2017-05-29 16:01:10 UTC
Review of attachment 334354 [details] [review]:

Thanks, Bastien. Looks good.
Comment 5 Debarshi Ray 2017-05-29 16:14:10 UTC
Review of attachment 334353 [details] [review]:

It doesn't break 'make distcheck'. Let's push it and see if Continuous is happy.
Comment 6 Debarshi Ray 2017-05-29 16:15:17 UTC
(In reply to Bastien Nocera from comment #0)
> Still not enough to get it to work in jhbuild though, the PKG_CONFIG_PATH
> is not set to an absolute path.

I need to figure out how jhbuild and Continuous differ in their 'srcdir != builddir' setups. It's been on my list for a while. Haven't gotten to it, yet.
Comment 7 Emmanuele Bassi (:ebassi) 2017-05-29 16:20:02 UTC
The main difference is that Continuous will call autogen.sh inside srcdir, then call configure in the builddir — whereas jhbuild calls autogen.sh in the builddir.
Comment 8 Debarshi Ray 2017-05-29 16:47:53 UTC
(In reply to Bastien Nocera from comment #0)
> Still not enough to get it to work in jhbuild though, the PKG_CONFIG_PATH
> is not set to an absolute path.

It looks like it already works. Did you 'git clean -fdx' both the gnome-online-accounts and telepathy-account-widgets directories before trying again?

(In reply to Emmanuele Bassi (:ebassi) from comment #7)
> The main difference is that Continuous will call autogen.sh inside srcdir,
> then call configure in the builddir — whereas jhbuild calls autogen.sh in
> the builddir.

I see.

I did a 'git clean -fdx' in both the gnome-online-accounts and telepathy-accounts-widget directories, created a build directory in a different part of the filesystem, ran autogen.sh --enable-gtk-doc && make from there and it worked. I can even 'make distcheck' from this build directory.

config.log says:
PKG_CONFIG_PATH='/opt/lib/pkgconfig:/opt/lib64/pkgconfig:/opt/share/pkgconfig:telepathy-account-widgets/'

... and telepathy-account-widgets-uninstalled.pc is instantiated inside $builddir/telepathy-account-widgets.