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

--enable-mainscroll build fails, "‘height’ was not declared in this scope" #129

Closed mxmilkiib closed 6 years ago

mxmilkiib commented 6 years ago

with tempo_track;

In file included from /usr/include/c++/7.2.1/x86_64-pc-linux-gnu/bits/os_defines.h:39:0,
                 from /usr/include/c++/7.2.1/x86_64-pc-linux-gnu/bits/c++config.h:533,
                 from /usr/include/c++/7.2.1/cctype:41,
                 from mainwnd.cpp:80:
/usr/include/features.h:376:4: warning: #warning _FORTIFY_SOURCE requires compiling with optimization (-O) [-Wcpp]
 #  warning _FORTIFY_SOURCE requires compiling with optimization (-O)
    ^~~~~~~
mainwnd.cpp: In constructor ‘seq64::mainwnd::mainwnd(seq64::perform&, bool, int, int, int, bool)’:
mainwnd.cpp:1208:5: error: ‘height’ was not declared in this scope
     height += hpadding * (m_mainwid_rows - 1);
     ^~~~~~
mainwnd.cpp:1208:5: note: suggested alternative: ‘sig_t’
     height += hpadding * (m_mainwid_rows - 1);
     ^~~~~~
     sig_t
mainwnd.cpp:1209:5: error: ‘width’ was not declared in this scope
     width += wpadding * (m_mainwid_columns - 1);
     ^~~~~
mainwnd.cpp:1209:5: note: suggested alternative: ‘wcwidth’
     width += wpadding * (m_mainwid_columns - 1);
     ^~~~~
     wcwidth
make[2]: *** [Makefile:648: mainwnd.lo] Error 1
make[2]: Leaving directory '/var/tmp/makepkg/sequencer64-git/src/sequencer64/seq_gtkmm2/src'
make[1]: *** [Makefile:456: all-recursive] Error 1
make[1]: Leaving directory '/var/tmp/makepkg/sequencer64-git/src/sequencer64/seq_gtkmm2'
make: *** [Makefile:571: all-recursive] Error 1
==> ERROR: A failure occurred in build().
ahlstromcj commented 6 years ago

I fixed this in the latest tempo_track, however, main-scroll is a feature still in progress and not yet fully fleshed out. But feel free to play with it, although by default the main window is not resizable (by my window manager, Fluxbox). Thanks!

mxmilkiib commented 6 years ago

Building now :)