[natives] make move debug prints enabled by env variable

.github/workflows/build.yaml

@@ -6,7 +6,7 @@ on:
       - release
       - ci-test
   pull_request:
-    types: [opened, reopened]
+    types: [opened, reopened, synchronize]
     branches:
       - release
 

.github/workflows/lint.yaml

@@ -6,7 +6,7 @@ on:
       - release
       - ci-test
   pull_request:
-    types: [opened, reopened]
+    types: [opened, reopened, synchronize]
     branches:
       - release
 

.github/workflows/publish.yaml

@@ -80,4 +80,4 @@ jobs:
           file: target/cli/diem
           tag: ${{ github.ref }}
           overwrite: true
-          file_glob: true
+          file_glob: true

diem-move/framework/src/natives/debug.rs

@@ -23,7 +23,14 @@ use move_vm_types::{
     values::{Reference, Struct, Value},
 };
 use smallvec::{smallvec, SmallVec};
-use std::{collections::VecDeque, sync::Arc};
+use std::{collections::VecDeque, env, sync::Arc};
+use once_cell::sync::Lazy;
+// instead of compile time feature, we use environment variable to enable debug prints from the Move vm
+// Note: downstream libra does not use compile time features.
+const MOVE_DEBUG_ENV: Lazy<bool> = Lazy::new(|| match env::var("DIEM_MOVE_DEBUG") {
+    Ok(_) => true,
+    Err(_) => false,
+});
 
 /***************************************************************************************************
  * native fun print
@@ -39,7 +46,7 @@ fn native_print(
     debug_assert!(ty_args.is_empty());
     debug_assert!(args.len() == 1);
 
-    if cfg!(feature = "testing") {
+    if *MOVE_DEBUG_ENV {
         let val = safely_pop_arg!(args, Struct);
         let bytes = val.unpack()?.next().unwrap();
 
@@ -69,7 +76,7 @@ fn native_stack_trace(
 
     let mut s = String::new();
 
-    if cfg!(feature = "testing") {
+    if *MOVE_DEBUG_ENV{
         context.print_stack_trace(&mut s)?;
     }
 

diem-move/framework/src/natives/transaction_context.rs

@@ -87,7 +87,7 @@ fn native_generate_unique_address(
 ) -> SafeNativeResult<SmallVec<[Value; 1]>> {
     context.charge(gas_params.base)?;
 
-    let mut transaction_context = context
+    let transaction_context = context
         .extensions_mut()
         .get_mut::<NativeTransactionContext>();
     transaction_context.auid_counter += 1;