-
Notifications
You must be signed in to change notification settings - Fork 34
Rust helloworld app #45
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Valentin-Razvan Bogdan <[email protected]>
Signed-off-by: Valentin-Razvan Bogdan <[email protected]>
Signed-off-by: Valentin-Razvan Bogdan <[email protected]>
|
Hi @BVRazvan, thanks for this. You will need to follow the examples (e.g. nginx), and have the default |
|
Yes, I ran the app using the steps you have mentioned. Regarding .cfg files for xen/fc, I didn't think that it would be still the case to add them as long as in the catalog |
It's fine, you can leave the fc/xen files out. Can you please share a list of commands that you are using to build the application? I'm running Where does the filename in the |
|
Yeah, I forgot to mention the compilation command; this is |
|
Ah, works fine now. Please mention this in the README, and also add it to the build and run scripts when you create them. Thanks. |
Signed-off-by: Valentin-Razvan Bogdan <[email protected]>
Signed-off-by: Valentin-Razvan Bogdan <[email protected]>
Signed-off-by: Valentin-Razvan Bogdan <[email protected]>
Signed-off-by: Valentin-Razvan Bogdan <[email protected]>
Signed-off-by: Valentin-Razvan Bogdan <[email protected]>
StefanJum
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See the comment and please squash the commits, there should be only one (signed-off) commit. Looks good besides that.
| ./setup.sh | ||
| make distclean | ||
| rustup target add x86_64-unknown-none | ||
| cargo +stable rustc -- --emit=obj |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I get an error here
error: toolchain 'stable-x86_64-unknown-linux-gnu' is not installed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, can you paste here your output of rustup show command in cli?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Default host: x86_64-unknown-linux-gnu
rustup home: /home/stefan/.rustup
installed targets for active toolchain
--------------------------------------
x86_64-unknown-linux-gnu
x86_64-unknown-none
active toolchain
----------------
nightly-x86_64-unknown-linux-gnu (default)
rustc 1.87.0-nightly (aa8f0fd71 2025-03-23)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, so you did a custom installation probably, otherwise stable channel is default (I will explicitly mention it in the readme file) .
As a side note, I matched your toolchain (on local as well as on a vanilla ubuntu image) and it seems that the stable channel gets installed automatically, but I won't take it for granted.
I will squash the commits when everything is fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @StefanJum. Did you manage to run the app after my last commit?
Signed-off-by: Valentin-Razvan Bogdan <[email protected]>
Add
rs-hellodirectory:Makefile: builds the applicationMakefile.uk: includes the app's object fileCargo.lock,Cargo.toml,cargo/config.toml,src/main.rs: rust app configurationsetup: creates workdirgitignore: ignores generated filesLinked to #26