unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Openbox: Add obconf.
@ 2017-01-14 13:14 ng0
  2017-01-14 13:14 ` [PATCH] gnu: " ng0
  2017-01-14 16:58 ` Openbox: " Marius Bakke
  0 siblings, 2 replies; 6+ messages in thread
From: ng0 @ 2017-01-14 13:14 UTC (permalink / raw)
  To: guix-devel

[PATCH] gnu: Add obconf.

I noticed we don't have obconf, the configuration tool for openbox.
Nix creates an wrapper for this application (https://github.com/NixOS/nixpkgs/blob/56904d7c423f2b13b37fbd29f39bbb4b52bc7824/pkgs/tools/X11/obconf/default.nix#L18) I don't know if we'll need this. Running from within openbox it seemed to execute and go through the tabs without crashing.
I haven't tried changing any settings so far.

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

* [PATCH] gnu: Add obconf.
  2017-01-14 13:14 Openbox: Add obconf ng0
@ 2017-01-14 13:14 ` ng0
  2017-01-14 16:58 ` Openbox: " Marius Bakke
  1 sibling, 0 replies; 6+ messages in thread
From: ng0 @ 2017-01-14 13:14 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/openbox.scm (obconf): New variable.
---
 gnu/packages/openbox.scm | 39 ++++++++++++++++++++++++++++++++++++++-
 1 file changed, 38 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/openbox.scm b/gnu/packages/openbox.scm
index 36e39d826..c92f6f834 100644
--- a/gnu/packages/openbox.scm
+++ b/gnu/packages/openbox.scm
@@ -1,6 +1,7 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2014 Julien Lepiller <julien@lepiller.eu>
 ;;; Copyright © 2016 Efraim Flashner <efraim@flashner.co.il>
+;;; Copyright © 2017 ng0 <contact.ng0@cryptolab.net>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -29,7 +30,9 @@
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages python)
   #:use-module (gnu packages xml)
-  #:use-module (gnu packages xorg))
+  #:use-module (gnu packages xorg)
+  #:use-module (gnu packages xdisorg)
+  #:use-module (gnu packages gettext))
 
 (define-public openbox
   (package
@@ -67,4 +70,38 @@ implementations.")
     (home-page "http://openbox.org/wiki/Main_Page")
     (license gpl2+)))
 
+(define-public obconf
+  (package
+    (name "obconf")
+    (version "2.0.4")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "http://openbox.org/dist/" name
+                           "/" name "-" version ".tar.gz"))
+       (sha256
+        (base32
+         "1fanjdmd8727kk74x5404vi8v7s4kpq48l583d12fsi4xvsfb8vi"))))
+    (inputs
+     `(("gtk+-2" ,gtk+-2)
+       ("imlib2" ,imlib2)
+       ("libglade" ,libglade)
+       ("openbox" ,openbox)
+       ("startup-notification" ,startup-notification)
+       ("libsm" ,libsm)
+       ("librsvg" ,librsvg)
+       ("libxft" ,libxft)))
+    (native-inputs
+     `(("gettext" ,gettext-minimal)
+       ("pkg-config" ,pkg-config)))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:configure-flags (list "--enable-nls")))
+    (home-page "http://openbox.org/wiki/ObConf:About")
+    (synopsis "Openbox configuration tool")
+    (description
+     "Obconf is a tool for configuring the Openbox window manager.
+You can configure its appearance, themes, and much more.")
+    (license gpl2+)))
+
 ;;; openbox.scm ends here
-- 
2.11.0

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

* Re: Openbox: Add obconf.
  2017-01-14 13:14 Openbox: Add obconf ng0
  2017-01-14 13:14 ` [PATCH] gnu: " ng0
@ 2017-01-14 16:58 ` Marius Bakke
  2017-01-14 18:43   ` ng0
  1 sibling, 1 reply; 6+ messages in thread
From: Marius Bakke @ 2017-01-14 16:58 UTC (permalink / raw)
  To: ng0, guix-devel

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

ng0 <ng0@libertad.pw> writes:

> [PATCH] gnu: Add obconf.
>
> I noticed we don't have obconf, the configuration tool for openbox.
> Nix creates an wrapper for this application (https://github.com/NixOS/nixpkgs/blob/56904d7c423f2b13b37fbd29f39bbb4b52bc7824/pkgs/tools/X11/obconf/default.nix#L18) I don't know if we'll need this. Running from within openbox it seemed to execute and go through the tabs without crashing.
> I haven't tried changing any settings so far.

Could you try changing some stuff and make sure it works? IIRC obconf
does a lot of funky XML stuff which may require some patching or
wrapping.

I don't want to add a package just to get a bug report about it not
working :-)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

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

* Re: Openbox: Add obconf.
  2017-01-14 16:58 ` Openbox: " Marius Bakke
