Skip to content

Conversation

@JianyuWang0623
Copy link
Contributor

Summary

Enable PIPES to make nsh pipeline available.
More details, please see apache/nuttx-apps#2737

Impact

CLI of sim:nsh

Testing

  1. Selftest
# Test 1 with `FS_HOSTFS`
nsh> ls / | dd of=/data/test_pipeline
sh [4:100]
nsh> cat /data/test_pipeline
/:
 bin/
 data/
 dev/
 etc/
 proc/
 tmp/
 var/
nsh> ls /
/:
 bin/
 data/
 dev/
 etc/
 proc/
 tmp/
 var/
nsh>


# Test 2 with CMDPARAMS
## Without https://github.com/apache/nuttx-apps/pull/2841 - bug
nsh> echo $var | cat
sh [4:100]

nsh>
## With https://github.com/apache/nuttx-apps/pull/2841
nsh> set var `uname`
nsh> echo $var
NuttX
nsh> echo $var | cat
sh [4:100]
NuttX
nsh>
  1. NuttX CI

More details, please see apache/nuttx-apps#2737

Signed-off-by: wangjianyu3 <wangjianyu3@xiaomi.com>
@github-actions github-actions bot added Board: simulator Size: XS The size of the change in this PR is very small labels Nov 13, 2024
@xiaoxiang781216 xiaoxiang781216 merged commit 7e0ee8c into apache:master Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Board: simulator Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants