graalvm-reachability-metadata icon indicating copy to clipboard operation
graalvm-reachability-metadata copied to clipboard

Remove 'latest' key from index.json for itext metadata

Open vitali-pr opened this issue 1 year ago • 4 comments

Starting from 8.0.4 itext core includes reachability metadata into library.

What does this PR do?

Looking at https://www.graalvm.org/native-image/libraries-and-frameworks/ it seems there is a redundancy in terms of supported versions of itext core libs. I believe removing 'latest' key from index.json should remove '8.0.3 - latest' range from the page above and leave only 8.0.3. Though I don't see any similar example in the whole repo so not sure.

Checklist before merging

vitali-pr avatar May 28 '24 09:05 vitali-pr

@msupic , it's hanging for quite some time already. Could you, please have a look?

vitali-pr avatar Jun 06 '24 06:06 vitali-pr

@dnestoro - I will not have time to review this so if you or somebody else can review it would be great.

msupic avatar Jun 06 '24 08:06 msupic

I am not sure what to do in this case. @fniephaus since you are more familiar with libraries-and-frameworks list, what do you think?

dnestoro avatar Jun 14 '24 09:06 dnestoro

Thanks for bringing this to our attention, @vitali-pr. I'm not sure we really need to change the metadata but the changes you suggest look reasonable to me. Anyway, it will not automatically solve the double listing on the graalvm.org website. Let me take a look next week and see how we can improve the website. Maybe this change is actually not needed, in which case it may be better to delay the changes until they are really needed. :)

fniephaus avatar Jun 14 '24 16:06 fniephaus

We have improved the website and it now shows, for example, this: image

fniephaus avatar Sep 11 '24 08:09 fniephaus