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

platform-specific VisualGDB test & benchmark projects #6285

Merged
merged 1 commit into from
Apr 18, 2023

Conversation

gojimmypi
Copy link
Contributor

Description

This PR only affects VisualGDB project files for the wolfSSL test and benchmark apps on the ESP32.

Prior generic project files were removed, and new, platform and chipset-specific project and solution files added.

Fixes zd# n/a

Testing

How did you test?

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

Copy link
Contributor

@dgarske dgarske left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you know anyone else who uses the VisualGDB examples?

@gojimmypi
Copy link
Contributor Author

Do you know anyone else who uses the VisualGDB examples?

Yes, several people including @PaulMartinsen. He's been quite helpful in pushing the limits of our ESP32 code with a variety of issues and pull requests.

I also believe that VisualGDB makes it easier for newcomers to get started. It should also be appealing to any Visual Studio users. It's an awesome development environment.

@bandi13 bandi13 merged commit f7d7e4f into wolfSSL:master Apr 18, 2023
@gojimmypi
Copy link
Contributor Author

I've submitted a support request on the VisualGDB forum to see if the ESP-IDF version and Chipset selection edits by the end user might possibly be improved, thus eliminating the need for all the separate project files in this (and future) examples.

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

Successfully merging this pull request may close these issues.

3 participants