all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Lukas Gradl <lgradl@openmailbox.org>
To: Danny Milosavljevic <dannym@scratchpost.org>
Cc: guix-devel@gnu.org
Subject: Re: [Patch 10/10] Add ring-client-gnome.
Date: Sat, 17 Sep 2016 11:38:33 -0500	[thread overview]
Message-ID: <8760puxz06.fsf@openmailbox.org> (raw)
In-Reply-To: <20160915232737.329b5247@scratchpost.org> (Danny Milosavljevic's message of "Thu, 15 Sep 2016 23:27:37 +0200")

[-- Attachment #1: Type: text/plain, Size: 10302 bytes --]


Thank you for your review!
And Sorry for my late answer!

Danny Milosavljevic <dannym@scratchpost.org> writes:

>> +      (propagated-inputs
>> +       `(("libring" ,libring) ; This is needed to run dring.
>
> Wait... I don't understand why it's required to be propagated...
>

I was not sure how to handle this properly.  The package libring
provides a program 'dring', which is a daemon that, as far as I
understand, does all the communication with peers on the internet.  So
in order to place a call, one first needs to run 'dring &' and then
start the ring-client-gnome.  If the ring daemon is not running, then
ring-client-gnome won't do anything useful, it will just error out on
startup.

Looking at the manual, this does not justify propagation.  My intention
was that people can just install ring-client-gnome in a profile and it
will work.  If libring is not propagated, it will also need to be
installed by hand.

Is there a good way to specify this kind of runtime requirement?  I
could also just move it to 'inputs' and put a hint in the description
that libring should be installed alongside ring-client-gnome.


>> +         ("evolution-data-server" ,evolution-data-server)))
>
> Hmm. Why is it propagated? Most other packages seem to be able to use it as a regular input just fine. Are they wrong?

This is a similar issue.  I use ratpoison and do not have
evolution-data-server in my profile.  Then, if I do this:

---8<--- cut here -------------------- start --->8---
lukas@serenity$ guix package --list-installed | grep evolution-data-server
lukas@serenity$ 
lukas@serenity$ ./pre-inst-env guix environment --no-substitutes --no-grafts --ad-hoc ring-client-gnome
warning: failed to install locale: Invalid argument
accepted connection from pid 686, user lukas
lukas@serenity [env]$ dring &
[1] 708
lukas@serenity [env]$ ring.cx
Ring Daemon 2.3.0, by Savoir-faire Linux 2004-2016
http://www.ring.cx/
[Video support enabled]

10:59:14.283 os_core_unix.c !pjlib 2.4 for POSIX initialized

(gnome-ring:711): Gtk-WARNING **: Locale not supported by C library.
	Using the fallback 'C' locale.
** Message: Ring GNOME client version: 1.0.0
** Message: git ref: unknown

(gnome-ring:711): Clutter-WARNING **: Locale not supported by C library.
Using the fallback 'C' locale.
Updating all accounts
Loading "708d2bd2d3639223"
Account parameter " "Account.presenceSubscribeSupported" " not found
Loading "45662ba5f88c4297"
Loading vcf from: QDir( "/home/lukas/.local/share/gnome-ring/profiles" , nameFilters = { "*" },  QDir::SortFlags( Name | IgnoreCase ) , QDir::Filters( Dirs|Files|Drives|AllEntries ) )

(gnome-ring:711): GLib-GIO-ERROR **: Settings schema 'org.gnome.Evolution.DefaultSources' is not installed

/gnu/store/p2pxykqp746za9z3cxply4hpjff3qfxa-profile/bin/ring.cx: line 46:   711 Trace/breakpoint trap   gnome-ring $*
lukas@serenity [env]$
---8<--- cut here -------------------- end ----->8---

It throws an error that I do not get if I instead do:

---8<--- cut here -------------------- start --->8---
lukas@serenity$ ./pre-inst-env guix environment --no-substitutes --no-grafts --ad-hoc ring-client-gnome evolution-data-server
warning: failed to install locale: Invalid argument
accepted connection from pid 1120, user lukas
lukas@serenity [env]$ dring &
[1] 1133
lukas@serenity [env]$ Ring Daemon 2.3.0, by Savoir-faire Linux 2004-2016
http://www.ring.cx/
[Video support enabled]

11:22:50.300 os_core_unix.c !pjlib 2.4 for POSIX initialized

lukas@serenity [env]$ ring.cx

(gnome-ring:1150): Gtk-WARNING **: Locale not supported by C library.
	Using the fallback 'C' locale.
** Message: Ring GNOME client version: 1.0.0
** Message: git ref: unknown

(gnome-ring:1150): Clutter-WARNING **: Locale not supported by C library.
Using the fallback 'C' locale.
Updating all accounts
Loading "708d2bd2d3639223"
Account parameter " "Account.presenceSubscribeSupported" " not found
Loading "45662ba5f88c4297"
Loading vcf from: QDir( "/home/lukas/.local/share/gnome-ring/profiles" , nameFilters = { "*" },  QDir::SortFlags( Name | IgnoreCase ) , QDir::Filters( Dirs|Files|Drives|AllEntries ) )

(gnome-ring:1150): Gtk-WARNING **: gtkwidget.c:8523: widget not within a GtkWindow
QDBusMarshaller: type `VectorString' attempts to redefine basic D-BUS type 'as' (QStringList) (Did you forget to call beginStructure() ?)
QDBusMarshaller: type `MapStringVectorString' produces invalid D-BUS signature `a{s}' (Did you forget to call beginStructure() ?)
QDBusMarshaller: type `QMap<QString,MapStringVectorString>' produces invalid D-BUS signature `a{s}' (Did you forget to call beginStructure() ?)
Invalid current audio plugin
Selection changed 0
Selection changed 0

** (gnome-ring:1150): CRITICAL **: Unable to create EDS registry: Error calling StartServiceByName for org.gnome.evolution.dataserver.Sources5: GDBus.Error:org.freedesktop.DBus.Error.Spawn.ChildSignaled: Process org.gnome.evolution.dataserver.Sources5 received signal 5

  C-c C-c
lukas@serenity [env]$ 
---8<--- cut here -------------------- end ----->8---

The main window pops up and it works.  I tried to solve this by
propagating evolution-data-server, however, the same thing still
happens.  If evolution-data-server is not explicitly installed, then
ring-client-gnome doesn't find it.  I am not sure how to proceed on
this.  I thought I had seen the same behavior with the package
'gnome-calendar' which also uses evolution-data-server, however, I am
not able to reproduce it now:

---8<--- cut here -------------------- start --->8---
lukas@serenity$ ./pre-inst-env guix environment --no-substitutes --no-grafts --ad-hoc gnome-calendar
warning: failed to install locale: Invalid argument
accepted connection from pid 6782, user lukas
lukas@serenity [env]$ gnome-calendar

(process:6793): libecal-CRITICAL **: e_cal_util_get_system_timezone: assertion 'location != NULL' failed

** (process:6793): WARNING **: Failed to access calendar configuration: Error calling StartServiceByName for org.gnome.evolution.dataserver.Sources5: GDBus.Error:org.freedesktop.DBus.Error.Spawn.ChildSignaled: Process org.gnome.evolution.dataserver.Sources5 received signal 5

** (process:6793): CRITICAL **: e_cal_data_model_set_timezone: assertion 'zone != NULL' failed

(gnome-calendar:6793): Gtk-WARNING **: Locale not supported by C library.
	Using the fallback 'C' locale.

(gnome-calendar:6793): libecal-CRITICAL **: e_cal_util_get_system_timezone: assertion 'location != NULL' failed

(gnome-calendar:6793): libecal-CRITICAL **: e_cal_util_get_system_timezone: assertion 'location != NULL' failed

(gnome-calendar:6793): GLib-GIO-ERROR **: Settings schema 'org.gnome.shell.calendar' is not installed

Trace/breakpoint trap
lukas@serenity [env]$ exit
lukas@serenity$ ./pre-inst-env guix environment --no-substitutes --no-grafts --ad-hoc gnome-calendar evolution-data-server
warning: failed to install locale: Invalid argument
accepted connection from pid 6805, user lukas
The following derivations will be built:
   /gnu/store/3p1ggs14p303s7l348qvvkaxlhx93g96-profile.drv
   /gnu/store/wkv432071y8ny4805wx7l8dz738il5ap-ca-certificate-bundle.drv
   /gnu/store/m7cx7q37qfk0bzql20cvgb38pqy2qjyx-xdg-mime-database.drv
   /gnu/store/jjj4628gf3mrvy4psaqsl3frl4wl310j-xdg-desktop-database.drv
   /gnu/store/i66lqa6sik03m8f1qnw7yha1p2cnc25a-info-dir.drv
   /gnu/store/82gf9ak5sr7s6imbvgdi8aahw6z3xb3y-gtk-icon-themes.drv
   /gnu/store/3lhi0yx05k5f3fplnpasksyqfsw5gkyp-fonts-dir.drv
warning: collision encountered: /gnu/store/grppcf85880nbzfvj9rg4fc90zx7papz-gtk+-3.20.3/share/icons/hicolor/icon-theme.cache /gnu/store/153wx94y4505kjlbvfyr20q8bafjv998-gnome-calendar-3.20.2/share/icons/hicolor/icon-theme.cache 
warning: arbitrarily choosing /gnu/store/grppcf85880nbzfvj9rg4fc90zx7papz-gtk+-3.20.3/share/icons/hicolor/icon-theme.cache
warning: collision encountered: /gnu/store/ry5ylm1fyjpr875gwga39vj459ccld4q-gtk-icon-themes/share/icons/hicolor/icon-theme.cache /gnu/store/grppcf85880nbzfvj9rg4fc90zx7papz-gtk+-3.20.3/share/icons/hicolor/icon-theme.cache /gnu/store/153wx94y4505kjlbvfyr20q8bafjv998-gnome-calendar-3.20.2/share/icons/hicolor/icon-theme.cache 
warning: arbitrarily choosing /gnu/store/ry5ylm1fyjpr875gwga39vj459ccld4q-gtk-icon-themes/share/icons/hicolor/icon-theme.cache
warning: collision encountered: /gnu/store/kav2jb48w3m18d43bk1f7rxziawpi9sa-evolution-data-server-3.20.1/share/glib-2.0/schemas/gschemas.compiled /gnu/store/grppcf85880nbzfvj9rg4fc90zx7papz-gtk+-3.20.3/share/glib-2.0/schemas/gschemas.compiled /gnu/store/153wx94y4505kjlbvfyr20q8bafjv998-gnome-calendar-3.20.2/share/glib-2.0/schemas/gschemas.compiled 
warning: arbitrarily choosing /gnu/store/kav2jb48w3m18d43bk1f7rxziawpi9sa-evolution-data-server-3.20.1/share/glib-2.0/schemas/gschemas.compiled
lukas@serenity [env]$ gnome-calendar

(process:6859): libecal-CRITICAL **: e_cal_util_get_system_timezone: assertion 'location != NULL' failed

** (process:6859): WARNING **: Failed to access calendar configuration: Error calling StartServiceByName for org.gnome.evolution.dataserver.Sources5: GDBus.Error:org.freedesktop.DBus.Error.Spawn.ChildSignaled: Process org.gnome.evolution.dataserver.Sources5 received signal 5

** (process:6859): CRITICAL **: e_cal_data_model_set_timezone: assertion 'zone != NULL' failed

(gnome-calendar:6859): Gtk-WARNING **: Locale not supported by C library.
	Using the fallback 'C' locale.

(gnome-calendar:6859): libecal-CRITICAL **: e_cal_util_get_system_timezone: assertion 'location != NULL' failed

(gnome-calendar:6859): libecal-CRITICAL **: e_cal_util_get_system_timezone: assertion 'location != NULL' failed

(gnome-calendar:6859): GLib-GIO-ERROR **: Settings schema 'org.gnome.shell.calendar' is not installed

Trace/breakpoint trap

---8<--- cut here -------------------- end ----->8---

It fails in both cases.

To come back to your question:  I am not fully convinced that other
packages using evolution-data-server are able to properly use it as an
input.  I am pretty sure that propagating evolution-data-server does not
make it a lot better.  Should I move it to 'inputs'?


Thank you for your review!
Best,
Lukas

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 818 bytes --]

  reply	other threads:[~2016-09-17 16:38 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-13  2:22 [Patch 0/10] Add Ring Lukas Gradl
2016-09-13  2:23 ` [Patch 1/10] Add pjproject Lukas Gradl
2016-09-19  7:41   ` Ricardo Wurmus
2016-09-20  5:39     ` Lukas Gradl
2016-09-22  2:46       ` Ricardo Wurmus
2016-09-26  6:41         ` Lukas Gradl
2016-11-05 22:25           ` Ricardo Wurmus
2016-11-05 22:49             ` Mathieu Lirzin
2016-11-05 22:54               ` Ricardo Wurmus
2016-09-13  2:24 ` [Patch 2/10] Add dbus-c++ Lukas Gradl
2016-09-28  9:09   ` Ludovic Courtès
2016-09-13  2:25 ` [Patch 3/10] Add gsm Lukas Gradl
2016-09-28  9:16   ` Ludovic Courtès
2016-09-13  2:26 ` [Patch 4/10] argon2: Install pkg-config file Lukas Gradl
2016-09-15 21:24   ` Danny Milosavljevic
2016-09-17 16:47     ` Lukas Gradl
2016-09-18 21:32       ` Lukas Gradl
2016-09-18 21:43     ` Lukas Gradl
2016-09-19 19:15       ` Danny Milosavljevic
2016-09-25 22:45         ` Lukas Gradl
2016-09-25 23:13           ` Danny Milosavljevic
2016-09-25 23:32             ` Lukas Gradl
2016-09-28  9:18               ` Ludovic Courtès
2016-09-13  2:27 ` [Patch 5/10] opendht: Propagate argon2 Lukas Gradl
2016-09-25 22:54   ` Lukas Gradl
2016-09-25 23:24     ` Danny Milosavljevic
2016-09-25 23:43       ` Lukas Gradl
2016-09-13  2:28 ` [Patch 6/10] telephony: Add license prefix Lukas Gradl
2016-09-15 21:25   ` Danny Milosavljevic
2016-09-28  9:25   ` Ludovic Courtès
2016-09-13  2:29 ` [Patch 7/10] opendht: Propagate msgpack Lukas Gradl
2016-09-28  9:48   ` Ludovic Courtès
2016-09-13  2:30 ` [Patch 8/10] Add libring Lukas Gradl
2016-11-05 22:30   ` Ricardo Wurmus
2016-09-13  2:30 ` [Patch 9/10] Add libringclient Lukas Gradl
2016-09-13  2:31 ` [Patch 10/10] Add ring-client-gnome Lukas Gradl
2016-09-15 21:27   ` Danny Milosavljevic
2016-09-17 16:38     ` Lukas Gradl [this message]
2016-09-25 23:23       ` Lukas Gradl
2016-09-25 23:35       ` GNOME Settings Schemas - How are they searched for? Danny Milosavljevic
2016-10-23 15:36         ` Pjotr Prins
2016-11-05  4:59 ` [Patch 0/10] Add Ring Chris Marusich
2016-11-09 18:07   ` Lukas Gradl
2016-11-09 22:43     ` Leo Famulari
2016-11-12 23:39       ` Lukas Gradl
2016-11-13  0:52     ` Mike Gerwitz
2016-11-13 21:00       ` Lukas Gradl

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8760puxz06.fsf@openmailbox.org \
    --to=lgradl@openmailbox.org \
    --cc=dannym@scratchpost.org \
    --cc=guix-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.