all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Chris Marusich <cmmarusich@gmail.com>
To: Ricardo Wurmus <rekado@elephly.net>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: Trying to fix IBus
Date: Thu, 11 Aug 2016 01:41:37 -0700	[thread overview]
Message-ID: <871t1vhf3y.fsf@gmail.com> (raw)
In-Reply-To: <87lh04covy.fsf@elephly.net> (Ricardo Wurmus's message of "Wed, 10 Aug 2016 23:09:21 +0200")

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

Ricardo Wurmus <rekado@elephly.net> writes:

> NixOS encountered the same problem:
>
>     https://github.com/NixOS/nixpkgs/pull/14568
>
> I don’t like their solution to set a variable NIX_PROFILES and let GTK
> look for immodule files in each of the directories.

Why don't you like their solution?  Why do you believe that your
proposed solution is better than their solution?  We should make sure to
choose the best solution available, and right now I'm not sure which one
is better.

> Instead, I think we should patch both GTK versions to respect
> GUIX_GTK2_IM_MODULE_FILE and GUIX_GTK3_IM_MODULE_FILE, and generate
> the immodule cache files in a profile hook.
>
> We did something similar before with GUIX_GTK2_PATH and GUIX_GTK3_PATH.

I believe you are referring to this thread:

https://lists.gnu.org/archive/html/guix-devel/2015-12/msg00046.html

Did that patch actually get committed?  If so, why didn't it solve the
problem?  I've read all the relevant discussions I could find [1], and
it isn't clear to me why we need to do what you're suggesting ("patch
both GTK versions to respect GUIX_GTK2_IM_MODULE_FILE and
GUIX_GTK3_IM_MODULE_FILE, and generate the immodule cache files in a
profile hook") if we've already committed the patch presented in the
thread above.

> What do you think?  Is this acceptable/reasonable?

Because upstream has made it clear that they won't accept a patch like
this, I think it'd be great to patch it ourselves and fix it.  I think
it would be nice to have a solution that is similar to what NixOS does,
but if your patch is really better than NixOS' solution, I would love to
see it committed.

Ricardo Wurmus <rekado@elephly.net> writes:

> Alex Griffin <a@ajgrf.com> writes:
>
>> On Wed, Aug 10, 2016, at 04:09 PM, Ricardo Wurmus wrote:
>>> What do you think?  Is this acceptable/reasonable?
>>
>> It seems to me like it's probably fine, for whatever that's worth, and
>> also cleaner than the NIX_PROFILES solution. How will this work on
>> foreign distros, though?
>
> On foreign distros you would have to use IBus from Guix with GTK from
> Guix.  This means that installing IBus from Guix and expecting it to
> work with the foreign distro’s applications won’t work.  Likewise you
> cannot use the distro’s IBus and use it in applications installed via
> Guix.
>
> That’s a general problem — we already experience this with R or Python
> (mixing modules installed via different means causes crashes).

As unpleasant as that would be, it would still be better than the
current situation, in which IBus doesn't work for all installed
applications due to the GTK+ major version incompatibility.

Footnotes: 
[1] All the threads here:

* GuixSD discussion:
  https://lists.gnu.org/archive/html/guix-devel/2015-09/msg00306.html

* GuixSD solution (?):
  https://lists.gnu.org/archive/html/guix-devel/2015-12/msg00046.html

* More GuixSD discussion (I wasn't aware of the other threads at the
  time):https://lists.gnu.org/archive/html/help-guix/2016-04/msg00006.html

* NixOS solution: https://github.com/NixOS/nixpkgs/pull/14568

* NixOS discussion: https://github.com/NixOS/nixpkgs/pull/14417

* Upstream response to NixOS patch:
  https://bugzilla.gnome.org/show_bug.cgi?id=764551

-- 
Chris

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

  parent reply	other threads:[~2016-08-11  8:41 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-10 21:09 Trying to fix IBus Ricardo Wurmus
2016-08-10 22:35 ` Alex Griffin
2016-08-11  5:50   ` Ricardo Wurmus
2016-08-11  8:41 ` Chris Marusich [this message]
2016-08-11 10:14   ` Ricardo Wurmus
2016-08-11 10:51     ` Chris Marusich
2016-08-11 13:12       ` Ricardo Wurmus
2016-08-11 13:17         ` Ricardo Wurmus
2016-08-12 16:34           ` Chris Marusich

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=871t1vhf3y.fsf@gmail.com \
    --to=cmmarusich@gmail.com \
    --cc=guix-devel@gnu.org \
    --cc=rekado@elephly.net \
    /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.