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

Quality of life issues while using Ambient #27

Open
Pombal opened this issue Jun 6, 2023 · 3 comments
Open

Quality of life issues while using Ambient #27

Pombal opened this issue Jun 6, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@Pombal
Copy link

Pombal commented Jun 6, 2023

Hi,

I've been trying to use https://github.com/AmbientRun/Ambient on latest VS22 and I've noticed a few issues:

When I select 'Start Debugging' for ambient.exe (app/Cargo.toml as startup item) then the code is rebuilt from scratch instead of using the existing binary.

When I select 'Start Debugging' for client_fog.exe (guest/rust/examples/basics/fog/Cargo.toml as startup item) then I get an error with the Manifest parsing logic where it's unable to find the .exe. I've deleted the .vs folder as the error message says but it continues.

It's possible these are due to user error but it would be great if you could have a look at making rust-analyzer.vs work well with Ambient.

Thanks!

@parthopdas
Copy link
Member

looking into it.

@parthopdas
Copy link
Member

apologies for the delay.

for issue 1: a build does happen but it is an incremental build. unfortunately without significant architectural changes this is unavoidable (e.g. the extension could bypass the build if the bins were update - which it does for running unit tests but extension api does not support it for debug case)

for issue 2: it appears the example is being built but the binary is not generated. same is the result from command line - see pic below. please can you see what the issue is. i'd wager if the commandline build works, IDE will as well, if not i'll take a look.

image

@Pombal
Copy link
Author

Pombal commented Jun 29, 2023

Thanks for looking into it.

I believe that for issue 2 what is being generated by default is a wasm file instead of a binary.
I will look into seeing if there's something that be changed within Ambient.

@parthopdas parthopdas added the enhancement New feature or request label Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants