ahlstromcj / sequencer64

A major reboot of Seq24. Current release 0.97.0 (2021-05-13), native JACK MIDI, Song recording, playlists, and a Windows/Qt version. For fresher code, see the Seq66 project. Note that trigger and mute-group-in-MIDI-file formats have evolved! Back up your work!
GNU Affero General Public License v3.0
235 stars 29 forks source link

m4/configure: Add missing QtWidgets check #211

Closed simonvanderveldt closed 3 years ago

simonvanderveldt commented 3 years ago

Small fix to make compilation of the Qt5 GUI work when QtDesigner isn't installed. Somehow the check for designer also includes the check for QtWidgets (and other Qt packages) so you probably never noticed :)

ahlstromcj commented 3 years ago

Done. Thanks! You might check out the highly modified m4 file used in Seq66. I pared down the originals to a shorter file, m4/ax_have_qt_min.m4. If you see anything wrong with that one, please let me know! Thanks again! -- Chris

-------- Simon van der Veldt 07:00 Sun 25 Oct --------

Small fix to make compilation of the Qt5 GUI work when QtDesigner isn't installed. Somehow the check for designer also includes the check for QtWidgets (and other Qt packages) so you probably never noticed :) You can view, comment on, or merge this pull request online at:

https://github.com/ahlstromcj/sequencer64/pull/211

-- Commit Summary --

  • configure: Add missing QtWidgets check

-- File Changes --

M m4/ax_have_qt.m4 (1)

-- Patch Links --

https://github.com/ahlstromcj/sequencer64/pull/211.patch https://github.com/ahlstromcj/sequencer64/pull/211.diff

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/ahlstromcj/sequencer64/pull/211

-- Oh yeah. Forgot about those. Getting senile, I guess... -- Larry Wall in 199710261551.HAA17791@wall.org