• Tags
  • Documentation
  • Order
  • Register
  • Login
Duet3D Logo Duet3D
  • Tags
  • Documentation
  • Order
  • Register
  • Login

New build system introduced, testing and feedback needed

Scheduled Pinned Locked Moved
PanelDue
7
57
2.8k
Loading More Posts
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • undefined
    mfs12
    last edited by 13 Aug 2021, 09:50

    @oliof i think it drops somehow the file extension "exe" and therefore doesn't find it. Probably something is going wrong in the toolchain detection.

    Visit me on github at https://github.com/mfs12/

    undefined 1 Reply Last reply 13 Aug 2021, 09:57 Reply Quote 0
    • undefined
      oliof @mfs12
      last edited by oliof 13 Aug 2021, 09:57

      @mfs12 I fixed it by installing the gnu-arm-eabi-gcc packages in Ubuntu again (this wasn't needed before, but I'll live).

      More notes:

      • CMake needs to be at least version 3.19 to support REAL_PATH subcommand. This should be pointed out in the dependency documentation. Ubuntu 20.04 (LTS release) is on version 3.16 (but kitware offers an upstream repo to get latest cmake, so this is barely an inconvenience)
      • with compiler and cmake sorted out, I run into an error in the end:
      [ 94%] Linking CXX executable paneldue.elf
      arm-none-eabi-g++: error: Wagener/PanelDueFirmware/src/ASF/sam/utils/linker_scripts/sam4s/sam4s4/gcc/flash.ld: No such file or directory
      make[2]: *** [CMakeFiles/paneldue.elf.dir/build.make:866: paneldue.elf] Error 1
      make[2]: Leaving directory '/mnt/c/Users/Harald Wagener/PanelDueFirmware/build'
      make[1]: *** [CMakeFiles/Makefile2:156: CMakeFiles/paneldue.elf.dir/all] Error 2
      make[1]: Leaving directory '/mnt/c/Users/Harald Wagener/PanelDueFirmware/build'
      make: *** [Makefile:91: all] Error 2
      make: Leaving directory '/mnt/c/Users/Harald Wagener/PanelDueFirmware/

      I assume this is due to spaces in the directory path -- generated build files should handle this gracefully.

      EDIT: indeed, fixing build/CMakeFiles/paneldue.elf.dir/link.txt by escaping the space in the home directory name allows the build to complete.

      <>RatRig V-Minion Fly Super5Pro RRF<> V-Core 3.1 IDEX k*****r <> RatRig V-Minion SKR 2 Marlin<>

      1 Reply Last reply Reply Quote 2
      • undefined
        oliof
        last edited by 13 Aug 2021, 10:15

        And here is the tiniest pull request to eliminate a compiler warning: https://github.com/Duet3D/PanelDueFirmware/pull/177

        oliof opened this pull request 13 Aug 2021, 10:14 in Duet3D/PanelDueFirmware

        closed Eliminate compiler warning in FlashStorage.cpp #177

        <>RatRig V-Minion Fly Super5Pro RRF<> V-Core 3.1 IDEX k*****r <> RatRig V-Minion SKR 2 Marlin<>

        1 Reply Last reply Reply Quote 2
        • undefined
          mfs12
          last edited by 14 Aug 2021, 10:25

          Hey @oliof could you test the updated version on you windows machine?

          https://github.com/Duet3D/PanelDueFirmware/pull/179

          mfs12 opened this pull request 14 Aug 2021, 10:22 in Duet3D/PanelDueFirmware

          closed Cmake improvements #179

          Visit me on github at https://github.com/mfs12/

          1 Reply Last reply Reply Quote 0
          • undefined
            resam
            last edited by 14 Aug 2021, 11:11

            Seems to compile fine on latest macOS:

            brew install gcc-arm-embedded
            mkdir build
            cd build
            cmake -DDEVICE=v2-5.0 ..
            make -j 4

            (compiles everything, but plenty of warnings)

            It produces a paneldue.bin with 144kB - which looks reasonable compared to the most recent official release binaries.

            1 Reply Last reply Reply Quote 1
            • undefined
              oozeBot
              last edited by 16 Aug 2021, 20:28

              We'd love to see a quick tutorial on this from someone who has been successful building it on Windows. This was something we tried to tackle today with no success. Thanks

              undefined 1 Reply Last reply 17 Aug 2021, 02:57 Reply Quote 0
              • undefined
                oliof @oozeBot
                last edited by oliof 17 Aug 2021, 02:57

                @oozebot I successfully did that, see the discussion at https://github.com/Duet3D/PanelDueFirmware/pull/179. You may need to pull that PR into your local repo to successfully build as it's not committed yet (or check out https://github.com/mfs12/PanelDueFirmware/tree/cmake-improvements).

                1. If you don't have the GNU ARM Compiler and Build Tools installed yet, follow steps 1 and 3 from the RRF Build Instructions. I did re-use the 2018-q4 release but
                2. Install CMake from the CMake Download Page. This replaces step 2 in the build instructions mentioned above.
                3. Make sure to add the directories the binaries end up in to your Path.
                4. Using Powershell, check you can find make, rm using (Get-Command make).Path and (Get-Command rm).Path.
                5. Generate the build files running make -G "Unix Makefiles" -B build -DDEVICE="5.0i" -DCROSS_COMPILE="C:/Program\ Files\ (x86)/GNU\ Tools\ ARM\ Embedded/8\ 2018-q4-major/bin/arm-none-eabi-"(adjusting the value for-DCROSS_COMPILE parameter to your version and install path and the value for -DDEVICE for the PanelDue you want to build for).
                6. Build the binary running make -C build all (you can run this with -j4 -O for maximum parallelism)

                And that should be it.

                @mfs12 I think it might be a good idea to follow the RRF change of moving the build instructions to the GitHub wiki and replacing both BuildInstructions.md (outdated now) and the Development section in Readme.md (more current).

                mfs12 opened this pull request 14 Aug 2021, 10:22 in Duet3D/PanelDueFirmware

                closed Cmake improvements #179

                <>RatRig V-Minion Fly Super5Pro RRF<> V-Core 3.1 IDEX k*****r <> RatRig V-Minion SKR 2 Marlin<>

                1 Reply Last reply Reply Quote 2
                • undefined
                  mfs12
                  last edited by mfs12 17 Aug 2021, 06:46

                  Hey @oliof and @oozeBot,

                  I updated the readme file accordingly.

                  Updated the master branch https://github.com/duet3d/PanelDueFirmware/tree/master with fixes from @oliof and better documentation.

                  Feedback on the documentation is very welcome so people will be able to setup things as quick as possible.

                  Visit me on github at https://github.com/mfs12/

                  1 Reply Last reply Reply Quote 1
                  • undefined
                    mfs12
                    last edited by 17 Aug 2021, 06:47

                    And what's really a bummer is the path handling for the CROSS_COMPILE variable on windows. If you have any good solutions for that I want to learn about them.

                    Visit me on github at https://github.com/mfs12/

                    1 Reply Last reply Reply Quote 0
                    • undefined
                      oliof
                      last edited by 17 Aug 2021, 09:28

                      I can confirm that a clean checkout of PanelDueFirmware master branch builds cleanly on Windows now.

                      Will this build environment also come to RepRapFirmware?

                      <>RatRig V-Minion Fly Super5Pro RRF<> V-Core 3.1 IDEX k*****r <> RatRig V-Minion SKR 2 Marlin<>

                      undefined 1 Reply Last reply 17 Aug 2021, 09:31 Reply Quote 1
                      • undefined
                        mfs12 @oliof
                        last edited by mfs12 17 Aug 2021, 09:31

                        Will this build environment also come to RepRapFirmware?

                        This is still unclear. It depends on the experiences we collect with paneldue.

                        Visit me on github at https://github.com/mfs12/

                        1 Reply Last reply Reply Quote 1
                        • undefined
                          oliof
                          last edited by 17 Aug 2021, 09:35

                          You have my vote of confidence (-:

                          And also let me point out that when I implemented the Colinear Tripteron kinematics, I resorted to building new versions of RepRapFirmware with the Makefiles Eclipse generated to relieve my anemic machine from running Eclipse all the time. So it's possible and desirable (-:

                          <>RatRig V-Minion Fly Super5Pro RRF<> V-Core 3.1 IDEX k*****r <> RatRig V-Minion SKR 2 Marlin<>

                          1 Reply Last reply Reply Quote 1
                          • undefined
                            gloomyandy
                            last edited by 17 Aug 2021, 09:54

                            This is probably sacrilege but I find using cmake overly complex and much prefer good old make. cmake is great when it works, but when there is a problem it can be very hard to identify what is the cause. The LPC and STM32 builds of RRF both use make, for the recent ESP32 version of the Duet WiFi software I used cmake (to fit in with the espressif build system), it is way more complex and hard to debug than the corresponding make system I use to build the ESP8266 version. Maybe I'm just too old school, but having watched many software projects disappear down a rat hole of a "new improved build system", sometimes simple is best.

                            undefined 1 Reply Last reply 17 Aug 2021, 12:54 Reply Quote 2
                            • undefined
                              mfs12
                              last edited by 17 Aug 2021, 10:12

                              @gloomyandy, settting cmake up i stumbled also in quite some pitfalls. although i think the found solution is quite handy but still not perfect. but neither is make, it also has a lot of pitfalls from my experience.

                              But once either of them is working things should be ok.

                              So far we are living with eclipse build file generation, which is even harder to debug compared to make and cmake.

                              What convinced me to use cmake is the support for different IDEs, I hope this will lower the bar to build and develop the project.

                              Let's give it a shot. 🙂

                              Visit me on github at https://github.com/mfs12/

                              1 Reply Last reply Reply Quote 1
                              • undefined
                                mfs12
                                last edited by 17 Aug 2021, 10:16

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

                                The LPC and STM32 builds of RRF both use make

                                Can you point me to the repository? Eventually it would be an option to port your solution to RRF.

                                Visit me on github at https://github.com/mfs12/

                                undefined undefined 2 Replies Last reply 17 Aug 2021, 10:26 Reply Quote 2
                                • undefined
                                  jay_s_uk @mfs12
                                  last edited by 17 Aug 2021, 10:26

                                  @mfs12
                                  Here's the main build repo https://github.com/gloomyandy/RRFBuild/tree/v3.4-dev

                                  Owns various duet boards and is the main wiki maintainer for the Teamgloomy LPC/STM32 port of RRF. Assume I'm running whatever the latest beta/stable build is

                                  1 Reply Last reply Reply Quote 1
                                  • undefined
                                    oliof @gloomyandy
                                    last edited by 17 Aug 2021, 12:54

                                    @gloomyandy I kind of agree. Considering I need to manually pass the really important info (where are the build binaries, what do I want to build) manually into CMake to make it work on Windows, a well-constructed set of Makefiles might be cleaner and easier to follow along in the end. But I won't say no to CMake, and it's @mfs12´s weapon of choice (-:

                                    <>RatRig V-Minion Fly Super5Pro RRF<> V-Core 3.1 IDEX k*****r <> RatRig V-Minion SKR 2 Marlin<>

                                    1 Reply Last reply Reply Quote 0
                                    • undefined
                                      gloomyandy @mfs12
                                      last edited by gloomyandy 17 Aug 2021, 13:09

                                      @mfs12 The repo that jay pointed you to is the overall build repo. The actual make files we currently keep in the CoreN2G repo (which may or may not be a good idea):
                                      https://github.com/gloomyandy/CoreN2G/tree/v3.4-dev/src/STM32F4/makefiles
                                      https://github.com/gloomyandy/CoreN2G/tree/v3.4-dev/src/LPC/makefiles

                                      I doubt if these qualify as a "well constructed set of makefiles", but they do the job. For historical reasons we currently keep two separate sets of build files for the two different mcus, it would probably make sense to combine them and have a different make build target.

                                      cmake is certainly a lot easier to handle than the eclipse build file system!

                                      1 Reply Last reply Reply Quote 1
                                      • undefined
                                        oozeBot
                                        last edited by 17 Aug 2021, 14:46

                                        Thanks all. I'm getting further and can create the build files, but am now stuck at the following:

                                        PS C:\Eclipse\PanelDueFirmware> make -C build all -j12
                                        make: Entering directory 'C:/Eclipse/PanelDueFirmware/build'
                                        Makefile:32: *** missing separator. Stop.

                                        I've tried other variations with no luck. FYI - (Get-Command make).Path returns:

                                        C:\Program Files\GNU ARM Eclipse\Build Tools\2.6-201507152002\bin\make.exe

                                        Thanks in advance.

                                        1 Reply Last reply Reply Quote 0
                                        • undefined
                                          mfs12
                                          last edited by 17 Aug 2021, 15:02

                                          @oozeBot
                                          can you send the file 'C:/Eclipse/PanelDueFirmware/build/Makefile'

                                          There's something wrong with the makefile. any path you pass as commandline parameter needs to be in quotes, backslash replaced with slash and whitespaces escaped with backslash.

                                          Can you share your cmake call?

                                          Visit me on github at https://github.com/mfs12/

                                          undefined 1 Reply Last reply 17 Aug 2021, 15:11 Reply Quote 0
                                          14 out of 57
                                          • First post
                                            14/57
                                            Last post
                                          Unless otherwise noted, all forum content is licensed under CC-BY-SA