all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Alice BRENON <alice.brenon@ens-lyon.fr>
To: Maxim Cournoyer <maxim.cournoyer@gmail.com>
Cc: 52576@debbugs.gnu.org, Taiju HIGASHI <higashi@taiju.info>
Subject: [bug#52576] [PATCH] gnu: ibus-anthy: Update to 1.15.12
Date: Thu, 30 Jun 2022 22:15:47 +0200	[thread overview]
Message-ID: <20220630221547.3ba3cee3@ens-lyon.fr> (raw)
In-Reply-To: <83F454AE-32A5-41B1-A5E5-11B585979656@gmail.com>

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

Alright, thanks for the last missing piece of the puzzle.

Here is a fix hopefully not too different from the one you'd have
written. If everyone agrees on it, I think someone with commit rights
could merge it on master even before you come back closer to a proper
keyboard to avoid more users hitting the problem. And, in any case,
before this happens there's the quickfix I suggested above : )

Cheers,

Alice

Le Wed, 29 Jun 2022 22:25:24 -0400,
Maxim Cournoyer <maxim.cournoyer@gmail.com> a écrit :

> On June 29, 2022 10:35:04 AM EDT, Alice BRENON
> <alice.brenon@ens-lyon.fr> wrote:
> 
> [...]
> 
> >And now all I want to know is if we can revert `inputs` back to
> >`outputs` or if there is a good reason not to do so, and hence what
> >should be done in that case, but it seems that it is a matter on
> >which Maxim Cournoyer has the expertise, since he was the one to
> >introduce the change.  
> 
> [...]
> 
> Seems an oversight on my part, apologies!  'inputs' in
> 
> + ,(search-input-directory
> + inputs "lib/girepository-1.0")))))
> 
> Should have been 'outputs'.  Unfortunately I'm nowhere near a
> keyboard for a week, so won't be able to commit the trivial fix in
> that time.
> 
> Thanks,
> 
> Maxim
> 
> Hi,


[-- Attachment #2: 0001-gnu-ibus-anthy-Fix-wrapped-programs.patch --]
[-- Type: text/x-patch, Size: 1676 bytes --]

From e851ab2c68cb2fab26bf686d56fdb1488ef9e78d Mon Sep 17 00:00:00 2001
From: Alice BRENON <alice.brenon@ens-lyon.fr>
Date: Thu, 30 Jun 2022 22:02:22 +0200
Subject: [PATCH] gnu: ibus-anthy: Fix wrapped programs

This is in follow up to 89db6e7ec77bf0f33474e47945755f5ab45e9f06, the
proper fix wasn't to add inputs to the arguments but to use outputs to
set the GI_TYPELIB_PATH.

* gnu/packages/ibus.scm (ibus-anthy)
[phase]{wrap-programs}: Search within outputs instead of inputs to set
GI_TYPELIB_PATH.
---
 gnu/packages/ibus.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/ibus.scm b/gnu/packages/ibus.scm
index 234d4bd75f..9e8b0f124b 100644
--- a/gnu/packages/ibus.scm
+++ b/gnu/packages/ibus.scm
@@ -329,7 +329,7 @@ (define-public ibus-anthy
        #:phases
        (modify-phases %standard-phases
          (add-after 'install 'wrap-programs
-           (lambda* (#:key inputs outputs #:allow-other-keys)
+           (lambda* (#:key outputs #:allow-other-keys)
              (for-each (lambda (prog)
                          (wrap-program (search-input-file
                                         outputs (string-append "libexec/" prog))
@@ -338,7 +338,7 @@ (define-public ibus-anthy
                            `("GI_TYPELIB_PATH" ":" prefix
                              (,(getenv "GI_TYPELIB_PATH")
                               ,(search-input-directory
-                                inputs "lib/girepository-1.0")))))
+                                outputs "lib/girepository-1.0")))))
                        '("ibus-engine-anthy" "ibus-setup-anthy")))))))
     (native-inputs
      (list gettext-minimal
-- 
2.36.1


  reply	other threads:[~2022-06-30 20:16 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-17 13:40 [bug#52576] [PATCH] gnu: ibus-anthy: Update to 1.15.12 Taiju HIGASHI
2021-12-17 16:54 ` Taiju HIGASHI
2021-12-22 22:20 ` Ludovic Courtès
2021-12-23  5:44   ` Taiju HIGASHI
2021-12-24  9:25   ` Taiju HIGASHI
2021-12-24  9:33   ` Taiju HIGASHI
2021-12-24 23:18 ` [bug#52576] [PATCH v2] gnu: ibus-anthy: Update to 1.5.14 Taiju HIGASHI
2022-02-06 15:56 ` [bug#52576] About the upgrade of ibus-anthy Taiju HIGASHI
2022-02-06 20:51   ` Liliana Marie Prikler
2022-02-07  3:03     ` Taiju HIGASHI
2022-02-07 14:50       ` Julien Lepiller
2022-02-07 15:51         ` Taiju HIGASHI
2022-06-24  2:53         ` bug#52576: [PATCH] gnu: ibus-anthy: Update to 1.15.12 Maxim Cournoyer
2022-06-24  3:10           ` [bug#52576] " Taiju HIGASHI
2022-06-24  4:21           ` Liliana Marie Prikler
2022-06-24 14:50             ` Maxim Cournoyer
2022-06-28  8:43 ` Alice BRENON
2022-06-29  8:50   ` Taiju HIGASHI
2022-06-29  9:25     ` Taiju HIGASHI
2022-06-29 13:46       ` Alice BRENON
2022-06-29 14:06         ` Taiju HIGASHI
2022-06-29 14:19           ` Taiju HIGASHI
2022-06-29 14:35             ` Alice BRENON
2022-06-29 14:49               ` Taiju HIGASHI
2022-06-29 14:56                 ` Taiju HIGASHI
2022-06-30  2:25               ` Maxim Cournoyer
2022-06-30 20:15                 ` Alice BRENON [this message]
2022-07-01  4:22                 ` Liliana Marie Prikler
2022-07-02  0:14                   ` Liliana Marie Prikler
2022-07-03  1:13                     ` Dominic Martinez

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=20220630221547.3ba3cee3@ens-lyon.fr \
    --to=alice.brenon@ens-lyon.fr \
    --cc=52576@debbugs.gnu.org \
    --cc=higashi@taiju.info \
    --cc=maxim.cournoyer@gmail.com \
    /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.