unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
@ 2016-10-25 17:46 Kei Kebreau
  2016-10-26  3:07 ` Marius Bakke
  0 siblings, 1 reply; 7+ messages in thread
From: Kei Kebreau @ 2016-10-25 17:46 UTC (permalink / raw)
  To: guix-devel


[-- Attachment #1.1: Type: text/plain, Size: 99 bytes --]

This patch allows Window Maker to run wmsetbg properly instead of
showing an error as it has been.

[-- Attachment #1.2: 0001-gnu-windowmaker-Fix-invocation-of-wmsetbg.patch --]
[-- Type: text/plain, Size: 1510 bytes --]

From a113116da6f9f6574e9fde12e69a9d3acdc87769 Mon Sep 17 00:00:00 2001
From: Kei Kebreau <kei@openmailbox.org>
Date: Sun, 23 Oct 2016 13:50:05 -0400
Subject: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.

* gnu/packages/gnustep.scm (windowmaker)[arguments]: Add substitution of
40-character limit with a 107-character limit.
---
 gnu/packages/gnustep.scm | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/gnustep.scm b/gnu/packages/gnustep.scm
index 6c365cb..8f72bb3 100644
--- a/gnu/packages/gnustep.scm
+++ b/gnu/packages/gnustep.scm
@@ -60,7 +60,13 @@
                         (string-append "\"" bin "/wmaker.inst")))
                      (substitute* '("src/defaults.c" "WPrefs.app/Menu.c")
                        (("\"wmsetbg")
-                        (string-append "\"" bin "/wmsetbg")))))
+                        (string-append "\"" bin "/wmsetbg")))
+                     ;; Add enough cells to the command character array to
+                     ;; allow passing our large path to the wmsetbg binary.
+                     ;; The path to wmsetbg in Guix requires 67 extra characters.
+                     (substitute* "src/defaults.c"
+                       (("len = strlen\\(text\\) \\+ 40;")
+                        (string-append "len = strlen(text) + 107;")))))
                  (alist-cons-after
                   'install 'wrap
                   (lambda* (#:key outputs #:allow-other-keys)
-- 
2.10.1


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

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

* Re: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
  2016-10-25 17:46 [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg' Kei Kebreau
@ 2016-10-26  3:07 ` Marius Bakke
  2016-10-26  4:14   ` Kei Kebreau
  0 siblings, 1 reply; 7+ messages in thread
From: Marius Bakke @ 2016-10-26  3:07 UTC (permalink / raw)
  To: Kei Kebreau, guix-devel

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

Kei Kebreau <kei@openmailbox.org> writes:

> This patch allows Window Maker to run wmsetbg properly instead of
> showing an error as it has been.
> From a113116da6f9f6574e9fde12e69a9d3acdc87769 Mon Sep 17 00:00:00 2001
> From: Kei Kebreau <kei@openmailbox.org>
> Date: Sun, 23 Oct 2016 13:50:05 -0400
> Subject: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
>
> * gnu/packages/gnustep.scm (windowmaker)[arguments]: Add substitution of
> 40-character limit with a 107-character limit.
> ---
>  gnu/packages/gnustep.scm | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)

I haven't tested this, but the diff looks reasonable. Good catch. Any
chance you can submit this upstream as well?

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

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

* Re: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
  2016-10-26  3:07 ` Marius Bakke
@ 2016-10-26  4:14   ` Kei Kebreau
  2016-12-09 21:09     ` Kei Kebreau
  2016-12-10  1:02     ` Leo Famulari
  0 siblings, 2 replies; 7+ messages in thread
From: Kei Kebreau @ 2016-10-26  4:14 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

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

Marius Bakke <mbakke@fastmail.com> writes:

> Kei Kebreau <kei@openmailbox.org> writes:
>
>> This patch allows Window Maker to run wmsetbg properly instead of
>> showing an error as it has been.
>> From a113116da6f9f6574e9fde12e69a9d3acdc87769 Mon Sep 17 00:00:00 2001
>> From: Kei Kebreau <kei@openmailbox.org>
>> Date: Sun, 23 Oct 2016 13:50:05 -0400
>> Subject: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
>>
>> * gnu/packages/gnustep.scm (windowmaker)[arguments]: Add substitution of
>> 40-character limit with a 107-character limit.
>> ---
>>  gnu/packages/gnustep.scm | 8 +++++++-
>>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> I haven't tested this, but the diff looks reasonable. Good catch. Any
> chance you can submit this upstream as well?

I could bring it up, but I'm not so sure that it would be a change worth
considering. The Window Maker mailing list is very low volume, and no
one seems to have complained about this.

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

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

* Re: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
  2016-10-26  4:14   ` Kei Kebreau
