Build system bugfixes #90

merged 5 commits into from Aug 27, 2012


None yet
2 participants

Winter-Guerra commented Jul 15, 2012

Hey Makerbot folks!

I know this codebase has fallen out of active development, but here's a few small fixes for compilation errors in the scons buildsystem that have proved to be a nuisance to firmware hackers.

Here's a summary of the changes:
-Instead of using preprocessor #ifdef statements to guard blocks of code incompatible with certain boards, everything is now neatly handled directly in the scons build system. This fix in particular squashes compilation errors due to the build system trying to compile interface kit related firmware files when the extruder or motherboard doesn't support it.

-Added a new platform in the SConstruct.extruder file to make compiling and uploading FW for the newer ecv34 extruder board powered by the ATMEL328P a breeze.

-Changed the default upload port that scons will try to use to upload newly compiled firmware. It now uses some shell scripting trickery to autodetect connected arduino-based boards.

Keep it printing!

Winter-Guerra added some commits Jul 15, 2012

Fixed build errors for boards that don't support interface boards usi…
…ng the scons build system.

This is an improvement over the previous method of assigning preprocessor "ifdef" statements.
Now, the files related to the running of the interface boards will not compile for some motherboards, and all extruders.
Added a new SCons platform to take care of compiling the newer ec34 b…
…oards w/ atmel328p microcontrollers

This platform can be compiled by either using the ./ script or by using the scons command
$ scons platform=ecv34_328p
The compiled hex file will be placed in firmware/build/ecv34_328p/


Added automatic upload port selection for firmware builds uploaded di…
…rectly through scons.

Now, as long as tere is only one arduino-powered board plugged into the machine, scons should automatically upload directly to the correct port.
Fixed SCon's upload port autodetect bash script
The previous script was not properly escaped and would evaluate prematurely.

aleonard added a commit that referenced this pull request Aug 27, 2012

@aleonard aleonard merged commit f163398 into makerbot:master Aug 27, 2012

aleonard added a commit that referenced this pull request Nov 7, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment