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

  • Done
  • quality assurance status badge
Details
3 participants
  • wednesday
  • Maxim Cournoyer
  • zimoun
Owner
unassigned
Submitted by
wednesday
Severity
normal

Debbugs page

W
W
wednesday wrote on 30 Apr 2019 14: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 30 Apr 2019 15: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 2022 05:53
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 2022 14:22
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 2022 12:51
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 an email to 35514@patchwise.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 35514
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch