[PATCH 0/1] gnu: emacs: Add libselinux to inputs.

DoneSubmitted by wednesday.
Details
3 participants
  • wednesday
  • Maxim Cournoyer
  • zimoun
Owner
unassigned
Severity
normal
W
W
wednesday wrote on 30 Apr 2019 23:57
(address . guix-patches@gnu.org)(name . wednesday)(address . jessejohngildersleve@zohomail.eu)
20190430215755.4894-1-jessejohngildersleve@zohomail.eu
Adding libselinux to the inputs of emacs allows it to support it's
extended file attributes, so emacs can get and set the selinux context
of a file, as explained near the bottom of these pages here:

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

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

--
2.21.0
W
W
wednesday wrote on 1 May 2019 00:06
[PATCH 1/1] gnu: emacs: Add libselinux to inputs.
(address . 35514@debbugs.gnu.org)(name . wednesday)(address . jessejohngildersleve@zohomail.eu)
20190430220656.5039-2-jessejohngildersleve@zohomail.eu
* gnu/packages/emacs.scm (emacs)[inputs]: Add libselinux to support
extended attributes.
---
gnu/packages/emacs.scm | 2 ++
1 file changed, 2 insertions(+)

Toggle diff (22 lines)
diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 7c60cc6021..9881dadcd7 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -57,6 +57,7 @@
   #:use-module (gnu packages linux) ; alsa-lib
   #:use-module (gnu packages ncurses)
   #:use-module (gnu packages pkg-config)
+  #:use-module (gnu packages selinux) ; for libselinux
   #:use-module (gnu packages texinfo)
   #:use-module (gnu packages tls)
   #:use-module (gnu packages webkit)
@@ -146,6 +147,7 @@
        ("ncurses" ,ncurses)
 
        ;; TODO: Add the optional dependencies.
+       ("libselinux" ,libselinux)
        ("libx11" ,libx11)
        ("gtk+" ,gtk+)
        ("libxft" ,libxft)
-- 
2.21.0
Z
Z
zimoun wrote on 7 Apr 14:53 +0200
Re: bug#35514: [PATCH 0/1] gnu: emacs: Add libselinux to inputs.
(name . wednesday)(address . jessejohngildersleve@zohomail.eu)(address . 35514@debbugs.gnu.org)
865ynl2gy7.fsf@gmail.com
Hi,

Thank you for your contribution [1]. It appears that we missed it. Sorry
for that.



On Tue, 30 Apr 2019 at 22:57, wednesday <jessejohngildersleve@zohomail.eu> wrote:

Toggle quote (9 lines)
> Adding libselinux to the inputs of emacs allows it to support it's
> extended file attributes, so emacs can get and set the selinux context
> of a file, as explained near the bottom of these pages here:
> https://www.gnu.org/software/emacs/manual/html_node/elisp/Extended-Attributes.html
> https://www.gnu.org/software/emacs/manual/html_node/elisp/Changing-Files.html
>
> wednesday (1):
> gnu: emacs: Add libselinux to inputs.

Well, I am not sure it is worth to add this selinux for the base Emacs.
Maybe, such addition could be via a variant. WDYT?


Cheers,
simon
Z
Z
zimoun wrote on 19 May 23:22 +0200
control message for bug #35514
(address . control@debbugs.gnu.org)
86h75l8bfk.fsf@gmail.com
tags 35514 + moreinfo
quit
M
M
Maxim Cournoyer wrote on 21 May 21:51 +0200
Re: bug#35514: [PATCH 0/1] gnu: emacs: Add libselinux to inputs.
(name . zimoun)(address . zimon.toutoune@gmail.com)
87ee0mbr5t.fsf_-_@gmail.com
Hello,

zimoun <zimon.toutoune@gmail.com> writes:

Toggle quote (22 lines)
> Hi,
>
> Thank you for your contribution [1]. It appears that we missed it. Sorry
> for that.
>
> 1: <http://issues.guix.gnu.org/issue/35514>
>
>
> On Tue, 30 Apr 2019 at 22:57, wednesday <jessejohngildersleve@zohomail.eu> wrote:
>
>> Adding libselinux to the inputs of emacs allows it to support it's
>> extended file attributes, so emacs can get and set the selinux context
>> of a file, as explained near the bottom of these pages here:
>> https://www.gnu.org/software/emacs/manual/html_node/elisp/Extended-Attributes.html
>> https://www.gnu.org/software/emacs/manual/html_node/elisp/Changing-Files.html
>>
>> wednesday (1):
>> gnu: emacs: Add libselinux to inputs.
>
> Well, I am not sure it is worth to add this selinux for the base Emacs.
> Maybe, such addition could be via a variant. WDYT?

It only adds about 4 MiB to the 1.1 GiB+ closure of Emacs, so I applied
it to the staging branch as 15870cc08d.

Closing; thanks!

Maxim
Closed
?
Your comment

This issue is archived.

To comment on this conversation send email to 35514@debbugs.gnu.org