Update TDA Plugin to 3.2

[irockel]

Hey VisualVM Team,

I recently reworked my little TDA - Thread Dump Analyzer and did a new release which aside from MCP Support reworks the UI, supports Virtual Threads, newer JDKs and fixes several bugs.

The new plugin version (with updated dependencies) can be found here:

https://github.com/irockel/tda/releases/tag/3.0

The Plugin consists of three modules (same as in the past) and I successfully tested it with VisualVM 2.2.1.

Would be great if you can update the dependency in the plugin center.

Best,

Ingo

[irockel]

Just an update, i just published version 3.1, this fixes a wrong version information inside the VisualVM Plugin and a missing license reference (among some other fixes):
https://github.com/irockel/tda/releases/tag/3.1

[thurka]

I reviewed the latest version of the TDA plugin. It would be great if you could take a look at the following minor issues.

1. VisualVM 2.2 is based on NetBeans 22, while your plugin uses dependency on NetBeans 29. It would be better to use same dependency for your plugin.
2. Can you please fix the following warning:

WARNING [org.netbeans.core.modules]: had to upgrade dependencies for module net.java.dev.tda.visualvm.logfile: added = [module org.openide.util.lookup > 8.0] removed = []; details: [#170056: Separate module for Lookup API]

3. There are few deprecated usages of Utilities.loadImage(). It would be great if you can change it to ImageUtilities.loadImage()
4. The module dependencies seem to be defined in two places, pom.xml and OpenIDE-Module-Module-Dependencies in manifest.mf. Can this be unified?

Thanks.

[irockel]

See the linked PR for the requested changes, except for the manifest, I tried this (again), but auto-creating the manifest always led to a broken one and as I hardly change them I'll stay with this, knowing it's duplicated.

Please let me know, if this works for you and I will do a release.

[thurka]

Thanks for the update. I did look at the PR and it looks fine. You can go ahead and do the release.

[irockel]

Did a new release, you can find it here:
https://github.com/irockel/tda/releases/tag/3.2

[thurka]

The plugin is now available on VisualVM 2.2.1 plugin center.

When publishing I noticed that visualvm-lib-component-3.2.nbm module is not hidden. I fixed that manually in updates.xml, but it would be great if you could fix it for the next version.

diff --git a/visualvm-lib-component/manifest.mf b/visualvm-lib-component/manifest.mf
index ec55653..e35622a 100644
--- a/visualvm-lib-component/manifest.mf
+++ b/visualvm-lib-component/manifest.mf
@@ -3,6 +3,7 @@ OpenIDE-Module-Module-Dependencies: org.graalvm.visualvm.application/2 > 2.0, or
 OpenIDE-Module-Java-Dependencies: Java > 11
 OpenIDE-Module: net.java.dev.tda/2
 OpenIDE-Module-Localizing-Bundle: net/java/dev/tda/Bundle.properties
+AutoUpdate-Show-In-Client: false
 OpenIDE-Module-Specification-Version: 3.2
 OpenIDE-Module-Requires: org.openide.modules.ModuleFormat1
 Class-Path: ext/tda.jar ext/swingx.jar ext/gson.jar ext/filters.jar

[irockel]

Thanks for making the new version available!

I added the fix and will include it in an upcoming release.