Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

[Documentation Bug] Broken link svm options link in README and graal 'option' needs clarification #388

Open
harikrishnan83 opened this issue Sep 19, 2020 · 0 comments

Comments

@harikrishnan83
Copy link

harikrishnan83 commented Sep 19, 2020

Thank you for building this tool. It is immensely helpful.

What happened?

  1. The link to svm options in README is broken.
  2. Specifying multiple graal 'option's in not immediately clear. I specified all options space separated under single 'option' as shown below. And it failed with the error saying invalid configuration.
option "--no-fallback -H:ReflectionConfigurationFiles=$reflectConfig --initialize-at-run-time=io.netty.channel.kqueue.KQueueEventLoop"

After reading through GraalExtension I realised each of these needed to be specified in separate 'option'. Maybe it was just me. However it does not hurt to add some clarification here.

What did you want to happen?

  1. Update broken link
  2. Add documentation on how to pass multiple Graal options

Nice to have: Error message that guides users not to have more than two tokens separated by a space in one option instead of saying invalid configuration or file not found etc.

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

Successfully merging a pull request may close this issue.

1 participant