bugGNU Octave - Bugs: bug #50572, error during compiling

 
 

bug #50572: error during compiling

Submitter:  None
Submitted:  Fri 17 Mar 2017 11:02:35 PM UTC
   
 
Category:  GUI Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Fixed Assigned to:  siko1056
Originator Name:  Saleh Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * dev
Operating System:  * GNU/Linux Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Tue 28 Mar 2017 08:37:12 AM UTC, comment #17: 
Kai Torben Ohlhus <siko1056>
Group Member
Tue 28 Mar 2017 06:46:37 AM UTC, comment #16: 

@Mike: yes, downloading the latest 2.10 (from February 2017) release tar from https://www.riverbankcomputing.com/software/qscintilla/download and then under Qt4Qt5/qscintilla.pro (some Makefile like file) you find this target of comment #14. My point is, before the qscintilla developers had chosen this target name, they could have made a short survey, as I did in 2 Minutes:

https://software.opensuse.org/package/qscintilla-qt5
https://www.archlinux.org/packages/extra/x86_64/qscintilla-qt5/
https://apps.fedoraproject.org/packages/qscintilla-qt5
https://packages.debian.org/sid/libqt5scintilla2-dev

I push the cset soon.

Kai Torben Ohlhus <siko1056>
Group Member
Tue 28 Mar 2017 01:30:54 AM UTC, comment #15: 

Kai - do you mean that change was made in upstream Qscintilla? If that's the way upstream is going to start naming the library, then we should probably support it, and maybe distros that made their own naming convention will adapt.

Mike Miller <mtmiller>
Group Member
Mon 27 Mar 2017 10:47:16 PM UTC, comment #14: 

Regarding comment #11 a change of the target in qscintilla 2.10 is to blame:


TARGET = qscintilla2_qt$${QT_MAJOR_VERSION}


Most distributions prefer the "qscintilla2-qtX" separation and this will cause lots of patches for stable distros switching to 2.10. I like jwe's closing words on bug #48773, if any other maintainer agrees, I'll apply the patch as it does no harm. But I'm afraid this list is to grow longer if qscintilla does not adopt library naming conventions.

Kai Torben Ohlhus <siko1056>
Group Member
Thu 23 Mar 2017 01:18:09 AM UTC, comment #13: 

Attached a patch to make octave detect qscintilla by it self (at least on Manjaro).

The patch can be applied to octave/m4/acinclude.m4

Dear developers: can you apply the patch and close the bug

regards
Saleh


