Re: [PATCH] gnu: Add qscintilla.

2016-10-03 Thread Ludovic Courtès
Kei Kebreau <k...@openmailbox.org> skribis: > From ae862b8a51d15f165dc9b40dc46c2e209736f3a8 Mon Sep 17 00:00:00 2001 > From: Kei Kebreau <k...@openmailbox.org> > Date: Mon, 19 Sep 2016 10:11:20 -0400 > Subject: [PATCH] gnu: Add qscintilla-qt4. > > * gnu/packages/m

Re: [PATCH] gnu: Add qscintilla.

2016-09-25 Thread Kei Kebreau
Leo Famulari writes: > On Sat, Sep 24, 2016 at 01:13:33AM -0400, Kei Kebreau wrote: >> Leo Famulari writes: >> > I was getting failures reminiscent of an ABI break. But, I can't >> > reproduce it now. >> > >> > Kei, maybe you should try applying the

Re: [PATCH] gnu: Add qscintilla.

2016-09-25 Thread Leo Famulari
On Sat, Sep 24, 2016 at 01:13:33AM -0400, Kei Kebreau wrote: > Leo Famulari writes: > > I was getting failures reminiscent of an ABI break. But, I can't > > reproduce it now. > > > > Kei, maybe you should try applying the patches to a fresh Git checkout? > > If they work like

Re: [PATCH] gnu: Add qscintilla.

2016-09-23 Thread Leo Famulari
On Thu, Sep 22, 2016 at 09:02:20PM +0200, Ricardo Wurmus wrote: > > Leo Famulari writes: > > > I'm not sure yet about the other problem. I tried to define 'hi', which > > inherits from 'hello', and I got a similar error. > > I don’t understand this. What is the problem

Re: [PATCH] gnu: Add qscintilla.

2016-09-22 Thread Ricardo Wurmus
Leo Famulari writes: > I'm not sure yet about the other problem. I tried to define 'hi', which > inherits from 'hello', and I got a similar error. I don’t understand this. What is the problem here? I’m successfully inheriting from packages in other modules. For example,

Re: [PATCH] gnu: Add qscintilla.

2016-09-19 Thread Leo Famulari
On Mon, Sep 19, 2016 at 05:45:35PM -0400, Kei Kebreau wrote: > * gnu/packages/qt.scm (qscintilla): New variable. > + (uri (string-append > "mirror://sourceforge/QScintilla2/QScintilla-" > + version "/QScintilla_gpl-" version > ".tar.gz")) This URI

Re: [PATCH] gnu: Add qscintilla.

2016-09-19 Thread Kei Kebreau
From: Kei Kebreau <k...@openmailbox.org> Date: Mon, 19 Sep 2016 10:09:33 -0400 Subject: [PATCH] gnu: Add qscintilla. * gnu/packages/qt.scm (qscintilla): New variable. --- gnu/packages/qt.scm | 43 +++ 1 file changed, 43 insertions(+) diff --git

Re: [PATCH] gnu: Add qscintilla.

2016-09-19 Thread Leo Famulari
On Mon, Sep 19, 2016 at 10:47:56AM -0400, Kei Kebreau wrote: > Using > > #:use-module (gnu packages qt) > > and > > (define qscintilla-qt4 > (package > (inherit qscintilla) > ...)) > > gives me an unbound variable error. Any idea what's going wrong here? Can you send the

Re: [PATCH] gnu: Add qscintilla.

2016-09-19 Thread Kei Kebreau
Ricardo Wurmus writes: > Kei Kebreau writes: > >> (how do I inherit a package from another module?) > > Just like inheriting from any other package: you declare that the module > containing the package variable be used, and then inherit from the >

Re: [PATCH] gnu: Add qscintilla.

2016-09-19 Thread Andreas Enge
On Mon, Sep 19, 2016 at 09:18:45AM -0400, Kei Kebreau wrote: > To get this right: qscintilla will be defined publicly in qt.scm, > qscintilla-qt4 will be defined privately in maths.scm (how do I inherit > a package from another module?), and Octave will include qscintilla-qt4 > and qt-4 as inputs.

Re: [PATCH] gnu: Add qscintilla.

2016-09-19 Thread Ricardo Wurmus
Kei Kebreau writes: > (how do I inherit a package from another module?) Just like inheriting from any other package: you declare that the module containing the package variable be used, and then inherit from the package variable. ~~ Ricardo

Re: [PATCH] gnu: Add qscintilla.

2016-09-19 Thread Kei Kebreau
Efraim Flashner writes: > On Sun, Sep 18, 2016 at 04:56:34PM -0400, Leo Famulari wrote: >> On Sun, Sep 18, 2016 at 04:10:15PM -0400, Kei Kebreau wrote: >> > Andreas Enge writes: >> > > On Tue, Sep 13, 2016 at 04:37:35PM -0400, Kei Kebreau wrote: >> > >>

