New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
libgdata: add gnome-online-accounts to propagatedBuildInputs #65442
Conversation
I'm suspect that the error
that line being gdata = dependency('libgdata') means that I should probably fix something in It seems the changes in the Here's a diff of the two files --- libgdata-old.pc 2019-07-26 20:48:19.032408179 -0400
+++ libgdata-new.pc 2019-07-26 20:48:44.987912414 -0400
@@ -1,13 +1,14 @@
-prefix=/nix/store/hyakiglm5s4xw24jj001yq51pxglbb3q-libgdata-0.17.9
+prefix=/nix/store/mg16sjmcq5k2nbq63ni9q118a7x4hd5v-libgdata-0.17.10
+libdir=${prefix}/lib
+includedir=/nix/store/icz2m06whc3lwlx1pwgbqpqx9p6irky9-libgdata-0.17.10-dev/include
+
exec_prefix=${prefix}
-libdir=${exec_prefix}/lib
-includedir=${prefix}/include
goa_enabled=true
Name: libgdata
Description: GData client library
-Version: 0.17.9
-Requires: gobject-2.0 glib-2.0 >= 2.44.0 gio-2.0 >= 2.44.0 libxml-2.0 libsoup-2.4 >= 2.42.0 json-glib-1.0 >= 0.15
-Requires.private: gthread-2.0 oauth >= 0.9.4 gcr-base-3 libxml-2.0 goa-1.0 >= 3.8
+Version: 0.17.10
+Requires: gobject-2.0, glib-2.0 >= 2.44.0, gio-2.0 >= 2.44.0, gio-unix-2.0, json-glib-1.0 >= 0.15, libxml-2.0, libsoup-2.4 >= 2.42.0, goa-1.0 >= 3.8
+Requires.private: gthread-2.0, oauth >= 0.9.4, gcr-base-3
Libs: -L${libdir} -lgdata
Cflags: -I${includedir}/libgdata It seems at some point goa was intentionally private I guess this should be done again upstream for the generated |
Ahh that was an intentional change We should add |
Oh yeah, that's a better approach. I'll start testing that; note that it might take me a while to complete test builds of all the reverse dependencies. |
90fe75d
to
6da2fb6
Compare
Updated to export |
LGTM, could you fetchpatch the commits off master from these PR's ?
This is something I forgot to do initially. |
Required since `libgdata` now uses Meson. See also: NixOS#64295
6da2fb6
to
30c81c8
Compare
I'll take a look. |
Added the Meson patches. They seem to work ok and all reverse-dependencies build. EDIT: Oops, hang on, I used patches from the merge requests. Let me change it to use the master commits in case the MRs disappear. |
13a4734
to
9a2bddf
Compare
Fixed, the Meson fixes now use the master commits. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much ✨
Shotwell now builds locally, everything committed has been done correctly.
Motivation for this change
libgdata
's new Meson config listsgnome-online-accounts
as a dependency. Downstream apps which use Meson (likeshotwell
) refuse to build withoutgnome-online-accounts
in the build env.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)