Merge #205

205: Stop using randomized symbol names r=therealprof a=jonas-schievink

It isn't possible to do this by incrementing a global counter, since the expansion order of macros isn't guaranteed and might change between compiler invocations.

Fixes #212
Closes #196
Closes #195

Co-authored-by: Jonas Schievink <[email protected]>

Author: bors[bot]

CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased]
 
+### Changed
+
+- Macros now generate a second trampoline function instead of randomizing the
+  function's symbol name. This makes the build deterministic.
+- [breaking-change] `static mut` resources no longer have `'static` lifetime
+  except in the `#[entry]` function (this is a soundness fix; see [#212]).
+
+[#212]: https://github.com/rust-embedded/cortex-m-rt/issues/212
+
 ## [v0.6.10] - 2019-07-25
 
 ### Fixed

Cargo.toml

@@ -24,18 +24,17 @@ cortex-m = "0.6"
 panic-halt = "0.2.0"
 cortex-m-semihosting = "0.3"
 
-[dev-dependencies.rand]
-default-features = false
-features = ["small_rng"]
-version = "0.7"
-
 [target.'cfg(not(target_os = "none"))'.dev-dependencies]
 compiletest_rs = "0.4.0"
 
 [[example]]
 name = "device"
 required-features = ["device"]
 
+[[test]]
+name = "compiletest"
+required-features = ["device"]
+
 [features]
 device = []
 

ci/script.sh

@@ -23,7 +23,6 @@ main() {
         override-exception
         pre_init
         qemu
-        rand
         state
         unsafe-default-handler
         unsafe-entry

examples/rand.rs

@@ -21,10 +21,5 @@ proc-macro2 = "1.0"
 features = ["extra-traits", "full"]
 version = "1.0"
 
-[dependencies.rand]
-default-features = false
-features = ["small_rng"]
-version = "0.7"
-
 [dev-dependencies]
 cortex-m-rt = { path = "..", version = "0.6" }