Re: [PATCH] gnu: Add qscintilla.

2016-09-18 Thread Efraim Flashner
On Sun, Sep 18, 2016 at 04:56:34PM -0400, Leo Famulari wrote: > On Sun, Sep 18, 2016 at 04:10:15PM -0400, Kei Kebreau wrote: > > Andreas Enge writes: > > > On Tue, Sep 13, 2016 at 04:37:35PM -0400, Kei Kebreau wrote: > > >> In this case, should I leave qtscintilla-qt4 as a public

Re: [PATCH] gnu: Add qscintilla.

2016-09-18 Thread Leo Famulari
On Sun, Sep 18, 2016 at 04:10:15PM -0400, Kei Kebreau wrote: > Andreas Enge writes: > > On Tue, Sep 13, 2016 at 04:37:35PM -0400, Kei Kebreau wrote: > >> In this case, should I leave qtscintilla-qt4 as a public package in qt.scm > >> instead of maths.scm as Leo suggested? > > > >

Re: [PATCH] gnu: Add qscintilla.

2016-09-18 Thread Kei Kebreau
Andreas Enge writes: > Hello, > > On Tue, Sep 13, 2016 at 04:37:35PM -0400, Kei Kebreau wrote: >> In this case, should I leave qtscintilla-qt4 as a public package in qt.scm >> instead of maths.scm as Leo suggested? > > since it is used for only one package and relies on the

Re: [PATCH] gnu: Add qscintilla.

2016-09-18 Thread Andreas Enge
Hello, On Tue, Sep 13, 2016 at 04:37:35PM -0400, Kei Kebreau wrote: > In this case, should I leave qtscintilla-qt4 as a public package in qt.scm > instead of maths.scm as Leo suggested? since it is used for only one package and relies on the deprecated qt@4, I would leave it private, regardless

Re: [PATCH] gnu: Add qscintilla.

2016-09-15 Thread Kei Kebreau
Kei Kebreau writes: > Efraim Flashner writes: > >> On September 14, 2016 11:37:46 PM GMT+03:00, Kei Kebreau >> wrote: >>>Leo Famulari writes: >>> On Wed, Sep 14, 2016 at 02:11:44PM -0400, Kei Kebreau

Re: [PATCH] gnu: Add qscintilla.

2016-09-14 Thread Efraim Flashner
On September 14, 2016 11:37:46 PM GMT+03:00, Kei Kebreau wrote: >Leo Famulari writes: > >> On Wed, Sep 14, 2016 at 02:11:44PM -0400, Kei Kebreau wrote: >>> Leo Famulari writes: >>> >>> > On Tue, Sep 13, 2016 at 08:48:43PM +,

Re: [PATCH] gnu: Add qscintilla.

2016-09-14 Thread Kei Kebreau
qscintilla-qt4, >> Enable Octave's Qt 4 GUI." :) > > The patches "Add qscintilla" and "Add qscintilla-qt4" both add both > packages. Fixed it. From cb95955eaea1210af6ef197f4157583f24cf4eac Mon Sep 17 00:00:00 2001 From: Kei Kebreau <k...@openmailbox.org>

Re: [PATCH] gnu: Add qscintilla.

2016-09-14 Thread Leo Famulari
On Wed, Sep 14, 2016 at 02:11:44PM -0400, Kei Kebreau wrote: > Leo Famulari writes: > > > On Tue, Sep 13, 2016 at 08:48:43PM +, Efraim Flashner wrote: > >> On September 13, 2016 11:37:35 PM GMT+03:00, Kei Kebreau > >> wrote: > >> >In this case,

Re: [PATCH] gnu: Add qscintilla.

2016-09-14 Thread Kei Kebreau
order of application is "Add qscintilla, Add qscintilla-qt4, Enable Octave's Qt 4 GUI." :) From 9304e67ff085853ebbd5d7f1c976171de498bf5b Mon Sep 17 00:00:00 2001 From: Kei Kebreau <k...@openmailbox.org> Date: Wed, 14 Sep 2016 13:56:33 -0400 Subject: [PATCH] gnu: Add qscintilla. * gnu/packages/qt.scm (qscintilla): N

Re: [PATCH] gnu: Add qscintilla.

2016-09-14 Thread Leo Famulari
On Tue, Sep 13, 2016 at 08:48:43PM +, Efraim Flashner wrote: > On September 13, 2016 11:37:35 PM GMT+03:00, Kei Kebreau > wrote: > >In this case, should I leave qtscintilla-qt4 as a public package in > >qt.scm > >instead of maths.scm as Leo suggested? > > That's where

Re: [PATCH] gnu: Add qscintilla.

