Skip to content

Conversation

gojimmypi
Copy link
Contributor

Description

This is a no-code-change update that only adds VisualGDB project and solution files to the test and benchmark Espressif examples.

Fixes zd# n/a

Testing

How did you test? n/a

Checklist

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

@gojimmypi
Copy link
Contributor Author

The failing PRB is not likely related to this PR.

@gojimmypi gojimmypi requested a review from dgarske July 13, 2023 16:59
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.

"+1,492 −28" -> My preference is for these to be in wolfssl-examples. The VisualGDB creates a lot of cruft for something very few people use. You are welcome to dispute my remark.

@gojimmypi
Copy link
Contributor Author

"+1,492 −28" -> My preference is for these to be in wolfssl-examples. The VisualGDB creates a lot of cruft for something very few people use. You are welcome to dispute my remark.

Well, since you left the door open to discussion... :)

I certainly agree that there are more project files than desired. I reached out to the Sysprogs folks regarding the difficulties changing targets. As of now it is a known problem with no easy resolution.

My objective here is to make things as easy for the new user as possible. Having the project files for the different chipssets that are just ready to go is extremely convenient. Very little instruction is needed. This is much more likely to give a positive first impression.

I fully understand that development in Visual Studio with VisualGDB for embedded targets does not have as big of an audience as it could have. I'm trying to grow that audience. I want to show all those Windows developers that they can easily get into the embedded world using familiar tools. I happen to be working on Espressif chipsets, but VisualGDB supports over 1000 devices in a single IDE.

I plan to feature VisualGDB in my upcoming webinar. It is an incredibly productive tool.

To put the wolfssl Espressif example project and solution files in the separate wolfssl-examples would be awkward and cumbersome. I suppose we could duplicate the TLS client/server, test, and benchmark apps there... but having duplicate code and keeping them in sync is not very desirable.

I suppose we could cut the file count in half by omitting the solution .sln files, and perhaps even only include the most recent ESP-IDF versions. However it is nice to have the pre-defined Solution Items file shortcuts available for quick reference:

image

I could update the respective README.md files to inform the end users of the key files that may be of interest to add to the solution, such as the wolfssl user_settings.h if you'd be willing to have all the project files, but no solution files.

It's certainly possible to have just one project file per example (total)... but the instructions would need to be considerably longer.

Keep in mind I plan to also [propose to] add project files to the TLS client & server Espressif examples as well.

Given this information, please let me know what you prefer.

@gojimmypi gojimmypi requested a review from dgarske July 14, 2023 00:22
@gojimmypi gojimmypi requested review from JacobBarthelmeh and wolfSSL-Bot and removed request for dgarske August 15, 2023 19:14
@gojimmypi gojimmypi marked this pull request as draft November 16, 2023 22:36
@dgarske dgarske removed the request for review from wolfSSL-Bot May 16, 2024 16:53
@gojimmypi
Copy link
Contributor Author

Closing this lingering PR as the project files are typically updated with each example.

@gojimmypi gojimmypi closed this Jun 18, 2024
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.

2 participants