[23.0] Introduce --disable-debuginfo-stripping option

[zakkak]

Enables the use of --disable-debuginfo-stripping which was introduced
in oracle/graal#7624

[zakkak]

Follow up PRs will be created for 23.1, 24.0, and master once this gets approved.

[jerboaa]

Enables the use of --disable-debuginfo-stripping which was introduced in oracle/graal#7624

Right, that was introduced to master with that change, but it also got backported to 23.0 here:
oracle/graal@cdb0389

Therefore, adding that option to mandrel-packaging seems fine with that in place.

[jerboaa]

Patch itself is fine, but could you update the workflow file to check out the right mx version, like our mandrel CI does?

PR checker fails due to:

2024-01-11T13:21:02.9038428Z Exception in thread "main" java.lang.RuntimeException: java.nio.file.NoSuchFileException: /home/runner/work/mandrel-packaging/mandrel-packaging/mandrel/substratevm/mxbuild/linux-amd64/com.oracle.svm.native.libchelper/amd64/liblibchelper.a
2024-01-11T13:21:02.9038637Z 	at FileSystem.copy(build.java:1529)
2024-01-11T13:21:02.9038742Z 	at build.main(build.java:182)
2024-01-11T13:21:02.9039805Z Caused by: java.nio.file.NoSuchFileException: /home/runner/work/mandrel-packaging/mandrel-packaging/mandrel/substratevm/mxbuild/linux-amd64/com.oracle.svm.native.libchelper/amd64/liblibchelper.a

Suggesting it's using mx suitable for master, only.

[zakkak]

Thanks for mentioning this, I was struggling to see what's wrong... :)

[jerboaa]

@zakkak Oh, I've just realized this doesn't yet work on Windows:

 VERSION=$(jq -r .mx_version ${MANDREL_REPO}/common.json): D:\a\_temp\9b0ef8ba-4f46-4891-af53-35e0c27ce28b.ps1:2
Line |
   2 |  VERSION=$(jq -r .mx_version ${MANDREL_REPO}/common.json)
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The term 'VERSION=$(jq -r .mx_version ${MANDREL_REPO}/common.json)' is not recognized as a name of a cmdlet,
     | function, script file, or executable program. Check the spelling of the name, or if a path was included, verify
     | that the path is correct and try again.

[jerboaa]

Nice. GHA clean now.