[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add qscintilla.
From: |
Kei Kebreau |
Subject: |
Re: [PATCH] gnu: Add qscintilla. |
Date: |
Tue, 13 Sep 2016 16:37:35 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Efraim Flashner <address@hidden> writes:
> On Tue, Sep 13, 2016 at 02:01:33PM -0400, Kei Kebreau wrote:
>> Leo Famulari <address@hidden> writes:
>>
>> > On Tue, Sep 13, 2016 at 10:37:56AM -0400, Kei Kebreau wrote:
>> >> Kei Kebreau <address@hidden> 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 changed the patch to
>> >> adjust this. Should I leave the old patch as is and add a qt4 package
>> >> that inherits from it?
>> >
>> > Since Qt 4 is no longer supported upstream, we are trying to remove
>> > users of the qt-4 package so that we can eventually remove the qt-4
>> > package itself.
>> >
>> > With that in mind, how about a qscintilla-for-octave package? This
>> > package can inherit from a qscintilla that uses qtbase, and I think it
>> > should be declared privately [using (define) instead of (define-public].
>> >
>> >> + (replace 'configure
>> >> + (lambda _
>> >> + (chdir "Qt4Qt5")
>> >> + (zero? (system* "qmake" "qscintilla.pro"))))
>> >
>> > I would change directory in a separate 'chdir' phase.
>> >
>> >> + (substitute* (find-files "." "Makefile")
>> >> + (((string-append "INSTALL_ROOT)" qt))
>> >> + (string-append "INSTALL_ROOT)" out)))))))))
>> > ^
>> > Inconsistent indentation. Also, this phase should return #t, since the
>> > return value of substitute* is unspecified.
>> >
>> >> + (synopsis "Qt5 port of the Scintilla editing component")
>> >
>> > Make sure to adjust the Qt name as appropriate :)
>>
>> Like below? And how could I then access qscintilla-for-octave from
>> maths.scm if it isn't defined publicly?
>>
>> (define-public qscintilla
>> (package
>> (name "qscintilla")
>> (version "2.9.3")
>> (source (origin
>> (method url-fetch)
>> (uri (string-append
>> "mirror://sourceforge/QScintilla2/QScintilla-"
>> version "/QScintilla_gpl-" version
>> ".tar.gz"))
>> (sha256
>> (base32
>> "0znvdncpj64zcpbkyvj11dm8bdc3nfn5girggj33ammhfcyvkalq"))))
>> (build-system gnu-build-system)
>> (arguments
>> `(#:phases
>> (modify-phases %standard-phases
>> (add-before 'configure 'change-directory
>> (lambda _ (chdir "Qt4Qt5") #t))
>> (replace 'configure
>> (lambda _ (zero? (system* "qmake" "qscintilla.pro"))))
>> (add-before 'install 'fix-Makefiles
>> (lambda* (#:key inputs outputs #:allow-other-keys)
>> (let ((out (assoc-ref outputs "out"))
>> (qtbase (assoc-ref inputs "qtbase")))
>> (substitute* (find-files "." "Makefile")
>> (((string-append "INSTALL_ROOT)" qtbase))
>> (string-append "INSTALL_ROOT)" out))))
>> #t)))))
>> (native-inputs
>> `(("python-pyqt" ,python-pyqt)
>> ("qtbase" ,qtbase))) ; for qmake
>> (home-page
>> "https://www.riverbankcomputing.com/software/qscintilla/intro")
>> (synopsis "Qt5 port of the Scintilla editing component")
>> (description
>> "QScintilla is a port to Qt of Neil Hodgson's Scintilla C++ editor
>> control.
>> As well as features found in standard text editing components, QScintilla
>> includes features especially useful when editing and debugging source code.
>> These include support for syntax styling, error indicators, code completion
>> and
>> call tips.")
>> (license (list license:bsd-2 ; Python/configure.py
>> license:expat ; src/ and include/
>> license:gpl3))))
>>
>> (define qtscintilla-for-octave
>
> This should be qtscintilla-qt4, we might have a use for it outside of
> octave
>
In this case, should I leave qtscintilla-qt4 as a public package in qt.scm
instead of maths.scm as Leo suggested?
signature.asc
Description: PGP signature
- [PATCH] gnu: Add qscintilla., Kei Kebreau, 2016/09/12
- Re: [PATCH] gnu: Add qscintilla., Kei Kebreau, 2016/09/13
- Re: [PATCH] gnu: Add qscintilla., Leo Famulari, 2016/09/13
- Re: [PATCH] gnu: Add qscintilla., Kei Kebreau, 2016/09/13
- Re: [PATCH] gnu: Add qscintilla., Leo Famulari, 2016/09/13
- Re: [PATCH] gnu: Add qscintilla., Kei Kebreau, 2016/09/13
- Re: [PATCH] gnu: Add qscintilla., Leo Famulari, 2016/09/13
- Re: [PATCH] gnu: Add qscintilla., Kei Kebreau, 2016/09/13
- Re: [PATCH] gnu: Add qscintilla., Efraim Flashner, 2016/09/13
- Re: [PATCH] gnu: Add qscintilla.,
Kei Kebreau <=
- Re: [PATCH] gnu: Add qscintilla., Efraim Flashner, 2016/09/13
- Re: [PATCH] gnu: Add qscintilla., Leo Famulari, 2016/09/14
- Re: [PATCH] gnu: Add qscintilla., Kei Kebreau, 2016/09/14
- Re: [PATCH] gnu: Add qscintilla., Leo Famulari, 2016/09/14
- Re: [PATCH] gnu: Add qscintilla., Kei Kebreau, 2016/09/14
- Re: [PATCH] gnu: Add qscintilla., Efraim Flashner, 2016/09/14
- Re: [PATCH] gnu: Add qscintilla., Kei Kebreau, 2016/09/15
- Re: [PATCH] gnu: Add qscintilla., Kei Kebreau, 2016/09/15
- Re: [PATCH] gnu: Add qscintilla., Andreas Enge, 2016/09/18
- Re: [PATCH] gnu: Add qscintilla., Kei Kebreau, 2016/09/18