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

Ardunio Fixes relating to internal Intel Galileo Tests #7007

Merged
merged 1 commit into from Dec 7, 2023

Conversation

night1rider
Copy link
Contributor

@night1rider night1rider commented Nov 28, 2023

Changes library setup process to match the recursive style laid out for Arduino IDE 1.5 and newer as described here: https://arduino.github.io/arduino-cli/0.35/library-specification/#libraryproperties-file-format, and Fixed issues regarding internal testing.

@night1rider night1rider self-assigned this Nov 28, 2023
@night1rider night1rider marked this pull request as ready for review November 30, 2023 16:10
IDE/ARDUINO/sketches/wolfssl_client/wolfssl_client.ino Outdated Show resolved Hide resolved
IDE/ARDUINO/sketches/wolfssl_client/wolfssl_client.ino Outdated Show resolved Hide resolved
IDE/ARDUINO/sketches/wolfssl_client/wolfssl_client.ino Outdated Show resolved Hide resolved
IDE/ARDUINO/wolfssl-arduino.sh Outdated Show resolved Hide resolved
@night1rider
Copy link
Contributor Author

night1rider commented Dec 6, 2023

I decided to go ahead and update the setup script because to make the old "flat" style library to work we had to put both the src files in the top directory of the created wolfSSL along with a new src folder that contained the same files.
See issue: #6360
I move to the "recursive" style library described by Ardunio "https://arduino.github.io/arduino-cli/0.35/library-specification/#libraryproperties-file-format" for versions 1.5 and newer. Internally we use the 1.8.10 of Ardunio. Changing this makes the file structure similar to how the repo is setup. Only downside is the needing of a src/ folder in the src/ folder.... I also added the generation of a library.properties file as also described in the link above.

Internally Jenkins passes with this new file structure please see the test: nightly-Ardunio-Build-WolfSSL-v2 #76

along with updating the file structure construction for the library.
@night1rider
Copy link
Contributor Author

Jenkins retest this please

@JacobBarthelmeh JacobBarthelmeh merged commit c4b77ad into wolfSSL:master Dec 7, 2023
108 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants