This is an automated email from the ASF dual-hosted git repository.

xiaoxiang781216 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git


The following commit(s) were added to refs/heads/master by this push:
     new 5c441693fa8 boards/risc-v/qemu-rv/rv-virt: drop stale venv activate 
from citest64 run.sh
5c441693fa8 is described below

commit 5c441693fa8d1175bd62ea74cf8d800f3e788023
Author: Brijesh Thakkar <[email protected]>
AuthorDate: Sun Oct 25 00:22:31 2026 +0530

    boards/risc-v/qemu-rv/rv-virt: drop stale venv activate from citest64 run.sh
    
    citest64/run.sh still activates a virtual environment around the NTFC
    call:
    
      source ${NTFCDIR}/venv/bin/activate
      ...
      deactivate
    
    4497bacb1c ("boards: use NTFC from pip for citest configs") switched the
    citest scripts to the ntfc installed by pip and removed these lines from
    the sabre-6quad, qemu-armv8a, rv-virt:citest and sim scripts, but missed
    this one. CI does not create ${NTFCDIR}/venv, so the log has
    
      run.sh: line 32: .../nuttx-ntfc/venv/bin/activate: No such file or
      directory
      run.sh: line 44: deactivate: command not found
    
    Both errors are harmless because the script does not use "set -e", and
    the NTFC exit status is saved in $ret before the "deactivate" line. Remove
    the two lines so that citest64/run.sh matches the other citest scripts.
    The exit status and the saved artifacts are unchanged.
    
    Assisted-by: Claude Code:claude-sonnet-5-5
    Signed-off-by: Brijesh Thakkar <[email protected]>
---
 boards/risc-v/qemu-rv/rv-virt/configs/citest64/run.sh | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/boards/risc-v/qemu-rv/rv-virt/configs/citest64/run.sh 
b/boards/risc-v/qemu-rv/rv-virt/configs/citest64/run.sh
index d3f04ee0438..131dd60038d 100755
--- a/boards/risc-v/qemu-rv/rv-virt/configs/citest64/run.sh
+++ b/boards/risc-v/qemu-rv/rv-virt/configs/citest64/run.sh
@@ -28,9 +28,6 @@ olddir=$(pwd)
 nuttdir=${CURRENTCONFDIR}/../../../../../../
 cd ${nuttdir}
 
-# enable venv
-source ${NTFCDIR}/venv/bin/activate
-
 # run NTFC
 confpath=${CURRENTCONFDIR}/config.yaml
 jsonconf=${CURRENTCONFDIR}/session.json
@@ -40,9 +37,6 @@ ntfc test --testpath=${testpath} --confpath=${confpath} 
--jsonconf=${jsonconf}
 ret="$?"
 echo $ret
 
-# disable venv
-deactivate
-
 # export test results
 artifacts=${ARTIFACTCONFDIR}/ntfc
 mkdir -p ${artifacts}

Reply via email to