Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Rakudo Star binaries cataloger #3224

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

gdams
Copy link

@gdams gdams commented Sep 12, 2024

I seem to be missing something here as I can't get it to detect the binary

Signed-off-by: George Adams <georgeadams1995@gmail.com>
@wagoodman
Copy link
Contributor

wagoodman commented Sep 12, 2024

Taking a look at the binary you selected and the regex you have I also don't see the version embedded in the binary:

$strings classifiers/bin/rakudo-star/2024.08/linux-amd64/rakudo | grep -i rakudo     
ERROR: RAKUDO_HOME is invalid: %s
--rakudo-home=
RAKUDO_HOME

$ strings classifiers/bin/rakudo-star/2024.08/linux-amd64/rakudo | grep 2024
<no results>

Maybe this version is embedded not as a string? If that's the case then detecting the version might not be possible with this approach.

@LaurentGoderre
Copy link
Contributor

I have been looking at rakudo and it's a lot more complex than most of the binaries I have done. The binaries do not seem to include any version info and instead it is embed in the moarvm. I was able to detect moarvm though

@wagoodman
Copy link
Contributor

@gdams @LaurentGoderre It looks like the path forward is cataloging the moarvm binary as mentioned above -- is any help wanted with this? Or does that not quite fit what you're trying to accomplish @gdams ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants