Skip to content

Installer improvements #88

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

Merged
merged 7 commits into from
Jun 27, 2022
Merged

Installer improvements #88

merged 7 commits into from
Jun 27, 2022

Conversation

SergioGasquez
Copy link
Member

  • Deleted unused functions like install_rust
  • Avoid installing Xtensa GCC
  • Install only nightly toolchain for Xtensa targets, instead of stable and nightly as is sufficient
  • LLVM Clang is now only installed for Xtensa targets
  • Move the logic where we check if we need to install esp-idf
  • Update test-rust-toolchain CI to allow different git repos

Closes #72

@SergioGasquez SergioGasquez requested a review from georgik June 15, 2022 12:45
@georgik georgik added this to the v1.62 milestone Jun 16, 2022
@georgik
Copy link
Collaborator

georgik commented Jun 16, 2022

Thank you @SergioGasquez for the contribution. It's always good to optimize development process.

The change require additional testing. I suggest to include it with 1.62.0.0 build. I've market the PR with the milestone.

@georgik georgik changed the base branch from main to build/1.62.0.0 June 27, 2022 12:01
@georgik
Copy link
Collaborator

georgik commented Jun 27, 2022

Changes looks good. We should be careful about minimal profile, we need to check tests whether examples are building without problem.

@georgik georgik merged commit 61547e9 into build/1.62.0.0 Jun 27, 2022
@SergioGasquez SergioGasquez deleted the feature/toolchains-test branch June 27, 2022 12:50
georgik added a commit that referenced this pull request Jul 13, 2022
* ci: update version to 1.62.0.0
* Installer improvements (#88)
* Use nightly toolchain with profile-minimal
* Install llvm clang only for Xtensa targets
* Add cargo-generate url and bin file for aarch64-unknown-linux-gnu (#89)
* Support `--build-target all` (#92)
* Fix ldproxy links (#94)
* ci: add system package dependency on openssl
* Remove PIP_USER (#96)
* Remove espmonitor (#99)
- replaced by functionality in espflash
* Add `ldproxy` binary for `aarch64-unknown-linux-gnu` (#100)
* Update espflash and cargo-espflash binaries (#103)
* Fix: Use cargo-espflash binary (#105)

Co-authored-by: Sergio Gasquez Arcos <[email protected]>
@kyrias
Copy link
Contributor

kyrias commented Jul 13, 2022

This being merged to master in #93 combined with #85 means that rust-build no longer installs a C compiler at all. Is this intentional?

@kyrias
Copy link
Contributor

kyrias commented Jul 13, 2022

Additionally, the install script is still adding the GCC bin path to $PATH in the export file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Research dependency of toolchains nightly and stable
3 participants