New build system introduced, testing and feedback needed

[oozeBot]

@oliof yes, I did and the root lib/librrf contains many files, so I think I did it right!

[oozeBot]

@mfs12 I just responded that I thought I had, but it must be that didn't work correctly.. I'll rerun those steps again here soon. Thanks again

[oliof]

After you rerun those steps, clean out the build/ directory and rerun cmake.

[oozeBot]

This is the console output. I'll clean up the build folder and try again here soon. Thanks

git submodule update --recursive
Cloning into '/home/pi/PanelDueFirmware/lib/librrf'...
Submodule path 'lib/librrf': checked out 'f3e75ecfe8802ca2cdac07cf20dee3ac05479ea7'

[oozeBot]

I'm back at it.. I cleaned up the build folder, validated I had the latest copy of the windows build tools, verified I had the submodules, and then re-ran.

Here is snippet of the files "cmake -B build" creates (using env.cmake). Note there are now no makefiles! Grr- any ideas?

[mfs12]

@oozeBot looks like cmake decided to create project files for visual code.

Check for more details about different IDE support.

https://cmake.org/cmake/help/latest/manual/cmake-generators.7.html

In such an early stage i would recommend to delete the build folder when things behave weird.

[oozeBot]

@mfs12 Finally! By using the following, it built without any modifications! There were some warnings, but it created the bin files. This is what I was missing..!

I think this was problem all along..

cmake -B build -G "Eclipse CDT4 - Unix Makefiles"

[likhalabs]

This might be unrelated, but is there an ongoing official CMake effort for RepRapFirmware as well? I'm actually the person who posted: https://reprap.org/forum/read.php?147,874397. I'm currently using it to build firmware for Duet WiFi, Duet 3 Mini 5+, and the board I'm developing. However, it would also be nice if there is an official CMake support upstream.

[mfs12]

@oozebot are you using this with eclipse? if so could you check if code completion works nicely?

[mfs12]

@resam said in New build system introduced, testing and feedback needed:

(compiles everything, but plenty of warnings)

there's already task to current compiler warnings.

Progress will be reflected in the issue.

https://github.com/Duet3D/PanelDueFirmware/issues/183

mfs12 created this issue in Duet3D/PanelDueFirmware

closed Fix compiler warnings #183

[mfs12]

@likhalabs said in New build system introduced, testing and feedback needed:

but is there an ongoing official CMake effort for RepRapFirmware as well?

Hey, let's see how it goes with paneldue. If this goes well it can have potential. Do you have a repository with your build system?

[oozeBot]

@mfs12 said in New build system introduced, testing and feedback needed:

@oozebot are you using this with eclipse? if so could you check if code completion works nicely?

Umm - I just followed the directions! I'm not directly using Eclipse, but it is installed.

I just installed the firmware - it's working just fine! The PanelDue now reads version 3.4.0-dev.

[oozeBot]

@oozebot said in New build system introduced, testing and feedback needed:

@mfs12 Finally! By using the following, it built without any modifications! There were some warnings, but it created the bin files. This is what I was missing..!

I think this was problem all along..

cmake -B build -G "Eclipse CDT4 - Unix Makefiles"

I just tried with the following and it worked as well. I know I tried this before, but think I capitalized UNIX which is what caused the failure..

cmake -B build -G "Unix Makefiles"

[mfs12]

@oozeBot good to hear things are working for you now.

[oozeBot]

@mfs12 Real quick update that while I don't have any previous experience with building the firmware, I am digging the new build process! I can compile my changes within seconds and then immediately test. Plus the compiler does a pretty good job of explaining any errors..

[mfs12]

@oozeBot good to hear also a beginner is able to use this easily after setting it up.

[likhalabs]

@mfs12 I have https://github.com/likha3d, but the repos from this account are outdated. I'm currently building for 3.3, while these were for 3.1. I'll try to update the repos in this account.

[mfs12]

@likhalabs thanks. this is probably good enough to have a look at the build system.

[mfs12]

About a month ago cmake based build system was introduced to paneldue.

Is there any further feedback? Is there anybody using it? How is your experience? Any feedback is welcome. Do you miss a feature?

[oozeBot]

@mfs12 We are using it regularly, and it's working very well. But we can't compare against the previous methods as we only recently decided to release our own version of the firmware (we'll keep it in parity with the official releases) that contains several tweaks unique to our printers. Our only issues were with the setup of the environment which were documented earlier in this thread.