Ablu commented on code in PR #114:
URL: 
https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/114#discussion_r1435265884


##########
rust-toolchain.toml:
##########
@@ -18,6 +18,5 @@
 # Toolchain override for rustup
 
 [toolchain]
-channel = "nightly-2021-09-20"
-components = ["rust-src"]
+channel = "nightly"

Review Comment:
   I would recommend to keep this pinned to "some" nightly to prevent things 
from breaking in a non-bisectable manner.



##########
examples/authentication-rs/ta/build.rs:
##########
@@ -58,5 +82,12 @@ fn main() -> std::io::Result<()> {
 
     println!("cargo:rustc-link-search={}", search_path.display());
     println!("cargo:rustc-link-lib=static=utee");
+    println!("cargo:rustc-link-lib=static=utils");
+    println!("cargo:rustc-link-arg=-Tta.lds");
+    println!("cargo:rustc-link-arg=-e__ta_entry");
+    println!("cargo:rustc-link-arg=-pie");
+    println!("cargo:rustc-link-arg=-Os");
+    println!("cargo:rustc-link-arg=--sort-section=alignment");
+    println!("cargo:rustc-link-arg=--dynamic-list=dyn_list");

Review Comment:
   One could probably move the boilerplate code into a crate that is only used 
as dev-dependencies to avoid the duplication.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@teaclave.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@teaclave.apache.org
For additional commands, e-mail: dev-h...@teaclave.apache.org

Reply via email to