search

eclipse-embed-cdt / eclipse-plugins

The Eclipse Embedded CDT plug-ins for Arm & RISC-V C/C++ developers (formerly known as the GNU MCU Eclipse plug-ins). Includes the archive of previous plug-ins versions, as Releases.
http://eclipse-embed-cdt.github.io/
Eclipse Public License 2.0
558 stars 130 forks source link

Issue:512 - Refresh the UI, since while initialising the controls it would have missed some debug events #513

Closed vishnusarathashling closed 2 years ago

vishnusarathashling commented 2 years ago

Refresh the UI, since while initialising the controls it would have missed some debug events. https://github.com/eclipse-embed-cdt/eclipse-plugins/issues/512.

ilg-ul commented 2 years ago

@vishnusarathashling, the patch should be submitted agains the develop branch; it'll be merged into master a bit later, when the release will be prepared.

vishnusarathashling commented 2 years ago

@vishnusarathashling, the patch should be submitted agains the develop branch; it'll be merged into master a bit later, when the release will be prepared.

Done.

jonahgraham commented 2 years ago

Thank you @vishnusarathashling - well done on your first PR merged into the repo! I look forward to many more 😊

vishnusarathashling commented 2 years ago

Thank you @vishnusarathashling - well done on your first PR merged into the repo! I look forward to many more 😊

Thanks Jonah!

ilg-ul commented 2 years ago

@vishnusarathashling, your patch should be available in v6.2.1, just released.

Please confirm that upgrading Eclipse fixes your problem.

vishnusarathashling commented 2 years ago

Sure @ilg-ul I will test and update.

vishnusarathashling commented 2 years ago

@ilg-ul Since I am little busy for 2 weeks, not sure I will be able to test this up quickly. We need to have some setups to be done to test this scenario. I will do it and update once I have it ready.

ilg-ul commented 2 years ago

Normally it takes a few minutes to install a fresh Eclipse and update it, using the default procedure; you don't have to do anything special, the new plug-ins release should be available for all existing Eclipses.

vishnusarathashling commented 2 years ago

Yeah got that, but to test the original issue we mentioned, it takes some setup to be done. Only in a particular board + configuration we were able to reproduce that. I will try to update this ASAP.

vishnusarathashling commented 2 years ago

I verified this and it's working with new embed-CDT.

jonahgraham commented 2 years ago

I verified this and it's working with new embed-CDT.

Excellent - thanks @vishnusarathashling for confirming the change integrated properly and resolved the issue!

vishnusarathashling commented 2 years ago

Thanks @jonahgraham @ilg-ul .

ilg-ul commented 2 years ago

You're welcome! Thank you for your contribution.