koute / polkavm

A fast and secure RISC-V based virtual machine
Apache License 2.0
234 stars 47 forks source link

Update README.md to fix incorrect description #55

Closed starit closed 10 months ago

starit commented 10 months ago

"Other than" means not included. So it means it does not support amd64 and arm64(also does not support Windows, Linux and macOS)

I guess the initial purpose is to express it supports those architectures and OSs. So it should use "in addition to" or "no other than".

koute commented 10 months ago

I'm confused; to me it seems that the current usage is correct? "Full support for architectures other than amd64 and aarch64 [is a non-goal]" is exactly what I meant there.

starit commented 10 months ago

I'm confused; to me it seems that the current usage is correct? "Full support for architectures other than amd64 and aarch64 [is a non-goal]" is exactly what I meant there.

Do you mean that arm64 and amd64 are not supported by PolkaVM? Just to confirm...

koute commented 10 months ago

I'm confused; to me it seems that the current usage is correct? "Full support for architectures other than amd64 and aarch64 [is a non-goal]" is exactly what I meant there.

Do you mean that arm64 and amd64 are not supported by PolkaVM? Just to confirm...

No; those are the only architectures which will be supported. The section is called "non-goals", which means that every bullet point is supposed to be taken as a negative, as in "these things are not the goals". In this case, "no full support for architectures other than amd64 and aarch64".

starit commented 10 months ago

I'm confused; to me it seems that the current usage is correct? "Full support for architectures other than amd64 and aarch64 [is a non-goal]" is exactly what I meant there.

Do you mean that arm64 and amd64 are not supported by PolkaVM? Just to confirm...

No; those are the only architectures which will be supported. The section is called "non-goals", which means that every bullet point is supposed to be taken as a negative, as in "these things are not the goals". In this case, "no full support for architectures other than amd64 and aarch64".

Understood. I thought we usually put something nice to have on the non-goals. Misunderstood. Thanks for the explanation.