@ 2017-01-14 18:43   ` ng0
  2017-01-14 21:45     ` ng0
  0 siblings, 1 reply; 6+ messages in thread
From: ng0 @ 2017-01-14 18:43 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

Marius Bakke <mbakke@fastmail.com> writes:

> ng0 <ng0@libertad.pw> writes:
>
>> [PATCH] gnu: Add obconf.
>>
>> I noticed we don't have obconf, the configuration tool for openbox.
>> Nix creates an wrapper for this application (https://github.com/NixOS/nixpkgs/blob/56904d7c423f2b13b37fbd29f39bbb4b52bc7824/pkgs/tools/X11/obconf/default.nix#L18) I don't know if we'll need this. Running from within openbox it seemed to execute and go through the tabs without crashing.
>> I haven't tried changing any settings so far.
>
> Could you try changing some stuff and make sure it works? IIRC obconf
> does a lot of funky XML stuff which may require some patching or
> wrapping.
>
> I don't want to add a package just to get a bug report about it not
> working :-)

Okay, I'm currently reconfiguring a system with this branch,
might take a really long time.
-- 
♥Ⓐ  ng0 -- https://www.inventati.org/patternsinthechaos/

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

* Re: Openbox: Add obconf.
  2017-01-14 18:43   ` ng0
@ 2017-01-14 21:45     ` ng0
  2017-01-15 19:13       ` Marius Bakke
  0 siblings, 1 reply; 6+ messages in thread
From: ng0 @ 2017-01-14 21:45 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

ng0 <ng0@libertad.pw> writes:

> Marius Bakke <mbakke@fastmail.com> writes:
>
>> ng0 <ng0@libertad.pw> writes:
>>
>>> [PATCH] gnu: Add obconf.
>>>
>>> I noticed we don't have obconf, the configuration tool for openbox.
>>> Nix creates an wrapper for this application (https://github.com/NixOS/nixpkgs/blob/56904d7c423f2b13b37fbd29f39bbb4b52bc7824/pkgs/tools/X11/obconf/default.nix#L18) I don't know if we'll need this. Running from within openbox it seemed to execute and go through the tabs without crashing.
>>> I haven't tried changing any settings so far.
>>
>> Could you try changing some stuff and make sure it works? IIRC obconf
>> does a lot of funky XML stuff which may require some patching or
>> wrapping.
>>
>> I don't want to add a package just to get a bug report about it not
>> working :-)
>
> Okay, I'm currently reconfiguring a system with this branch,
> might take a really long time.

Done, I can confirm that changing settings works.

-- 
♥Ⓐ  ng0 -- https://www.inventati.org/patternsinthechaos/

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

* Re: Openbox: Add obconf.
  2017-01-14 21:45     ` ng0
@ 2017-01-15 19:13       ` Marius Bakke
  0 siblings, 0 replies; 6+ messages in thread
From: Marius Bakke @ 2017-01-15 19:13 UTC (permalink / raw)
  To: ng0; +Cc: guix-devel

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

ng0 <ng0@libertad.pw> writes:

> ng0 <ng0@libertad.pw> writes:
>
>> Marius Bakke <mbakke@fastmail.com> writes:
>>
>>> ng0 <ng0@libertad.pw> writes:
>>>
>>>> [PATCH] gnu: Add obconf.
>>>>
>>>> I noticed we don't have obconf, the configuration tool for openbox.
>>>> Nix creates an wrapper for this application (https://github.com/NixOS/nixpkgs/blob/56904d7c423f2b13b37fbd29f39bbb4b52bc7824/pkgs/tools/X11/obconf/default.nix#L18) I don't know if we'll need this. Running from within openbox it seemed to execute and go through the tabs without crashing.
>>>> I haven't tried changing any settings so far.
>>>
>>> Could you try changing some stuff and make sure it works? IIRC obconf
>>> does a lot of funky XML stuff which may require some patching or
>>> wrapping.
>>>
>>> I don't want to add a package just to get a bug report about it not
>>> working :-)
>>
>> Okay, I'm currently reconfiguring a system with this branch,
>> might take a really long time.
>
> Done, I can confirm that changing settings works.

Cool, thanks for checking! Pushed after sorting module imports.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

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

end of thread, other threads:[~2017-01-15 19:13 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-14 13:14 Openbox: Add obconf ng0
2017-01-14 13:14 ` [PATCH] gnu: " ng0
2017-01-14 16:58 ` Openbox: " Marius Bakke
2017-01-14 18:43   ` ng0
2017-01-14 21:45     ` ng0
2017-01-15 19:13       ` Marius Bakke

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).