Virtualenv: set -eu로 실행할 때 바인딩되지 않은 변수로 인해 activate가 실패할 수 있음

나는 또한 virtualenv 15.1.0을 실행하여 이것을 찾고 있습니다. 저는 Nextflow 파이프라인 내에서 환경을 사용하고 있으며 Nextflow는 기본적으로 엄격 모드(https://github.com/nextflow-io/nextflow/issues/302)로 실행됩니다. Nextflow는 엄격 모드 없이 실행되도록 재구성할 수 있지만 가능하면 바인딩되지 않은 변수를 사용하지 않는 것이 바람직하다는 Nextflow 개발자의 의견에 동의합니다.

activate 스크립트가 어떻게 생성되는지 정확히 모르지만 activate.sh 에서 가져온 경우 수정 사항은 57, 59 및 61 행에서 $PS1 를 다음으로 변경하는 것만큼 간단할 수 있습니다. ${PS1-} . (이 구문은 사용 가능한 경우 PS1 값을 사용하고 그렇지 않은 경우 빈 문자열을 사용합니다. PS1 의 값을 변경하지 않습니다. Documentation ). 적어도 내 환경에서 이렇게 생성된 activate 스크립트를 수정하면 오류 메시지가 사라집니다.

https://github.com/pypa/virtualenv/issues/991

bash 코드를 작성하는 방법을 배우는 데 몇 년이 걸릴지 궁금합니다. virtualenv의 바인딩되지 않은 변수 버그는 아주 오래되었고 수정하기 쉽고 나중에 간단한 한 줄만 추가하면 피할 수 있습니다. 가상 테스트: set -euox pipefail .

일반적으로 debian, ubuntu, centos, rhel, fedora 등에 패키지되어 있으므로 수정 사항이 주변의 모든 virtualenv 배포판에 도달하는 데 몇 년이 걸릴지는 말할 것도 없습니다. :( :( :(

프로젝트 관리자가 이 문제가 존재한다는 사실을 인정할 것입니까?

잘 모르겠지만 거의 2주 된 PR도 받았습니다. 가장 가능성이 높은 대답은 제출하지 않는다는 것입니다.

나는 irc, twitter, 심지어 메일링 리스트에서 약간의 소음을 만들려고 노력할 것입니다. 수정 사항을 병합할 수 있습니다.

virtualenv는 CI 작업에서 bash 엄격 모드를 기본값으로 만드는 것을 막는 유일한 것입니다.

+1

스크립트 시작 부분에서 nounset 를 비활성화

@jakub-bochenski 아마도 irc에 대한 몇 가지 의견에 도움을 줄 수도 있습니다. virtualenv core dev를 깨우기에 충분한 사용자를 확보할 수 있는지 봅시다.

@ssbarnea 확실하지 않습니다. 나는 오랫동안 IRC에 로그인하지 않았습니다. 메일링 리스트에 입소문이 나도록 도울 수는 있지만

한숨을 쉬다...

+1

7일 전에 pypa-dev에 이메일을 보냈지만 응답을 받지 못했습니다. 또한 어제 누군가가 설치된 win32 바이너리에 트로이 목마가 포함되어 있다고 게시했지만 다시 응답하지 않았습니다. 트로이 목마가 실제로 배포판에 포함되지 않았으면 하고 그것이 나에게 영향을 미치지 않기를 바랍니다.

https://groups.google.com/forum/#!forum/pypa -dev 참조

오늘 이것에 부딪쳤고 어딘가에 내 코드의 버그라고 생각했습니다.
:+1: 단위 테스트에 set -euo pipefail 를 포함합니다.

참고로 위에서 언급한 토론에 대한 직접 링크는 다음과 같습니다. https://groups.google.com/d/topic/pypa-dev/8iVHDOqsj9M/discussion

@pfmoore가 썼다

나는 이미 virtualenv-users 목록에 대해 훨씬 더 자세히 응답했습니다.

.. python-virtualenv 목록으로 판명되었습니다. https://groups.google.com/d/topic/python-virtualenv/5xKG8KoBl6g/discussion

FWIW, 내가 .devkit 에서 사용하는 해결 방법은 source 행에 VIRTUAL_ENV_DISABLE_PROMPT=true 설정하는 것입니다. 프롬프트 설정 동작을 모두 비활성화하기 때문에 PS1 설정하는 것보다 내 사용 사례에 더 잘 작동합니다.

@pjeby @jakub-bochenski @jpuskar @axd1967 이미 이에 대한 버그 수정이 있지만 병합하려면 두 개의 다른 PR을 검토하고 병합해야 합니다. 이는 활성화 테스트를 개선하고 개선해야 하기 때문입니다. 모음곡.

1. https://github.com/pypa/virtualenv/pull/1089 -- py36에서 CI 테스트 활성화 및 py33 삭제
2. https://github.com/pypa/virtualenv/pull/1087 -- CI에서 test_activate.sh 스크립트 사용 활성화
3. https://github.com/pypa/virtualenv/pull/1078 -- 언바운드 PS1 수정이 활성화됨

아마도 당신은 마지막 두 가지가 CI 테스트를 통과하지 못하고 있다는 것을 알게 될 것입니다. 그래서 우리는 다른 것들을 먼저 병합해야 합니다.

검토/댓글 부탁드립니다. virtualenv에는 검토 능력이 부족하기 때문에 다른 프로젝트보다 더 중요합니다. 이것이 제가 https://groups.google.com/d/msg/pypa 에서 관리자가 되라고 요청한 이유 중 하나입니다.

그럼에도 불구하고 내 자신의 변경 사항을 검토할 수 없기 때문에 둘 이상이 필요할 것 같습니다.

@ssbarnea 저희에게 리뷰어 권한도 요청

편집: 분명히 누구든지 PR을 검토할 수 있습니다.

1개 완료 2개 더 남았습니다 :)

이것이 병합되고 공개적으로 사용 가능한 시기에 대해 ETA를 얻을 수 있습니까?

편집 : 여전히이 문제가 발생하고 오늘 아침에 빌드를 중단했습니다.

AWS 람다 기반 이미지 처리 솔루션을 위한 빌드 스크립트를 작업할 때도 이것에 물렸습니다. https://github.com/awslabs/serverless-image-handler/blob/master/deployment/build-s3 -dist.sh

VIRTUAL_ENV_DISABLE_PROMPT=true source env/bin/activate 해결 방법을 사용했습니다.

@duaneking @robinbowes virtualenv 주변에 유지 관리 능력이 부족하며 이 문제를 해결하는 데 도움이 필요하면 https://groups.google.com/forum/#!topic/pypa -dev/SgK9vlu93BY를 읽고 의견을 말

내 인상은 PYPA 팀이 커뮤니티 피드백이 충분한 경우에만 이에 대응할 것이라는 것입니다.

FTR 우리는 여전히 #1087에서 병합을 기다리고 있습니다.

비공식 Bash 엄격 모드 사용 의 첫 번째 예가 무엇인지 맞춰보세요 부적합 문서 소싱 ?

예, python 2 virtualenv입니다.

우분투 16.04

• 작동하지 않음: VIRTUAL_ENV_DISABLE_PROMPT=true
• 작품: https://github.com/bogdando/tripleo-ci/commit/318d17a9e2d94723aa1a0d8e2a9ee345e67032ae

bogdando/ tripleo-ci@318d17a를 사용 하면 이전에 활성화되지 않았더라도 모드를 -u 재정의합니다. 정확히는 모범 사례 구성이 아닙니다.

이렇게 하면 이전 상태가 유지됩니다.

old_setting=${-//[^u]/}
...
if [[ -n "$old_setting" ]]; then set -u; fi

지금은 패치를 사용하는 것이 좋습니다(bash를 사용하거나 필요에 따라 변경).
작성자가 마침내 이 수정 사항을 게시하면 실패(실행 중단)가 시작되어 변경 사항이 명확하게 표시됩니다.

set +H -euo pipefail
pushd "${envdir}"
patch -p0 <<< '
--- bin/activate 2018-10-12 09:08:16.991113929 +0200
+++ bin/activate 2018-10-12 09:27:51.505054528 +0200
@@ -54,11 +54,11 @@
 fi

 if [ -z "${VIRTUAL_ENV_DISABLE_PROMPT-}" ] ; then
-    _OLD_VIRTUAL_PS1="$PS1"
+    _OLD_VIRTUAL_PS1="${PS1:-}"
     if [ "x" != x ] ; then
         PS1="$PS1"
     else
-        PS1="(`basename \"$VIRTUAL_ENV\"`) $PS1"
+        PS1="(`basename \"$VIRTUAL_ENV\"`) ${PS1:-}"
     fi
     export PS1
 fi
'
popd
. "${envdir}/bin/activate"

지금 수정되었습니다 https://github.com/pypa/virtualenv/pull/922