Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2017-01-15 Thread Mark H Weaver
John Darrington writes: > On Sat, Dec 24, 2016 at 10:39:40AM -0500, Mark H Weaver wrote: > John Darrington writes: > > > We can argue about this till we're blue in the face. > > > > But on a pragmatic level,

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-24 Thread John Darrington
On Sat, Dec 24, 2016 at 10:39:40AM -0500, Mark H Weaver wrote: John Darrington writes: > We can argue about this till we're blue in the face. > > But on a pragmatic level, Mark's question demonstrates perfectly > that our current system

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-24 Thread Mark H Weaver
John Darrington writes: > We can argue about this till we're blue in the face. > > But on a pragmatic level, Mark's question demonstrates perfectly > that our current system is lacking. No it doesn't. Our convention, taken from the GNU coding standards, is that

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-22 Thread John Darrington
We can argue about this till we're blue in the face. But on a pragmatic level, Mark's question demonstrates perfectly that our current system is lacking. Other projects I work on which have a more conventional approach do not suffer from this problem. J' On Thu, Dec 22, 2016 at 06:56:56AM

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-21 Thread Tobias Geerinckx-Rice
John, Danny, [Any exasperation is due only to the sustained level of FUD I encounter about the Guix/GNU changelog format, and not aimed at John.] On 20/12/16 12:03, John Darrington wrote: > Sure (I would like to see a convention where such explanations are > put in the commit messaage, but I

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-21 Thread John Darrington
On Wed, Dec 21, 2016 at 09:36:56AM +0100, Danny Milosavljevic wrote: > Sure (I would like to see a convention where such explanations are > put in the commit messaage, but I have previously been outvoted on > that issue): Hi Danny, A small request: Can you please fold the text of

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-21 Thread Danny Milosavljevic
> Sure (I would like to see a convention where such explanations are > put in the commit messaage, but I have previously been outvoted on > that issue): No, please don't put explanations into the commit message. But do put them into the source code as a comment. I'm also working on other

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-20 Thread John Darrington
On Tue, Dec 20, 2016 at 12:03:31PM +0100, John Darrington wrote: On Tue, Dec 20, 2016 at 02:17:42AM -0500, Mark H Weaver wrote: The scheme code contains a number of procedures similar to (load-extension "libguile-ncurses" "func"). We need the first string to contain the

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-20 Thread John Darrington
On Tue, Dec 20, 2016 at 02:17:42AM -0500, Mark H Weaver wrote: Hi John, John Darrington writes: > * gnu/packages/guile.scm (guile-ncurses) [arguments]: Install shared object before > attempting to build the package. Patch load-extension path before

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-19 Thread Mark H Weaver
Hi John, John Darrington writes: > * gnu/packages/guile.scm (guile-ncurses) [arguments]: Install shared object > before > attempting to build the package. Patch load-extension path before building > instead > of after. The first sentence above is mistaken or misleading:

Re: [PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-19 Thread Ludovic Courtès
John Darrington skribis: > * gnu/packages/guile.scm (guile-ncurses) [arguments]: Install shared object > before > attempting to build the package. Patch load-extension path before building > instead > of after. OK, thank you! Ludo’.

[PATCH] gnu: Fix load-extension path in packaging of guile-ncurses.

2016-12-19 Thread John Darrington
* gnu/packages/guile.scm (guile-ncurses) [arguments]: Install shared object before attempting to build the package. Patch load-extension path before building instead of after. --- gnu/packages/guile.scm | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git