all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#45629] [PATCH] gnu: Export xorg variables for startx.
@ 2021-01-03 16:27 John Soo
  2021-02-11  9:36 ` 宋文武
  0 siblings, 1 reply; 6+ messages in thread
From: John Soo @ 2021-01-03 16:27 UTC (permalink / raw)
  To: 45629

[-- Attachment #1: Type: text/plain, Size: 393 bytes --]

Hi Guix,

I wanted to try using startx instead of a display manager.  I finally
got it to work, but it required at very least these patches.  I also had
to create a new service type but I think that it needs more work to be
put in upstream.

Is startx something we want to support?  It seems like the user should
be able to construct a system as the they see fit.

What do you think?

- John


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-xinit-Specify-xinputs-in-configure-flags.patch --]
[-- Type: text/x-patch, Size: 1806 bytes --]

From 8ab08a2ca7e29c53468d72577e0b3b8286d652b5 Mon Sep 17 00:00:00 2001
From: John Soo <jsoo1@asu.edu>
Date: Mon, 30 Mar 2020 03:33:08 -0700
Subject: [PATCH 1/4] gnu: xinit: Specify xinputs in configure flags.

* gnu/packages/xorg.scm (xinit):[arguments] Specify twm, xauth, xclock,
  xorg-server, xrdb, xorg-server, and xterm in configure flags and propagate
  them.
---
 gnu/packages/xorg.scm | 28 +++++++++++++++++++++++++++-
 1 file changed, 27 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/xorg.scm b/gnu/packages/xorg.scm
index 42596aa0f1..d7f55394ab 100644
--- a/gnu/packages/xorg.scm
+++ b/gnu/packages/xorg.scm
@@ -5922,7 +5922,33 @@ The XCB util-wm module provides the following libraries:
     (native-inputs
      `(("pkg-config" ,pkg-config)))
     (propagated-inputs
-     `(("xauth" ,xauth)))
+     `(("twm" ,twm)
+       ("xauth" ,xauth)
+       ("xclock" ,xclock)
+       ("xorg-server" ,xorg-server)
+       ("xrdb" ,xrdb)
+       ("xterm" ,xterm)))
+    (arguments
+     `(#:configure-flags
+       (list
+        (string-append
+         "--with-twm="
+         (assoc-ref %build-inputs "twm") "/bin/twm")
+        (string-append
+         "--with-xauth="
+         (assoc-ref %build-inputs "xauth") "/bin/xauth")
+        (string-append
+         "--with-xclock="
+         (assoc-ref %build-inputs "xclock") "/bin/xclock")
+        (string-append
+         "--with-xrdb="
+         (assoc-ref %build-inputs "xrdb") "/bin/xrdb")
+        (string-append
+         "--with-xserver="
+         (assoc-ref %build-inputs "xorg-server") "/bin/X")
+        (string-append
+         "--with-xterm="
+         (assoc-ref %build-inputs "xterm") "/bin/xterm"))))
     (home-page "https://www.x.org/")
     (synopsis "Commands to start the X Window server")
     (description
-- 
2.29.2


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #3: 0002-gnu-xorg-configuration-directory-Expose-the-function.patch --]
[-- Type: text/x-patch, Size: 757 bytes --]

From fe434e4d4c53d9d8c27fcd8ab882aa4b027f4e17 Mon Sep 17 00:00:00 2001
From: John Soo <jsoo1@asu.edu>
Date: Wed, 1 Apr 2020 08:23:37 -0700
Subject: [PATCH 2/4] gnu: xorg-configuration-directory: Expose the function.

* gnu/services/xorg.scm (xorg-configuration-directory): Expose it.
---
 gnu/services/xorg.scm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gnu/services/xorg.scm b/gnu/services/xorg.scm
index 4590709187..1c76049d23 100644
--- a/gnu/services/xorg.scm
+++ b/gnu/services/xorg.scm
@@ -65,6 +65,7 @@
             xorg-configuration-server
             xorg-configuration-server-arguments
 
+            xorg-configuration-directory
             %default-xorg-modules
             %default-xorg-fonts
             xorg-wrapper
-- 
2.29.2


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #4: 0003-gnu-xorg-configuration-file-Expose-the-function.patch --]
[-- Type: text/x-patch, Size: 752 bytes --]

From dc7da1094047ebd47c0491f2a3ec0739362e2637 Mon Sep 17 00:00:00 2001
From: John Soo <jsoo1@asu.edu>
Date: Wed, 1 Apr 2020 08:34:20 -0700
Subject: [PATCH 3/4] gnu: xorg-configuration->file: Expose the function.

* gnu/services/xorg.scm (xorg-configuration->file): Expose it.
---
 gnu/services/xorg.scm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gnu/services/xorg.scm b/gnu/services/xorg.scm
index 1c76049d23..fcd9b4d4f1 100644
--- a/gnu/services/xorg.scm
+++ b/gnu/services/xorg.scm
@@ -66,6 +66,8 @@
             xorg-configuration-server-arguments
 
             xorg-configuration-directory
+            xorg-configuration->file
+
             %default-xorg-modules
             %default-xorg-fonts
             xorg-wrapper
-- 
2.29.2


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #5: 0004-gnu-default-xorg-server-arguments-Expose-variable.patch --]
[-- Type: text/x-patch, Size: 751 bytes --]

From c217dcdb5a40f6b921541ebda19e78fe400ae27f Mon Sep 17 00:00:00 2001
From: John Soo <jsoo1@asu.edu>
Date: Thu, 2 Apr 2020 03:58:07 -0700
Subject: [PATCH 4/4] gnu: %default-xorg-server-arguments: Expose variable.

* gnu/services/xorg.scm (%default-xorg-server-arguments): Expose it.
---
 gnu/services/xorg.scm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gnu/services/xorg.scm b/gnu/services/xorg.scm
index fcd9b4d4f1..6a1d9e07b6 100644
--- a/gnu/services/xorg.scm
+++ b/gnu/services/xorg.scm
@@ -68,6 +68,7 @@
             xorg-configuration-directory
             xorg-configuration->file
 
+            %default-xorg-server-arguments
             %default-xorg-modules
             %default-xorg-fonts
             xorg-wrapper
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#45629] [PATCH] gnu: Export xorg variables for startx.
  2021-01-03 16:27 [bug#45629] [PATCH] gnu: Export xorg variables for startx John Soo
@ 2021-02-11  9:36 ` 宋文武
  2021-02-22  3:39   ` John Soo
  0 siblings, 1 reply; 6+ messages in thread
From: 宋文武 @ 2021-02-11  9:36 UTC (permalink / raw)
  To: John Soo; +Cc: 45629

John Soo <jsoo1@asu.edu> writes:

> Hi Guix,
>
> I wanted to try using startx instead of a display manager.  I finally
> got it to work, but it required at very least these patches.  I also had
> to create a new service type but I think that it needs more work to be
> put in upstream.

Hello, I have added a 'xorg-server-service-type' for this usage.

It can be used with 'sx' (as a normal user), or 'xinit'.

For 'xinit', you need to create a '~/.xserverrc` file with:

  tty=$(tty)
  tty=${tty#/dev/tty}
  X vt${tty}

To use X in PATH and the current tty for rootless X server.


Do thoes patches still needed now?
Thank you!




^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#45629] [PATCH] gnu: Export xorg variables for startx.
  2021-02-11  9:36 ` 宋文武
@ 2021-02-22  3:39   ` John Soo
  2021-02-27  6:43     ` 宋文武
  0 siblings, 1 reply; 6+ messages in thread
From: John Soo @ 2021-02-22  3:39 UTC (permalink / raw)
  To: 宋文武; +Cc: 45629

[-- Attachment #1: Type: text/plain, Size: 168 bytes --]

     Hello!  

  
Are the patches merged? I’m not sure I see the service in master.    If they are, these patches may not be required.   
  

  
Thanks!
     

[-- Attachment #2: Type: text/html, Size: 218 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#45629] [PATCH] gnu: Export xorg variables for startx.
  2021-02-22  3:39   ` John Soo
@ 2021-02-27  6:43     ` 宋文武
  2021-03-03 20:00       ` John Soo
  0 siblings, 1 reply; 6+ messages in thread
From: 宋文武 @ 2021-02-27  6:43 UTC (permalink / raw)
  To: John Soo; +Cc: 45629

John Soo <jsoo1@asu.edu> writes:

> Are the patches merged? I’m not sure I see the service in master.  If they are, these patches may not be required. 

Yes, it's 'xorg-server-service-type' which will put a configured (by
xorg-configuration) X under /run/current-system/profile/bin.

With it, use 'sx' or 'xinit' as a normal user should works.

I haven't document it in the manual though...




^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#45629] [PATCH] gnu: Export xorg variables for startx.
  2021-02-27  6:43     ` 宋文武
@ 2021-03-03 20:00       ` John Soo
  2023-05-07 18:31         ` Maxim Cournoyer
  0 siblings, 1 reply; 6+ messages in thread
From: John Soo @ 2021-03-03 20:00 UTC (permalink / raw)
  To: 宋文武; +Cc: 45629

Hi again,

My setup for startx makes sure X is setgid and sets a few environment
variables for the process.  I am not sure symlinking X to the bin
directory is exactly what I need.

Check out what I do here: https://github.com/jsoo1/dotfiles/blob/release/guix/config.scm

The relevant pieces are the chown-program-service-type, and startx
definition.

Thanks again for improving the startx experience on Guix.

Kindly,

John





^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#45629] [PATCH] gnu: Export xorg variables for startx.
  2021-03-03 20:00       ` John Soo
@ 2023-05-07 18:31         ` Maxim Cournoyer
  0 siblings, 0 replies; 6+ messages in thread
From: Maxim Cournoyer @ 2023-05-07 18:31 UTC (permalink / raw)
  To: John Soo; +Cc: 宋文武, 45629

Hi John,

John Soo <jsoo1@asu.edu> writes:

> Hi again,
>
> My setup for startx makes sure X is setgid and sets a few environment
> variables for the process.  I am not sure symlinking X to the bin
> directory is exactly what I need.

Could you please try and report of any success/problems encountered when
using xorg-server-service-type?  If it works, we can close this issue.

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2023-05-07 18:32 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-03 16:27 [bug#45629] [PATCH] gnu: Export xorg variables for startx John Soo
2021-02-11  9:36 ` 宋文武
2021-02-22  3:39   ` John Soo
2021-02-27  6:43     ` 宋文武
2021-03-03 20:00       ` John Soo
2023-05-07 18:31         ` Maxim Cournoyer

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.