search

Meziu / rust-horizon

Empowering everyone to build reliable and efficient software.
https://www.rust-lang.org
Other
3 stars 1 forks source link

Remove support for env and process functionality #17

Closed Meziu closed 2 years ago

Meziu commented 2 years ago

Related to #6

Removed process support and changed env functions to avoid unsupported functionality. Tell me if anything else is left to remove, we can do one big cleanup here.

@AzureMarker @ian-h-chamberlain

AzureMarker commented 2 years ago

Interestingly, libctru implements exit, but when I tested it crashed the process to the home screen (with an error dialog). https://github.com/devkitPro/libctru/blob/c83c12357e52c30526c7aefaa263a6c86baeed27/libctru/source/system/syscalls.c#L98

Meziu commented 2 years ago

Oh wow, env is way more supported than I thought. Even EnvVars are present. We could do some cool stuff with this, like setting HOME to /3ds and such, though that would be something for ctru::init.

Edit: I'm not saying we should do the HOME thing in particular, but it can be useful for some things one would only expect present in env vars.

Meziu commented 2 years ago

Yes, env fully works and process has been removed. I can tick one off the #6 problems. Again, we should check for other functions and things to remove, but this is good for now.