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
next prev parent 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.