Menu

#629 ./configure now detects an incomplete wxWidgets install

None
closed-accepted
nobody
None
5
2013-07-05
2013-07-03
dima
No

There was a bug reported on the mailing list where a user was experiencing build errors due to an incomplete install of wxWidgets:

https://sourceforge.net/mailarchive/forum.php?thread_name=87ehbfblk6.fsf%40secretsauce.net&forum_name=gnuplot-info

The user had enough of the library installed to pass the checks in ./configure. This patch adds an extra check to ./configure to flag this case and refuse to build the wxt terminal.

On a Debian-based system this case can be replicated by running

$ sudo update-alternatives --config wx-config

and selecting the 'base' release.

This patch was tested working on Debian only. I have no windows or osx boxes, but this should be tested there too before merging.

Thanks

1 Attachments

Discussion

  • Ethan Merritt

    Ethan Merritt - 2013-07-04

    The thing I would worry about is the use of "grep -q". I'm not sure that idiom is universal.

    Is there a specific header file we could check for using AC_CHECK_HEADERS instead?

     
  • dima

    dima - 2013-07-04

    Hmmm. I'd rather use the wx-config tool itself to look for the headers, since the headers are more likely to change than the tool. How about the attached patch? It performs a similar check as before, but instead of using 'grep', it uses 'expr'. The test immediately above this on in the configure.in file uses expr as well, so it should be more acceptable, hopefully. I successfully tested this expr syntax on my Debian box as well as on a FreeBSD machine and an OSX box.

     
  • Ethan Merritt

    Ethan Merritt - 2013-07-05
    • status: open --> closed-accepted
    • Group: -->
     
  • Ethan Merritt

    Ethan Merritt - 2013-07-05

    OK. Now in cvs

     

Log in to post a comment.