package.yml
16–17	`glibc-devel` is not needed as builddep because it is part of system.devel. I built this localy and I get missing symbols for: libXtst.so.6 libXi.so.6 libasound.so.2 To fix this I suggest you to add `pkgconfig(alsa)` and `pkgconfig(xtst)` to builddeps.

This revision now requires changes to proceed.Nov 2 2022, 8:24 PM

Staudey added a subscriber: Staudey.Nov 2 2022, 8:29 PM

Staudey added inline comments.

package.yml

While we're at it, and since I only noticed this just now, can we change the summary and description to something like:

summary    : High-performance JDK distribution
description: |
    GraalVM is a high-performance JDK distribution designed to accelerate the execution of applications written in Java and other JVM languages along with support for JavaScript, Ruby, Python, and a number of other popular languages

(or at your discretion something else, but something that isn't open-ended like the current description or pure marketing language like the summary ?)

Adjust description and modify build dependencies as per packaging review.

Harbormaster completed remote builds in B4536: Diff 33489.Nov 3 2022, 8:49 AM

Okay made the description changes, I probably got the info from there website as I was not sure what to call it as it works with all sorts of languages.

I have updated the buildeps, I am guessing they are not needed for it to function it seemed to build and work when I tested.