Run `make dep` strictly before `make $(TARGET)` puzzle about fundamental HOT 14 CLOSED

Indeed, I updated the 2 delay modules to match Fundamental's code and makefile too.
Just tried make -j4, got this error:

src/DelayPlus.cpp:10:10: fatal error: 'samplerate.h' file not found #include "samplerate.h"
makealone works ok as @cschol said
Is just matter of building the dependencies first, right? (but makefile changes are way over my skills) This way is neater than make dep, make though.

from fundamental.

Try 03c14d5bb6b6076a91070b0b90c8366cdd50abbc in Rack and c9142a8 in Fundamental
This makes dep a target of all object files.

from fundamental.

Works now for both Fundamental and @AScustomWorks with make -j 4. Thank you!

from fundamental.

Sorry, it doesn't work. I didn't do a pull on Fundamental before I tested.

from fundamental.

@AndrewBelt Looks like a botched merge? For some reason 343b666 did not make it in the final Makefile version (the shell command that created the dep directory is not in the Makefile).

Result:

cd dep && curl -OL http://www.mega-nerd.com/SRC/libsamplerate-0.1.9.tar.gz
/bin/sh: 1: cd: can't cd to dep
Makefile:18: recipe for target 'dep/lib/libsamplerate.a' failed
make: *** [dep/lib/libsamplerate.a] Error 2

from fundamental.

Another issue: Fundamental (and @AScustomWorks) plugins don't load.

[0.032 warn] Failed to load library ./plugins/Fundamental/plugin.so: ./plugins/Fundamental/plugin.so: undefined symbol: src_delete
[0.045 warn] Failed to load library ./plugins/AS/plugin.so: ./plugins/AS/plugin.so: undefined symbol: src_delete

from fundamental.

Try 1878e96 of Rack

from fundamental.

Builds and loads for both plugins now. Thank you!

from fundamental.

@AndrewBelt The Makefile is still a problem. It does not create the dep directory if it is not already there. Fails on all platforms if dep is not there. Take a look at the Makefile in the latest revision. The merge took out the $(shell mkdir -p dep) line although one of the commits had it in there.

I didn't notice since make clean does not remove dep (not that it should). Only git clean -dfx does.

from fundamental.

Fixed in 0543c35

make clean should not remove dependencies.

from fundamental.

@AndrewBelt Agreed on make clean. I ran the wrong command and it did not remove dep.

Sorry, this does not work. The variable CONFIGURE is initialized when dep does not exist yet, so the --prefix argument has the wrong value, because realpath on a non-existing directory returns an empty string.

You could remove the ":" from CONFIGURE in dep.mk to defer initialization until it is used in Fundamental's Makefile.

from fundamental.

aaf7473

from fundamental.

@AndrewBelt Oh, yes. That's a better fix. Tested on Linux and it works.

Now, on Windows, libsamplerate compilation fails. This seems to be a different issue (possibly related to libsamplerate itself) and I'll open a separate issue to track.

from fundamental.

The Windows compilation issue is same as #38.

from fundamental.