GNU bug report logs - #35516
[PATCH 0/1] gnu: emacs: Add lcms as an input.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 35516 in the body.
You can then email your comments to 35516 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: wednesday <jessejohngildersleve <at> zohomail.eu>
To: guix-patches <at> gnu.org
Cc: wednesday <jessejohngildersleve <at> zohomail.eu>
Subject: [PATCH 0/1] gnu: emacs: Add lcms as an input.
Date: Tue, 30 Apr 2019 23:25:20 +0100

There is not much of a need or reason for this, other than futher completing
the TODO, which after this and my other patches I think may be complete. What
adding this does is add a few elisp functions for converting to some formats,
and working with lcms.

wednesday (1):
  gnu: emacs: Add lcms to inputs.

 gnu/packages/emacs.scm | 2 ++
 1 file changed, 2 insertions(+)

-- 
2.21.0

Message #8 received at 35516 <at> debbugs.gnu.org (full text, mbox):

From: wednesday <jessejohngildersleve <at> zohomail.eu>
To: 35516 <at> debbugs.gnu.org
Cc: wednesday <jessejohngildersleve <at> zohomail.eu>
Subject: [PATCH 1/1] gnu: emacs: Add lcms to inputs.
Date: Tue, 30 Apr 2019 23:26:37 +0100

* gnu/packages/emacs.scm (emacs)[inputs]: Add lcms for some color functions.
---
 gnu/packages/emacs.scm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 7c60cc6021..ce791ca3ab 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -48,6 +48,7 @@
   #:use-module (gnu packages fribidi)
   #:use-module (gnu packages gd)
   #:use-module (gnu packages gettext)
+  #:use-module (gnu packages ghostscript) ; for lcms
   #:use-module (gnu packages glib)
   #:use-module (gnu packages gnome) ; for librsvg
   #:use-module (gnu packages gtk)
@@ -146,6 +147,7 @@
        ("ncurses" ,ncurses)
 
        ;; TODO: Add the optional dependencies.
+       ("lcms" ,lcms)
        ("libx11" ,libx11)
        ("gtk+" ,gtk+)
        ("libxft" ,libxft)
-- 
2.21.0

Message #11 received at 35516 <at> debbugs.gnu.org (full text, mbox):

From: Brett Gilio <brettg <at> gnu.org>
To: wednesday <jessejohngildersleve <at> zohomail.eu>
Cc: 35516 <at> debbugs.gnu.org
Subject: Re: [bug#35516] [PATCH 0/1] gnu: emacs: Add lcms as an input.
Date: Fri, 24 Jul 2020 20:41:55 -0500

wednesday <jessejohngildersleve <at> zohomail.eu> writes:

> There is not much of a need or reason for this, other than futher completing
> the TODO, which after this and my other patches I think may be complete. What
> adding this does is add a few elisp functions for converting to some formats,
> and working with lcms.
>
> wednesday (1):
>   gnu: emacs: Add lcms to inputs.
>
>  gnu/packages/emacs.scm | 2 ++
>  1 file changed, 2 insertions(+)

Hi, I know this patch is quite old but our Emacs currently still does
not have LCMS support. LCMS was introduced in the 26 release, and I am
curious if anybody had any thoughts on adding this? From what I gather
27 is due to be marked as stable any week now.

Message #14 received at 35516 <at> debbugs.gnu.org (full text, mbox):

From: Marius Bakke <marius <at> gnu.org>
To: Brett Gilio <brettg <at> gnu.org>, wednesday <jessejohngildersleve <at> zohomail.eu>
Cc: 35516 <at> debbugs.gnu.org
Subject: Re: [bug#35516] [PATCH 0/1] gnu: emacs: Add lcms as an input.
Date: Sat, 25 Jul 2020 18:21:11 +0200

[Message part 1 (text/plain, inline)]

Brett Gilio <brettg <at> gnu.org> writes:

> wednesday <jessejohngildersleve <at> zohomail.eu> writes:
>
>> There is not much of a need or reason for this, other than futher completing
>> the TODO, which after this and my other patches I think may be complete. What
>> adding this does is add a few elisp functions for converting to some formats,
>> and working with lcms.
>>
>> wednesday (1):
>>   gnu: emacs: Add lcms to inputs.
>>
>>  gnu/packages/emacs.scm | 2 ++
>>  1 file changed, 2 insertions(+)
>
> Hi, I know this patch is quite old but our Emacs currently still does
> not have LCMS support. LCMS was introduced in the 26 release, and I am
> curious if anybody had any thoughts on adding this? From what I gather
> 27 is due to be marked as stable any week now.

No objections from me.  Emacs already has 'lcms' in its closure so there
would be no size difference.

[signature.asc (application/pgp-signature, inline)]

Message #17 received at 35516 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Marius Bakke <marius <at> gnu.org>
Cc: wednesday <jessejohngildersleve <at> zohomail.eu>, Brett Gilio <brettg <at> gnu.org>,
 35516 <at> debbugs.gnu.org
Subject: Re: bug#35516: [PATCH 0/1] gnu: emacs: Add lcms as an input.
Date: Thu, 07 Apr 2022 15:00:58 +0200

Hi,

Looking at old submission, it appears that

        <http://issues.guix.gnu.org/issue/35516>

had fallen into the cracks.


On Sat, 25 Jul 2020 at 18:21, Marius Bakke <marius <at> gnu.org> wrote:
> Brett Gilio <brettg <at> gnu.org> writes:
>> wednesday <jessejohngildersleve <at> zohomail.eu> writes:
>>
>>> There is not much of a need or reason for this, other than futher completing
>>> the TODO, which after this and my other patches I think may be complete. What
>>> adding this does is add a few elisp functions for converting to some formats,
>>> and working with lcms.
>>>
>>> wednesday (1):
>>>   gnu: emacs: Add lcms to inputs.
>>>
>>>  gnu/packages/emacs.scm | 2 ++
>>>  1 file changed, 2 insertions(+)
>>
>> Hi, I know this patch is quite old but our Emacs currently still does
>> not have LCMS support. LCMS was introduced in the 26 release, and I am
>> curious if anybody had any thoughts on adding this? From what I gather
>> 27 is due to be marked as stable any week now.
>
> No objections from me.  Emacs already has 'lcms' in its closure so there
> would be no size difference.

 Marius, since you have commit access, could you push this change?  And
thus close this. :-)


Cheers,
simon

Message #22 received at 35516-done <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Marius Bakke <marius <at> gnu.org>
Cc: wednesday <jessejohngildersleve <at> zohomail.eu>, 35516-done <at> debbugs.gnu.org,
 Brett Gilio <brettg <at> gnu.org>
Subject: Re: bug#35516: [PATCH 0/1] gnu: emacs: Add lcms as an input.
Date: Sat, 21 May 2022 15:50:48 -0400

Hi,

Marius Bakke <marius <at> gnu.org> writes:

> Brett Gilio <brettg <at> gnu.org> writes:
>
>> wednesday <jessejohngildersleve <at> zohomail.eu> writes:
>>
>>> There is not much of a need or reason for this, other than futher completing
>>> the TODO, which after this and my other patches I think may be complete. What
>>> adding this does is add a few elisp functions for converting to some formats,
>>> and working with lcms.
>>>
>>> wednesday (1):
>>>   gnu: emacs: Add lcms to inputs.
>>>
>>>  gnu/packages/emacs.scm | 2 ++
>>>  1 file changed, 2 insertions(+)
>>
>> Hi, I know this patch is quite old but our Emacs currently still does
>> not have LCMS support. LCMS was introduced in the 26 release, and I am
>> curious if anybody had any thoughts on adding this? From what I gather
>> 27 is due to be marked as stable any week now.
>
> No objections from me.  Emacs already has 'lcms' in its closure so there
> would be no size difference.

Applied to staging with fd5b7393a0.

Closing; thank you!

Maxim

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.