Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unable to compile missing cpp file #431

Closed
prototypen9e opened this issue Nov 18, 2020 · 4 comments
Closed

unable to compile missing cpp file #431

prototypen9e opened this issue Nov 18, 2020 · 4 comments

Comments

@prototypen9e
Copy link

im trying to compile this on slackware64 its unable to compile due to a missing baseButton,cpp file
pes.cpp' || echo './'`src/gui/elems/basics/boxtypes.cpp
mv -f src/gui/elems/basics/.deps/giada-boxtypes.Tpo src/gui/elems/basics/.deps/giada-boxtypes.Po
make: *** No rule to make target 'src/gui/elems/basics/baseButton.cpp', needed by 'src/gui/elems/basics/giada-baseButton.o'. Stop.

@dvzrv
Copy link
Contributor

dvzrv commented Nov 20, 2020

@monocasual This concerns version 0.17.0 btw. While it's nice to now have a basic cmake setup included it is not complete (no tests, no install target) and can not be used instead of the autotools based setup.

@prototypen9e It seems the files are not required anymore. You can remove the entries with sed '/baseButton/d' -i Makefile.am.

@monocasual
Copy link
Collaborator

@prototypen9e are you building with cmake or autotools? We are gradually moving all the autotools functionalities to cmake, something must have slipped through the cracks.

@dvzrv you said it's unusable because of the missing install target, the tests or both?

@dvzrv
Copy link
Contributor

dvzrv commented Nov 20, 2020

@monocasual this is an issue with the autotools setup.

The cmake setup is not really usable if one wants to run tests (I think that still requires autotools to build the tests anyways?), as cmake does not create a test target (maybe I missed something?).
Not having an install target is also not so great, as it means more manual intervention, which is why I stuck with the autotools setup for now.

@monocasual
Copy link
Collaborator

monocasual commented Nov 20, 2020

@dvzrv I'm planning to include tests & install target in the cmake setup (i.e. issue #422) for the next release 0.17.1. Thanks for the explanation and your patience!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants