* [bug#71565] [PATCH] gnu: ibus-minimal: Graft to 1.5.29.
@ 2024-06-15 7:18 Liliana Marie Prikler
2024-06-17 19:55 ` Maxim Cournoyer
0 siblings, 1 reply; 6+ messages in thread
From: Liliana Marie Prikler @ 2024-06-15 7:18 UTC (permalink / raw)
To: 71565; +Cc: liliana.prikler, maxim.cournoyer, vivien
This fixes an issue, where IBus was not able to query emoji lists.
See also <https://github.com/ibus/ibus/issues/2476>.
* gnu/packages/ibus.scm (ibus-minimal/fixed): New variable.
(ibus-minimal): Add ibus-minimal/fixed as replacement.
---
gnu/packages/ibus.scm | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
diff --git a/gnu/packages/ibus.scm b/gnu/packages/ibus.scm
index c821279bd3..a2775b2f84 100644
--- a/gnu/packages/ibus.scm
+++ b/gnu/packages/ibus.scm
@@ -81,6 +81,7 @@ (define-module (gnu packages ibus)
(define-public ibus-minimal
(package
(name "ibus")
+ (replacement ibus-minimal/fixed)
(version "1.5.27")
(source (origin
(method url-fetch)
@@ -279,6 +280,23 @@ (define-public ibus
(package-native-search-paths ibus-minimal)))
(properties (alist-delete 'hidden? (package-properties ibus-minimal)))))
+(define-public ibus-minimal/fixed
+ ;; Fixes <https://github.com/ibus/ibus/issues/2476>.
+ ;; Ungraft on gnome-team, core-updates or a dedicated ungrafing branch.
+ (package
+ (inherit ibus-minimal)
+ (version "1.5.29")
+ (source (origin
+ (method url-fetch)
+ (uri (string-append "https://github.com/ibus/ibus/"
+ "releases/download/"
+ version "/ibus-" version ".tar.gz"))
+ (sha256
+ (base32
+ "0vjybn3xq5sz616fdy21f5c4b4ajrj4wmfnbjqz6584xw887yiaa"))))
+ (inputs (modify-inputs (package-inputs ibus-minimal)
+ (prepend libdbusmenu)))))
+
(define-public ibus-libpinyin
(package
(name "ibus-libpinyin")
base-commit: 7770e4140cddd5dc9c9879cb2bfcbd6cd4de59a0
--
2.45.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* [bug#71565] [PATCH] gnu: ibus-minimal: Graft to 1.5.29.
2024-06-15 7:18 [bug#71565] [PATCH] gnu: ibus-minimal: Graft to 1.5.29 Liliana Marie Prikler
@ 2024-06-17 19:55 ` Maxim Cournoyer
2024-06-17 20:05 ` Liliana Marie Prikler
0 siblings, 1 reply; 6+ messages in thread
From: Maxim Cournoyer @ 2024-06-17 19:55 UTC (permalink / raw)
To: Liliana Marie Prikler; +Cc: vivien, 71565
Hi Liliana,
Liliana Marie Prikler <liliana.prikler@gmail.com> writes:
> This fixes an issue, where IBus was not able to query emoji lists.
> See also <https://github.com/ibus/ibus/issues/2476>.
>
> * gnu/packages/ibus.scm (ibus-minimal/fixed): New variable.
> (ibus-minimal): Add ibus-minimal/fixed as replacement.
> ---
> gnu/packages/ibus.scm | 18 ++++++++++++++++++
> 1 file changed, 18 insertions(+)
>
> diff --git a/gnu/packages/ibus.scm b/gnu/packages/ibus.scm
> index c821279bd3..a2775b2f84 100644
> --- a/gnu/packages/ibus.scm
> +++ b/gnu/packages/ibus.scm
> @@ -81,6 +81,7 @@ (define-module (gnu packages ibus)
> (define-public ibus-minimal
> (package
> (name "ibus")
> + (replacement ibus-minimal/fixed)
> (version "1.5.27")
> (source (origin
> (method url-fetch)
> @@ -279,6 +280,23 @@ (define-public ibus
> (package-native-search-paths ibus-minimal)))
> (properties (alist-delete 'hidden? (package-properties ibus-minimal)))))
>
> +(define-public ibus-minimal/fixed
Is it useful to have the package 'public' at al here? I think there are
considerations so that the package gets built by the CI, but I'm not
sure whether this applies to graft package (I guess it does). Perhaps
we should document these gotchas, as I can never remember myself.
--
Thanks,
Maxim
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#71565] [PATCH] gnu: ibus-minimal: Graft to 1.5.29.
2024-06-17 19:55 ` Maxim Cournoyer
@ 2024-06-17 20:05 ` Liliana Marie Prikler
2024-06-24 2:13 ` Maxim Cournoyer
0 siblings, 1 reply; 6+ messages in thread
From: Liliana Marie Prikler @ 2024-06-17 20:05 UTC (permalink / raw)
To: Maxim Cournoyer; +Cc: vivien, 71565
Am Montag, dem 17.06.2024 um 15:55 -0400 schrieb Maxim Cournoyer:
> Hi Liliana,
>
> Liliana Marie Prikler <liliana.prikler@gmail.com> writes:
>
> > This fixes an issue, where IBus was not able to query emoji lists.
> > See also <https://github.com/ibus/ibus/issues/2476>.
> >
> > * gnu/packages/ibus.scm (ibus-minimal/fixed): New variable.
> > (ibus-minimal): Add ibus-minimal/fixed as replacement.
> > ---
> > gnu/packages/ibus.scm | 18 ++++++++++++++++++
> > 1 file changed, 18 insertions(+)
> >
> > diff --git a/gnu/packages/ibus.scm b/gnu/packages/ibus.scm
> > index c821279bd3..a2775b2f84 100644
> > --- a/gnu/packages/ibus.scm
> > +++ b/gnu/packages/ibus.scm
> > @@ -81,6 +81,7 @@ (define-module (gnu packages ibus)
> > (define-public ibus-minimal
> > (package
> > (name "ibus")
> > + (replacement ibus-minimal/fixed)
> > (version "1.5.27")
> > (source (origin
> > (method url-fetch)
> > @@ -279,6 +280,23 @@ (define-public ibus
> > (package-native-search-paths ibus-minimal)))
> > (properties (alist-delete 'hidden? (package-properties ibus-
> > minimal)))))
> >
> > +(define-public ibus-minimal/fixed
>
> Is it useful to have the package 'public' at al here? I think there
> are considerations so that the package gets built by the CI, but I'm
> not sure whether this applies to graft package (I guess it does).
> Perhaps we should document these gotchas, as I can never remember
> myself.
I might have been cargo-culting the -public part from other
definitions. Are grafts typically not exported?
Cheers
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#71565] [PATCH] gnu: ibus-minimal: Graft to 1.5.29.
2024-06-17 20:05 ` Liliana Marie Prikler
@ 2024-06-24 2:13 ` Maxim Cournoyer
2024-06-24 3:44 ` Liliana Marie Prikler
0 siblings, 1 reply; 6+ messages in thread
From: Maxim Cournoyer @ 2024-06-24 2:13 UTC (permalink / raw)
To: Liliana Marie Prikler; +Cc: vivien, 71565
Hi Liliana,
Liliana Marie Prikler <liliana.prikler@gmail.com> writes:
> Am Montag, dem 17.06.2024 um 15:55 -0400 schrieb Maxim Cournoyer:
>> Hi Liliana,
>>
>> Liliana Marie Prikler <liliana.prikler@gmail.com> writes:
>>
>> > This fixes an issue, where IBus was not able to query emoji lists.
>> > See also <https://github.com/ibus/ibus/issues/2476>.
>> >
>> > * gnu/packages/ibus.scm (ibus-minimal/fixed): New variable.
>> > (ibus-minimal): Add ibus-minimal/fixed as replacement.
>> > ---
>> > gnu/packages/ibus.scm | 18 ++++++++++++++++++
>> > 1 file changed, 18 insertions(+)
>> >
>> > diff --git a/gnu/packages/ibus.scm b/gnu/packages/ibus.scm
>> > index c821279bd3..a2775b2f84 100644
>> > --- a/gnu/packages/ibus.scm
>> > +++ b/gnu/packages/ibus.scm
>> > @@ -81,6 +81,7 @@ (define-module (gnu packages ibus)
>> > (define-public ibus-minimal
>> > (package
>> > (name "ibus")
>> > + (replacement ibus-minimal/fixed)
>> > (version "1.5.27")
>> > (source (origin
>> > (method url-fetch)
>> > @@ -279,6 +280,23 @@ (define-public ibus
>> > (package-native-search-paths ibus-minimal)))
>> > (properties (alist-delete 'hidden? (package-properties ibus-
>> > minimal)))))
>> >
>> > +(define-public ibus-minimal/fixed
>>
>> Is it useful to have the package 'public' at al here? I think there
>> are considerations so that the package gets built by the CI, but I'm
>> not sure whether this applies to graft package (I guess it does).
>> Perhaps we should document these gotchas, as I can never remember
>> myself.
> I might have been cargo-culting the -public part from other
> definitions. Are grafts typically not exported?
Not thinking about CI shenanigans, I think it's more logical that they
be private, since we want the users to use the grafted package, not the
graft itself (although in the end the result should be the same).
--
Thanks,
Maxim
^ permalink raw reply [flat|nested] 6+ messages in thread
* [bug#71565] [PATCH] gnu: ibus-minimal: Graft to 1.5.29.
2024-06-24 2:13 ` Maxim Cournoyer
@ 2024-06-24 3:44 ` Liliana Marie Prikler
2024-06-24 4:50 ` bug#71565: " Maxim Cournoyer
0 siblings, 1 reply; 6+ messages in thread
From: Liliana Marie Prikler @ 2024-06-24 3:44 UTC (permalink / raw)
To: Maxim Cournoyer; +Cc: vivien, 71565
Hi Maxim,
Am Sonntag, dem 23.06.2024 um 22:13 -0400 schrieb Maxim Cournoyer:
> > […] I might have been cargo-culting the -public part from other
> > definitions. Are grafts typically not exported?
>
> Not thinking about CI shenanigans, I think it's more logical that
> they be private, since we want the users to use the grafted package,
> not the graft itself (although in the end the result should be the
> same).
You mean that users end up accidentally discovering and using the wrong
ibus through CLI? I don't think that's an issue for ibus-minimal,
though, since it's hidden anyway, and I do trust users who write the
files to be sensible here.
On that note, I think I remember a graft which had to be public to also
fix issues in certain dependent packages. (I might have committed that
one myself and IIRC, it was meson). Feel free to drop the -public part
from the ibus graft though; I had pushed it without resolving this
conversation.
Cheers
^ permalink raw reply [flat|nested] 6+ messages in thread
* bug#71565: [PATCH] gnu: ibus-minimal: Graft to 1.5.29.
2024-06-24 3:44 ` Liliana Marie Prikler
@ 2024-06-24 4:50 ` Maxim Cournoyer
0 siblings, 0 replies; 6+ messages in thread
From: Maxim Cournoyer @ 2024-06-24 4:50 UTC (permalink / raw)
To: Liliana Marie Prikler; +Cc: vivien, 71565-done
Hi Liliana,
Liliana Marie Prikler <liliana.prikler@gmail.com> writes:
> Hi Maxim,
>
> Am Sonntag, dem 23.06.2024 um 22:13 -0400 schrieb Maxim Cournoyer:
>> > […] I might have been cargo-culting the -public part from other
>> > definitions. Are grafts typically not exported?
>>
>> Not thinking about CI shenanigans, I think it's more logical that
>> they be private, since we want the users to use the grafted package,
>> not the graft itself (although in the end the result should be the
>> same).
>
> You mean that users end up accidentally discovering and using the wrong
> ibus through CLI? I don't think that's an issue for ibus-minimal,
> though, since it's hidden anyway, and I do trust users who write the
> files to be sensible here.
>
> On that note, I think I remember a graft which had to be public to also
> fix issues in certain dependent packages. (I might have committed that
> one myself and IIRC, it was meson). Feel free to drop the -public part
> from the ibus graft though; I had pushed it without resolving this
> conversation.
No big deal, but a reminder that we should add some proper grafting
guidelines to our doc.
--
Thanks,
Maxim
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-06-24 4:52 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-15 7:18 [bug#71565] [PATCH] gnu: ibus-minimal: Graft to 1.5.29 Liliana Marie Prikler
2024-06-17 19:55 ` Maxim Cournoyer
2024-06-17 20:05 ` Liliana Marie Prikler
2024-06-24 2:13 ` Maxim Cournoyer
2024-06-24 3:44 ` Liliana Marie Prikler
2024-06-24 4:50 ` bug#71565: " Maxim Cournoyer
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).