davidgiven / fluxengine

PSOC5 floppy disk imaging interface
MIT License
356 stars 68 forks source link

Regression: commandline-only build no longer works #729

Closed mistydemeo closed 9 months ago

mistydemeo commented 9 months ago

The commandline build (added after #523) doesn't seem to be working anymore. In a current build, running make fluxengine still ends up trying to invoke wx-config. Build logs:

loading build.py
loading build/c.py
loading build/protobuf.py
loading build/pkg.py
loading build/utils.py
loading config.py
loading src/build.py
loading dep/adflib/build.py
loading dep/agg/build.py
loading dep/fatfs/build.py
loading dep/hfsutils/build.py
loading dep/libusbp/build.py
loading dep/stb/build.py
loading lib/build.py
loading arch/build.py
loading src/formats/build.py
loading scripts/build.py
loading tests/build.py
loading src/gui/build.py
loading extras/build.py
loading src/gui/drivetypes/build.py
loading tools/build.py
loading dep/snowhouse/build.py
fluxengine-testdata not found; skipping corpus tests
.obj/build.mk:3125: *** Required binary 'wx-config' not found..  Stop.

I haven't had a chance to try bisecting yet, but the most recent successful build I did was on August 8th.

mistydemeo commented 9 months ago

Looks like it's when the buildsystem changed in f5f223f62210bcf30d9148b87f41a1c0c56ea72c. I'm sorry if I've missed something, but I've been looking at the build documentation and I wasn't able to see if there's still a way to build for CLI only.

davidgiven commented 9 months ago

Yup, it was a build system limitation. It should be fixed now --- a simple make fluxengine will build just the command line tool.

mistydemeo commented 9 months ago

Confirmed, thanks!