@ 2016-12-09 21:09     ` Kei Kebreau
  2016-12-09 21:33       ` Marius Bakke
  2016-12-10  1:02     ` Leo Famulari
  1 sibling, 1 reply; 7+ messages in thread
From: Kei Kebreau @ 2016-12-09 21:09 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

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

Kei Kebreau <kei@openmailbox.org> writes:

> Marius Bakke <mbakke@fastmail.com> writes:
>
>> Kei Kebreau <kei@openmailbox.org> writes:
>>
>>> This patch allows Window Maker to run wmsetbg properly instead of
>>> showing an error as it has been.
>>> From a113116da6f9f6574e9fde12e69a9d3acdc87769 Mon Sep 17 00:00:00 2001
>>> From: Kei Kebreau <kei@openmailbox.org>
>>> Date: Sun, 23 Oct 2016 13:50:05 -0400
>>> Subject: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
>>>
>>> * gnu/packages/gnustep.scm (windowmaker)[arguments]: Add substitution of
>>> 40-character limit with a 107-character limit.
>>> ---
>>>  gnu/packages/gnustep.scm | 8 +++++++-
>>>  1 file changed, 7 insertions(+), 1 deletion(-)
>>
>> I haven't tested this, but the diff looks reasonable. Good catch. Any
>> chance you can submit this upstream as well?
>
> I could bring it up, but I'm not so sure that it would be a change worth
> considering. The Window Maker mailing list is very low volume, and no
> one seems to have complained about this.

If it's okay with you, I'd like to push this change. I've heard nothing
from upstream yet.

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

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

* Re: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
  2016-12-09 21:09     ` Kei Kebreau
@ 2016-12-09 21:33       ` Marius Bakke
  2016-12-10  3:12         ` Kei Kebreau
  0 siblings, 1 reply; 7+ messages in thread
From: Marius Bakke @ 2016-12-09 21:33 UTC (permalink / raw)
  To: Kei Kebreau; +Cc: guix-devel

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

Kei Kebreau <kei@openmailbox.org> writes:

> Kei Kebreau <kei@openmailbox.org> writes:
>
>> Marius Bakke <mbakke@fastmail.com> writes:
>>
>>> Kei Kebreau <kei@openmailbox.org> writes:
>>>
>>>> This patch allows Window Maker to run wmsetbg properly instead of
>>>> showing an error as it has been.
>>>> From a113116da6f9f6574e9fde12e69a9d3acdc87769 Mon Sep 17 00:00:00 2001
>>>> From: Kei Kebreau <kei@openmailbox.org>
>>>> Date: Sun, 23 Oct 2016 13:50:05 -0400
>>>> Subject: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
>>>>
>>>> * gnu/packages/gnustep.scm (windowmaker)[arguments]: Add substitution of
>>>> 40-character limit with a 107-character limit.
>>>> ---
>>>>  gnu/packages/gnustep.scm | 8 +++++++-
>>>>  1 file changed, 7 insertions(+), 1 deletion(-)
>>>
>>> I haven't tested this, but the diff looks reasonable. Good catch. Any
>>> chance you can submit this upstream as well?
>>
>> I could bring it up, but I'm not so sure that it would be a change worth
>> considering. The Window Maker mailing list is very low volume, and no
>> one seems to have complained about this.
>
> If it's okay with you, I'd like to push this change. I've heard nothing
> from upstream yet.

Yes of course. I thought it was already pushed, sorry for holding it up!

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

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

* Re: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
  2016-10-26  4:14   ` Kei Kebreau
  2016-12-09 21:09     ` Kei Kebreau