2016-09-13 Thread Efraim Flashner
On September 13, 2016 11:37:35 PM GMT+03:00, Kei Kebreau wrote: >Efraim Flashner writes: > >> On Tue, Sep 13, 2016 at 02:01:33PM -0400, Kei Kebreau wrote: >>> Leo Famulari writes: >>> >>> > On Tue, Sep 13, 2016 at 10:37:56AM

Re: [PATCH] gnu: Add qscintilla.

2016-09-13 Thread Kei Kebreau
Efraim Flashner writes: > On Tue, Sep 13, 2016 at 02:01:33PM -0400, Kei Kebreau wrote: >> Leo Famulari writes: >> >> > On Tue, Sep 13, 2016 at 10:37:56AM -0400, Kei Kebreau wrote: >> >> Kei Kebreau writes: >> >> > A component

Re: [PATCH] gnu: Add qscintilla.

2016-09-13 Thread Kei Kebreau
Leo Famulari writes: > On Tue, Sep 13, 2016 at 02:59:31PM -0400, Kei Kebreau wrote: >> Leo Famulari writes: >> >> > On Tue, Sep 13, 2016 at 02:01:33PM -0400, Kei Kebreau wrote: >> >> Like below? And how could I then access qscintilla-for-octave from >>

Re: [PATCH] gnu: Add qscintilla.

2016-09-13 Thread Efraim Flashner
On Tue, Sep 13, 2016 at 02:01:33PM -0400, Kei Kebreau wrote: > Leo Famulari writes: > > > On Tue, Sep 13, 2016 at 10:37:56AM -0400, Kei Kebreau wrote: > >> Kei Kebreau writes: > >> > A component necessary for GNU Octave's GUI. > >> > How does it look? >

Re: [PATCH] gnu: Add qscintilla.

2016-09-13 Thread Leo Famulari
On Tue, Sep 13, 2016 at 02:59:31PM -0400, Kei Kebreau wrote: > Leo Famulari writes: > > > On Tue, Sep 13, 2016 at 02:01:33PM -0400, Kei Kebreau wrote: > >> Like below? And how could I then access qscintilla-for-octave from > >> maths.scm if it isn't defined publicly? > > > >

Re: [PATCH] gnu: Add qscintilla.

2016-09-13 Thread Leo Famulari
On Tue, Sep 13, 2016 at 02:01:33PM -0400, Kei Kebreau wrote: > Like below? And how could I then access qscintilla-for-octave from > maths.scm if it isn't defined publicly? Yes, that looks right. But I would put qscintilla-for-octave in maths.scm to avoid the issue you describe. signature.asc

Re: [PATCH] gnu: Add qscintilla.

2016-09-13 Thread Kei Kebreau
Leo Famulari writes: > On Tue, Sep 13, 2016 at 02:01:33PM -0400, Kei Kebreau wrote: >> Like below? And how could I then access qscintilla-for-octave from >> maths.scm if it isn't defined publicly? > > Yes, that looks right. But I would put qscintilla-for-octave in > maths.scm

Re: [PATCH] gnu: Add qscintilla.

2016-09-13 Thread Kei Kebreau
Leo Famulari writes: > On Tue, Sep 13, 2016 at 10:37:56AM -0400, Kei Kebreau wrote: >> Kei Kebreau writes: >> > A component necessary for GNU Octave's GUI. >> > How does it look? >> Not sure how packaging Qt packages goes, but I've discovered that >>

Re: [PATCH] gnu: Add qscintilla.

2016-09-13 Thread Leo Famulari
On Tue, Sep 13, 2016 at 10:37:56AM -0400, Kei Kebreau wrote: > Kei Kebreau writes: > > A component necessary for GNU Octave's GUI. > > How does it look? > Not sure how packaging Qt packages goes, but I've discovered that > GNU Octave's GUI only builds with Qt4 support. I've

Re: [PATCH] gnu: Add qscintilla.

2016-09-13 Thread Kei Kebreau
he old patch as is and add a qt4 package that inherits from it? From 51da032547ed3e1c2b7825b35b1f39a5e922b44b Mon Sep 17 00:00:00 2001 From: Kei Kebreau <k...@openmailbox.org> Date: Tue, 13 Sep 2016 10:34:59 -0400 Subject: [PATCH] gnu: Add qscintilla. * gnu/packages/qt.scm (qscintilla): New

[PATCH] gnu: Add qscintilla.

2016-09-12 Thread Kei Kebreau
A component necessary for GNU Octave's GUI. How does it look? From c6f182688ab5bd59aeca2e29d67bf527df869963 Mon Sep 17 00:00:00 2001 From: Kei Kebreau <k...@openmailbox.org> Date: Mon, 12 Sep 2016 13:27:05 -0400 Subject: [PATCH] gnu: Add qscintilla. * gnu/packages/qt.scm (qscintilla