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

add serial-monitor-rust #4

Merged
merged 62 commits into from
Jan 8, 2025
Merged

add serial-monitor-rust #4

merged 62 commits into from
Jan 8, 2025

Conversation

hacknus
Copy link
Contributor

@hacknus hacknus commented Dec 7, 2024

/claim #2

This should work.

@algora-pbc algora-pbc bot mentioned this pull request Dec 7, 2024
Copy link

algora-pbc bot commented Dec 7, 2024

💵 To receive payouts, sign up on Algora, link your Github account and connect with Stripe.

@hacknus
Copy link
Contributor Author

hacknus commented Dec 7, 2024

@wolfv can you run the workflow again?

@hacknus
Copy link
Contributor Author

hacknus commented Dec 10, 2024

This should work now. Bumped to 0.3.2 of serial-monitor as well.

@wolfv
Copy link
Member

wolfv commented Dec 10, 2024

Hi @hacknus awesome effort! Unfortunately, for a proper conda package we would need to get all dependencies (e.g. everything installed with apt) as conda packages as well. I think most things should be available on conda-forge.

@hacknus
Copy link
Contributor Author

hacknus commented Dec 10, 2024

Hi @hacknus awesome effort! Unfortunately, for a proper conda package we would need to get all dependencies (e.g. everything installed with apt) as conda packages as well. I think most things should be available on conda-forge.

Does this also apply to stuff installed with cargo?

@wolfv
Copy link
Member

wolfv commented Dec 10, 2024

No, this does not apply to crates that are built from source with cargo.

@hacknus
Copy link
Contributor Author

hacknus commented Dec 10, 2024

@wolfv on windows the rust version is too old for my cargo.lock:

 │ │   lock file version `4` was found, but this version of Cargo does not understand this lock file, perhaps Cargo needs to be updated?

and I cannot update it with rustup:

%SRC_DIR%>rustc --version 
 │ │ rustc 1.77.2 (25ef9e3d8 2024-04-09)
 │ │ %SRC_DIR%>cargo --version 
 │ │ cargo 1.77.2 (e52e36006 2024-03-26)
 │ │ %SRC_DIR%>rustup update 
 │ │ info: no updatable toolchains installed
 │ │ error: rustup is not installed at '%PREFIX%\.cargo'
 │ │ %SRC_DIR%>rustup show 

and also, what is happening here on windows:

│ %SRC_DIR%>conda install conda-forge::rust -y 
 │ │ 'conda' is not recognized as an internal or external command,
 │ │ operable program or batch file.

@hacknus
Copy link
Contributor Author

hacknus commented Dec 11, 2024

@wolfv I'm still struggling with the old windows version, why is rust 1.77 on there but on linux and macos rust 1.83 ist there?

The latest version of serial-monitor-rust requires 1.83 for the Cargo.lock file to be read.

@hacknus
Copy link
Contributor Author

hacknus commented Dec 11, 2024

Yeah, I totally do not understand why there is a linker error. It works when I run it on my local windows-machine.

@hacknus
Copy link
Contributor Author

hacknus commented Dec 11, 2024

solved.

@hacknus hacknus requested a review from wolfv December 11, 2024 22:05
@wolfv wolfv merged commit c86d8c9 into prefix-dev:main Jan 8, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants