[GR-48354] Consolidate application module name detection.

[graalvmbot]

This PR re-implements the module name detection needed in the driver to not require running a jvm command anymore.

In addition to that there are some smaller cleanups.

[jerboaa]

@olpaw Thanks for the heads-up. This still fails the build with --no-jlinking option. Do you want to fix this as part of this PR or shall I with a rebased version of #7205?

[olpaw]

This still fails the build with --no-jlinking option.

The failing that is related to the splitting up of graal-sdk/truffle jars can also be fixed in a follow PR provided by you.

I still have issues with this PR as it adds a dependency to libstdc++. This is not a problem for the driver but for native-image agent (which then also depends on libstdc++) that gets loaded into a jvm and then there can be libstdc++ conflicts ... so what I currently have is still WIP.

[olpaw]

@jerboaa this is now in good shape and will be merged soon. Once on master please create a PR with any remaining changes needed for mandrel.

[jerboaa]

@jerboaa this is now in good shape and will be merged soon. Once on master please create a PR with any remaining changes needed for mandrel.

@olpaw Excellent! Sure I have a PR handy locally. I'll be rebasing #7205 as far as the patch is concerned. Hope that's OK too?

[olpaw]

@olpaw Excellent! Sure I have a PR handy locally. I'll be rebasing #7205 as far as the patch is concerned. Hope that's OK too?

Yes, except for the changes in buildImage(). These you do not need anymore.

[jerboaa]

@olpaw Excellent! Sure I have a PR handy locally. I'll be rebasing #7205 as far as the patch is concerned. Hope that's OK too?

Yes, except for the changes in buildImage(). These you do not need anymore.

Sure, understood. Thanks!

[fniephaus]

@jerboaa do you approve this PR?

[jerboaa]

@jerboaa do you approve this PR?

Sure. Sorry if this was waiting on myself (if so, I wasn't aware).

[jerboaa]

@fniephaus @olpaw Any ETA when this will merge?

[fniephaus]

It's on the top of the merge queue, should be merged in the next couple of minutes.

[jerboaa]

Great, thanks!

[fniephaus]

should be merged in the next couple of minutes.

Famous last words. Turns out the PR hit an infrastructure issue and needs to go through the gate again.

[jerboaa]

OK. Thanks for the update.

[olpaw]

@jerboaa this is now on master. Feel free to create your PR now that fixes the remaining issues that are needed for mandrel.

[jerboaa]

@jerboaa this is now on master. Feel free to create your PR now that fixes the remaining issues that are needed for mandrel.

@olpaw Yes, thanks! Already done in #7205. Please review once you get a chance.