search

terjeio / grblHAL

This repo has moved to a new home https://github.com/grblHAL
231 stars 90 forks source link

Unable to compile using PIO MassStorageHost.cpp #329

Closed eringuet closed 2 years ago

eringuet commented 2 years ago

Hi, trying to build grblHAL for the first time to configure auto squaring. I pulled the repository with --recurse-submodules.

I'm definitely not a developer so there's a good chance I'm doing something wrong but it appears something might be missing.

Any help would be appreciated, compiler output below.

Processing teensy41 (board: teensy41; platform: teensy; framework: arduino)

Verbose mode can be enabled via -v, --verbose option CONFIGURATION: https://docs.platformio.org/page/boards/teensy/teensy41.html PLATFORM: Teensy (4.13.1) > Teensy 4.1 HARDWARE: IMXRT1062 600MHz, 512KB RAM, 7.75MB Flash DEBUG: Current (jlink) External (jlink) PACKAGES:

Environment Status Duration

teensy41 FAILED 00:00:04.473 ============================================================================ 1 failed, 0 succeeded in 00:00:04.473 ============================================================================ The terminal process "C:\Users\Etienne.platformio\penv\Scripts\platformio.exe 'run'" terminated with exit code: 1.

terjeio commented 2 years ago

This is likely due to PIO using an old version of the uSDFS library. Check/change this.

eringuet commented 2 years ago

Alright, I don't know much about PIO but I tried a few things to try to get this to work. In platformio.ini, I've changed: lib_deps =

https://github.com/wwatson4506/uSDFS#uSDFS-non-blocking

https://github.com/WMXZ-EU/uSDFS

To download the version in the link you suggested. 1.1.3 was installed. sd_config.h has #define USE_MSC 0.

That didn't work so I tried to change my_machine.h to #define SDCARD_ENABLE 0 since I don't believe I need sdcard support but it's still attempting and failing to build.

Thanks

terjeio commented 2 years ago

failing with the same errors?

eringuet commented 2 years ago

Compiling .pio\build\teensy41\libb57\MSC-non-blocking\attachInterruptEx.cpp.o In file included from .pio\libdeps\teensy41\MSC-non-blocking\src\msc.h:31:0, from .pio\libdeps\teensy41\MSC-non-blocking\src\MassStorageDriver.cpp:29: C:\Users\Etienne.platformio\packages\framework-arduinoteensy\libraries\USBHost_t36/USBHost_t36.h:2045:2: error: 'msSCSICapacity_t' does not name a type msSCSICapacity_t msCapacity; ^ C:\Users\Etienne.platformio\packages\framework-arduinoteensy\libraries\USBHost_t36/USBHost_t36.h:2046:2: error: 'msInquiryResponse_t' does not name a type msInquiryResponse_t msInquiry; ^ C:\Users\Etienne.platformio\packages\framework-arduinoteensy\libraries\USBHost_t36/USBHost_t36.h:2047:2: error: 'msRequestSenseResponse_t' does not name a type msRequestSenseResponse_t msSense; ^ C:\Users\Etienne.platformio\packages\framework-arduinoteensy\libraries\USBHost_t36/USBHost_t36.h:2048:2: error: 'msDriveInfo_t' does not name a type msDriveInfo_t msDriveInfo; ^ C:\Users\Etienne.platformio\packages\framework-arduinoteensy\libraries\USBHost_t36/USBHost_t36.h:2067:31: error: 'msSCSICapacity_t' has not been declared uint8_t msReadDeviceCapacity(msSCSICapacity_t const Capacity); ^ compilation terminated due to -fmax-errors=5. In file included from .pio\libdeps\teensy41\MSC-non-blocking\src\msc.h:31:0, from .pio\libdeps\teensy41\MSC-non-blocking\src\MassStorageHost.cpp:31: C:\Users\Etienne.platformio\packages\framework-arduinoteensy\libraries\USBHost_t36/USBHost_t36.h:2045:2: error: 'msSCSICapacity_t' does not name a type msSCSICapacity_t msCapacity; ^ C:\Users\Etienne.platformio\packages\framework-arduinoteensy\libraries\USBHost_t36/USBHost_t36.h:2046:2: error: 'msInquiryResponse_t' does not name a type msInquiryResponse_t msInquiry; ^ C:\Users\Etienne.platformio\packages\framework-arduinoteensy\libraries\USBHost_t36/USBHost_t36.h:2047:2: error: 'msRequestSenseResponse_t' does not name a type msRequestSenseResponse_t msSense; ^ C:\Users\Etienne.platformio\packages\framework-arduinoteensy\libraries\USBHost_t36/USBHost_t36.h:2048:2: error: 'msDriveInfo_t' does not name a type msDriveInfo_t msDriveInfo; ^ C:\Users\Etienne.platformio\packages\framework-arduinoteensy\libraries\USBHost_t36/USBHost_t36.h:2067:31: error: 'msSCSICapacity_t' has not been declared uint8_t msReadDeviceCapacity(msSCSICapacity_t const Capacity); ^ compilation terminated due to -fmax-errors=5. [.pio\build\teensy41\libb57\MSC-non-blocking\MassStorageDriver.cpp.o] Error 1 [.pio\build\teensy41\libb57\MSC-non-blocking\MassStorageHost.cpp.o] Error 1

eringuet commented 2 years ago

I started from scratch, fresh git pull, only changed the uSDFS version. No change to my_machine.h or anything else from the source. Thanks.

terjeio commented 2 years ago

See this issue for a fix.

eringuet commented 2 years ago

That worked. Thanks.