Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CLI 2.0.41 fails with startIndex cannot be larger than length of string. (Parameter 'startIndex') #639

Closed
CartBlanche opened this issue May 16, 2024 · 6 comments

Comments

@CartBlanche
Copy link

Describe the bug
Attempted meadow app deploy on Mac M2 (though customer Gabriel Mongeon is hitting the same error on Windows 11)
after it deletes various files it stops with
startIndex cannot be larger than length of string. (Parameter 'startIndex')

To Reproduce
As per above attempt a meadow app deploy

Expected behavior
the app deploy is successful

Developer tools (please complete the following information as best as you can):

  • OS and version: Apple M2 - macOS: 14.4.1 (23E224
  • IDE and version: N/A
  • Meadow extension for IDE version:N/A

Meadow (please complete the following information as best as you can):
Meadow by Wilderness Labs
Board Information
Model: F7Micro
Hardware version: F7FeatherV2
Device name: MeadowF7

Hardware Information
Processor type: STM32F777IIK6
ID: 26-00-25-00-0A-50-30-52-54-31-35-20
Serial number: 205A31845230
Coprocessor type: ESP32
MAC Address -
WiFi: 4C:75:25:D6:C7:A8

Firmware Versions
OS: 1.11.0.0
Runtime: 1.11.0.0
Coprocessor: 1.11.0.0
Protocol: 8

@adrianstevens
Copy link

The variable startIndex only appears in the macOS code path .... I'll take a closer look but this error doesn't make any sense when running on Windows.

@adrianstevens
Copy link

@CartBlanche can you confirm the CLI version?

@CartBlanche
Copy link
Author

CartBlanche commented May 16, 2024

@adrianstevens I'm just going off what the customer told me and what was on the public support post. Maybe ping the customer to confirm?
I only reproed the startIndex issue on my mac and it crashed in GetRelativePath path each time, when I stepped through the code.

I'm running latest CLI 2.0.41 update via usual means .

@adrianstevens
Copy link

I'm running
meadow app build
meadow app trim
meadow app deploy

Using CLI v2.0.41 and it's working correctly.

@adrianstevens
Copy link

There was a bug in the RelativePath method - here's a fix along with recursive folder support and some other cleanup
WildernessLabs/Meadow.CLI#565

People can try CLI v2.0.42.0-beta if they want to test. Otherwise expect a v2.0.43 release soon.

@adrianstevens
Copy link

Fixed in CLI v2.0.43

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

No branches or pull requests

2 participants