(file #40088)

Anonymous
Sun 19 Mar 2017 07:54:08 AM UTC, comment #12: 

@Stefan (in reply to comment #8): This is really strange. In the case where qscintilla is not found on your system, the code where the compilation error on file_editor_tab_widget occurred should never be compiled since it is located within


#if defined (HAVE_QSCINTILLA)
...
#endif


after the changeset 5bddd4089386.

Torsten Lilge <ttl>
Group Member
Sun 19 Mar 2017 01:27:21 AM UTC, comment #11: 

The problem is form my system.

I updated qscintilla. In the new version the name of the library
changed from qscintilla-q5.so to  qscintilla2_qt5.
I did not notice it at first. I am not sure if the name changing is caused by the person who buid qscintilla in Manjaro or by qscintilla itself.

I added the name(qscintilla2_qt5) to the name list in line 1331 of "acinclude.m4" to solve the problem

By doing that, octave is able to detect qscintilla by itself.

@Stefan can you confirm if this is the case for arch


Anonymous
Sat 18 Mar 2017 03:58:19 PM UTC, comment #10: 

It is weird. I can confirm it builds successfully  on manjaro. I also have both qscintilla-qt4 and qscintilla-qt5 installed.
However, octave cannot detect neither.
Note: the path for qscintillas are
/usr/lib/libqscintilla2_qt4.so
/usr/lib/libqscintilla2_qt5.so

Anonymous
Sat 18 Mar 2017 03:08:57 PM UTC, comment #9: 

Hello, obviously the issue can be worked around by deinstalling qscintilla-qt4.

Stefan Husmann <haawda>
Sat 18 Mar 2017 02:14:49 PM UTC, comment #8: 

Both questions have to be answered with yes. Maybe it is relevant that I have both qscintilla-qt4 and qscintilla-qt4 installed.

The error message is exactly the same as the ticket opener had.

Stefan Husmann <haawda>
Sat 18 Mar 2017 12:51:16 PM UTC, comment #7: 

@Stefan: You are taling about the error described in the original submission of this report? What is the exact error message and do you have QScintilla installed?

Torsten Lilge <ttl>
Group Member
Sat 18 Mar 2017 12:45:25 PM UTC, comment #6: 

I cannot confirm that 5bddd4089386 fixes the build error under Arch Linux.

Stefan Husmann <haawda>
Sat 18 Mar 2017 10:17:39 AM UTC, comment #5: 

@Torsten
with 5bddd4089386 change set, it is possible to build octave
even if scintilla not detected,
 
@Dan
I built octave from scratch in a new folder, but still cannot detect scintilla.
I believe there is something happened since my last successful build (March 12). At that time, octave was able to detect scintilla.

I was thinking maybe my system is broken, however there are other people having the same problem
https://aur.archlinux.org/packages/octave-hg/

Anonymous
Sat 18 Mar 2017 08:57:33 AM UTC, comment #4: 

Sounds familiar.  See

https://savannah.gnu.org/bugs/?func=detailitem&item_id=48773

That should have been fixed for your setup, though.  Have you built Octave from scratch?  Sometimes that is needed to get rid of some settings hanging around from previous builds.


Dan Sebald <sebald>
Sat 18 Mar 2017 08:50:25 AM UTC, comment #3: 

Thanks for the report. The bug leading to a compilation error when no qscintilla is available was introduced with changeset changeset 23276:ea143f4f76a5 and is fixed with
http://hg.savannah.gnu.org/hgweb/octave/rev/5bddd4089386
Could you please check this with your system?

The changeset does not fix the issue, that configure does not find qscintilla on your system anymore.

Torsten Lilge <ttl>
Group Member
Sat 18 Mar 2017 08:49:32 AM UTC, comment #2: 

As a work around, I created a link to scintilla wiht a name that
octave familiar with.

sudo ln -s /usr/lib/libqscintilla2_qt5.so /usr/lib/libqt5scintilla2.so

By doing that the problem is solved.

is it possible to make octave figure the correct name by itself?

Anonymous
Sat 18 Mar 2017 07:27:26 AM UTC, comment #1: 

I think I found a lead

checking for the QScintilla library for Qt 5... no
.
.
.
.
configure: WARNING: QScintilla library not found; disabling built-in Qt GUI editor


octave cannot find libqscintilla2

The last week is the last time I build octave successfully

the directory of the library on my manjaro linux is

/usr/lib/libqscintilla2_qt5.so


Anonymous
Fri 17 Mar 2017 11:02:35 PM UTC, original submission:  

I am using MAnjaro linux with gcc version 6.3

This bug probably related to #50567


CXX      libgui/src/m-editor/libgui_src_libgui_src_la-find-dialog.lo
../libgui/src/m-editor/file-editor.cc:2480:1: error: ‘file_editor_tab_widget’ has not been declared
 file_editor_tab_widget::mousePressEvent(QMouseEvent *me)
 ^~~~~~~~~~~~~~~~~~~~~~
../libgui/src/m-editor/file-editor.cc:2480:41: error: variable or field ‘mousePressEvent’ declared void
 file_editor_tab_widget::mousePressEvent(QMouseEvent *me)
                                         ^~~~~~~~~~~
../libgui/src/m-editor/file-editor.cc:2480:41: error: ‘QMouseEvent’ was not declared in this scope
../libgui/src/m-editor/file-editor.cc:2480:54: error: ‘me’ was not declared in this scope
 file_editor_tab_widget::mousePressEvent(QMouseEvent *me)



configured as follow

Qt CPPFLAGS:                   -I/usr/include/qt/QtNetwork -I/usr/include/qt -I/usr/include/qt/QtOpenGL -I/usr/include/qt -I/usr/include/qt/QtPrintSupport -I/usr/include/qt -I/usr/include/qt/QtWidgets -I/usr/include/qt -I/usr/include/qt/QtGui -I/usr/include/qt -I/usr/include/qt/QtCore -I/usr/include/qt
  Qt LDFLAGS:
  Qt libraries:                  -lQt5Network -lQt5OpenGL -lQt5PrintSupport -lQt5Widgets -lQt5Gui -lQt5Core
  Qt moc:                        moc-qt5
  Qt uic:                        uic-qt5
  Qt rcc:                        rcc-qt5
  Qt lrelease:                   lrelease-qt5
  READLINE libraries:            -lreadline
  Sndfile CPPFLAGS:
  Sndfile LDFLAGS:
  Sndfile libraries:             -lsndfile
  SuiteSparse config libraries:  -lsuitesparseconfig
  SUNDIALS_IDA CPPFLAGS:
  SUNDIALS_IDA LDFLAGS:
  SUNDIALS_IDA libraries:        -lsundials_ida
  SUNDIALS_NVECSERIAL CPPFLAGS:
  SUNDIALS_NVECSERIAL LDFLAGS:
  SUNDIALS_NVECSERIAL libraries: -lsundials_nvecserial
  TERM libraries:                -lncurses
  UMFPACK CPPFLAGS:
  UMFPACK LDFLAGS:
  UMFPACK libraries:             -lumfpack
  X11 include flags:
  X11 libraries:                 -lX11
  Z CPPFLAGS:
  Z LDFLAGS:
  Z libraries:                   -lz

  Default pager:                 less
  gnuplot:                       gnuplot

  Build Octave Qt GUI:                  yes



Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #40088:  patch.txt added by None (439B - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mtmiller (Posted a comment)
  • -email is unavailable- added by siko1056 (Posted a comment)
  • -email is unavailable- added by haawda (Posted a comment)
  • -email is unavailable- added by sebald (Posted a comment)
  • -email is unavailable- added by ttl (Posted a comment)
  • -email is unavailable- added by None (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-03-28 siko1056 StatusIn Progress Fixed
        Open/ClosedOpen Closed
    2017-03-28 siko1056 StatusPatch Reviewed In Progress
        Assigned toNone siko1056
    2017-03-27 siko1056 StatusNone Patch Reviewed
    2017-03-23 None Attached File- Added patch.txt, #40088

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code