@ 2016-12-10  1:02     ` Leo Famulari
  1 sibling, 0 replies; 7+ messages in thread
From: Leo Famulari @ 2016-12-10  1:02 UTC (permalink / raw)
  To: Kei Kebreau; +Cc: guix-devel

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

On Wed, Oct 26, 2016 at 12:14:03AM -0400, Kei Kebreau wrote:
> Marius Bakke <mbakke@fastmail.com> writes:
> 
> > Kei Kebreau <kei@openmailbox.org> writes:
> >
> >> This patch allows Window Maker to run wmsetbg properly instead of
> >> showing an error as it has been.
> >> From a113116da6f9f6574e9fde12e69a9d3acdc87769 Mon Sep 17 00:00:00 2001
> >> From: Kei Kebreau <kei@openmailbox.org>
> >> Date: Sun, 23 Oct 2016 13:50:05 -0400
> >> Subject: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
> >>
> >> * gnu/packages/gnustep.scm (windowmaker)[arguments]: Add substitution of
> >> 40-character limit with a 107-character limit.
> >> ---
> >>  gnu/packages/gnustep.scm | 8 +++++++-
> >>  1 file changed, 7 insertions(+), 1 deletion(-)
> >
> > I haven't tested this, but the diff looks reasonable. Good catch. Any
> > chance you can submit this upstream as well?
> 
> I could bring it up, but I'm not so sure that it would be a change worth
> considering. The Window Maker mailing list is very low volume, and no
> one seems to have complained about this.

I think it's a safe change, but I also think we should always let
upstream know when we are modifying string-handling code in C.

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

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

* Re: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
  2016-12-09 21:33       ` Marius Bakke
@ 2016-12-10  3:12         ` Kei Kebreau
  0 siblings, 0 replies; 7+ messages in thread
From: Kei Kebreau @ 2016-12-10  3:12 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

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

Marius Bakke <mbakke@fastmail.com> writes:

> Kei Kebreau <kei@openmailbox.org> writes:
>
>> Kei Kebreau <kei@openmailbox.org> writes:
>>
>>> Marius Bakke <mbakke@fastmail.com> writes:
>>>
>>>> Kei Kebreau <kei@openmailbox.org> writes:
>>>>
>>>>> This patch allows Window Maker to run wmsetbg properly instead of
>>>>> showing an error as it has been.
>>>>> From a113116da6f9f6574e9fde12e69a9d3acdc87769 Mon Sep 17 00:00:00 2001
>>>>> From: Kei Kebreau <kei@openmailbox.org>
>>>>> Date: Sun, 23 Oct 2016 13:50:05 -0400
>>>>> Subject: [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg'.
>>>>>
>>>>> * gnu/packages/gnustep.scm (windowmaker)[arguments]: Add substitution of
>>>>> 40-character limit with a 107-character limit.
>>>>> ---
>>>>>  gnu/packages/gnustep.scm | 8 +++++++-
>>>>>  1 file changed, 7 insertions(+), 1 deletion(-)
>>>>
>>>> I haven't tested this, but the diff looks reasonable. Good catch. Any
>>>> chance you can submit this upstream as well?
>>>
>>> I could bring it up, but I'm not so sure that it would be a change worth
>>> considering. The Window Maker mailing list is very low volume, and no
>>> one seems to have complained about this.
>>
>> If it's okay with you, I'd like to push this change. I've heard nothing
>> from upstream yet.
>
> Yes of course. I thought it was already pushed, sorry for holding it up!

No need to apologize! I misread the "as well" in your message as a
requirement before pushing. Pushed as fdb552bbd44e35d335ee1d430ffe5b0fc4bc237e.

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

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

end of thread, other threads:[~2016-12-10  3:12 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-25 17:46 [PATCH] gnu: windowmaker: Fix invocation of 'wmsetbg' Kei Kebreau
2016-10-26  3:07 ` Marius Bakke
2016-10-26  4:14   ` Kei Kebreau
2016-12-09 21:09     ` Kei Kebreau
2016-12-09 21:33       ` Marius Bakke
2016-12-10  3:12         ` Kei Kebreau
2016-12-10  1:02     ` Leo Famulari

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