Re: [PATCH 03/31] u_boot_pylib: Make pty optional

[Tom Rini]

On Mon, Apr 24, 2023 at 05:08:08PM -0600, Simon Glass wrote:
> This library is not available on Windows. Detect this and work around it
> by using a normal pipe.
> 
> Signed-off-by: Simon Glass <s...@chromium.org>
> ---
> 
>  scripts/make_pip.sh                   |  9 +++++++--
>  tools/u_boot_pylib/cros_subprocess.py | 28 ++++++++++++++++++++++-----
>  2 files changed, 30 insertions(+), 7 deletions(-)
> 
> diff --git a/scripts/make_pip.sh b/scripts/make_pip.sh
> index 4602dcf61c88..bcff65240ba1 100755
> --- a/scripts/make_pip.sh
> +++ b/scripts/make_pip.sh
> @@ -91,7 +91,12 @@ find ${dest} -name __pycache__ -type f -exec rm {} \;
>  find ${dest} -depth -name __pycache__ -exec rmdir 112 \;
>  
>  # Remove test files
> -rm -rf ${dest}/*test*
> +for path in ${dest}/*test*; do
> +     echo ${path}
> +     if ! [[ "${path}" =~ .*test_util.* ]]; then
> +             rm -rf ${path}
> +     fi
> +done
>  
>  mkdir ${dir}/tests
>  cd ${dir}
> @@ -108,7 +113,7 @@ echo "Completed build of ${tool}"
>  # Use --skip-existing to work even if the version is already present
>  if [ -n "${upload}" ]; then
>       echo "Uploading from ${dir}"
> -     python3 -m twine upload ${repo} -u __token__ dist/*
> +     python3 -m twine upload ${repo} --verbose -u __token__ dist/*
>       echo "Completed upload of ${tool}"
>  fi

The verbose here is I assume a debugging left-over.

-- 
Tom

signature.asc
Description: PGP signature