Skip to content

Update Rust nightly version#1546

Merged
bors[bot] merged 3 commits into
masterfrom
update-rust-2020-01
Jan 19, 2020
Merged

Update Rust nightly version#1546
bors[bot] merged 3 commits into
masterfrom
update-rust-2020-01

Conversation

@bradjc

@bradjc bradjc commented Jan 16, 2020

Copy link
Copy Markdown
Contributor

Pull Request Overview

It has been about 3 months, so a good time to stay current. Also motivated by rust-lang/rust#67887 which should make debugging easier.

Changes

  • New warning about () not needed.
  • Update to update script.

Testing Strategy

travis

TODO or Help Wanted

n/a

Documentation Updated

  • Updated the relevant files in /docs, or no updates are required.

Formatting

  • Ran make formatall.

rustup update nightly just doesn't print the needed version number for
the new rust compiler. I'm not sure how to get it, so we require the
user to type it in for us.
No longer need to manually include it in the makefile.

@gendx gendx left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarding rust-lang/rust#67887 (which sounds quite useful for debugging by the way), could you document what is the increase in code size for the various boards?

Comment thread boards/Makefile.common
@gendx

gendx commented Jan 17, 2020

Copy link
Copy Markdown
Contributor

Regarding rust-lang/rust#67887 (which sounds quite useful for debugging by the way), could you document what is the increase in code size for the various boards?

I had a quick look, and it seems that overall code and/or data size decreased on all boards with this change :)

@hudson-ayers

Copy link
Copy Markdown
Contributor

Regarding rust-lang/rust#67887 (which sounds quite useful for debugging by the way), could you document what is the increase in code size for the various boards?

I had a quick look, and it seems that overall code and/or data size decreased on all boards with this change :)

I think that the primary reason that we currently use .expect() rather than .unwrap() was to make it easy to trace the source of panics, so with this change I would be all for changing most .expect() calls back to .unwrap() which should reduce data size by removing storage for the strings passed to .except()


echo "Please enter the version of Rust to use."
echo "It is probably just one day later than whatever was printed out above."
read BEST_DATE

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is unfortunate, but really probably only run by Tock core folks, so not really a problem

@ppannuto

Copy link
Copy Markdown
Member

bors r+

bors Bot added a commit that referenced this pull request Jan 19, 2020
1546: Update Rust nightly version r=ppannuto a=bradjc

### Pull Request Overview

It has been about 3 months, so a good time to stay current. Also motivated by rust-lang/rust#67887 which should make debugging easier.

Changes
- New warning about `()` not needed.
- Update to update script.


### Testing Strategy

travis


### TODO or Help Wanted

n/a


### Documentation Updated

- [x] Updated the relevant files in `/docs`, or no updates are required.

### Formatting

- [x] Ran `make formatall`.


Co-authored-by: Brad Campbell <[email protected]>
@bors

bors Bot commented Jan 19, 2020

Copy link
Copy Markdown
Contributor

Build succeeded

@bors bors Bot merged commit e2fa42d into master Jan 19, 2020
@bors bors Bot deleted the update-rust-2020-01 branch January 19, 2020 22:31
ppannuto added a commit that referenced this pull request Jan 20, 2020
As discussed in rust-lang/rust#67875, there is a
difference between 'private' and 'hidden' items. Documenting private is
now the default for **bin** crates, but we also want documentation for
all of private library crates to be generated as well. Compare:

 - Before #1546: https://5e23ab06b4899600087c8c17--docs-tockosorg.netlify.com/nrf52/ficr/
 -  After #1546: https://deploy-preview-1546--docs-tockosorg.netlify.com/nrf52/ficr/

This restores our previous documentation
sirchnik pushed a commit to sirchnik/tock that referenced this pull request May 12, 2026
1546: Update Rust nightly version r=ppannuto a=bradjc

### Pull Request Overview

It has been about 3 months, so a good time to stay current. Also motivated by rust-lang/rust#67887 which should make debugging easier.

Changes
- New warning about `()` not needed.
- Update to update script.


### Testing Strategy

travis


### TODO or Help Wanted

n/a


### Documentation Updated

- [x] Updated the relevant files in `/docs`, or no updates are required.

### Formatting

- [x] Ran `make formatall`.


Co-authored-by: Brad Campbell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants