Menu

#26 NTK UI compatibility fixes

2.5.0
closed-accepted
None
5
2014-07-20
2014-07-20
No

Applied against git head c84b301:

  • BankSlot: add necessary redraw command
  • build: Need NTK_IMAGES_LIBRARIES when building NtkGui
2 Attachments

Discussion

  • Mark McCurry

    Mark McCurry - 2014-07-20
    • status: open --> pending
    • assigned_to: Mark McCurry
    • Group: Unstable_(example) --> 2.5.0
     
  • Mark McCurry

    Mark McCurry - 2014-07-20

    I'm unsure that NTK_IMAGES_LIBRARIES is needed.
    I think that with old versions of NTK this was needed, but it shouldn't be needed with the current version of NTK. (I could be wrong here)
    Are you using an up-to-date NTK?

    The bank redraw patch is merged

     
  • Wladimir van der Laan

    I'm using the latest git version of ntk from https://github.com/original-male/ntk . I build both the dynamic and static library and install it to /opt/ntk. I make sure that the appropriate paths are added to PKG_CONFIG_PATH as well as PATH so that cmake can find it.

    The full error is:

    /usr/include/x86_64-linux-gnu/bits/string3.h:81: warning: memset used with constant zero length parameter; this could be due to transposed parameters
    UI/libzynaddsubfx_gui.a(Connection.cpp.o): In function `GUI::createUi(Fl_Osc_Interface*, void*)':
    /home/.../music/zynaddsubfx-code/src/UI/Connection.cpp:48: undefined reference to `fl_register_images()'
    

    After applying the patch it builds and works.

     

    Last edit: Wladimir van der Laan 2014-07-20
  • Mark McCurry

    Mark McCurry - 2014-07-20
    • status: pending --> closed-accepted
     
  • Mark McCurry

    Mark McCurry - 2014-07-20

    Fair enough.
    Both patches are merged and available in the git master branch.

     

Log in to post a comment.