Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

bpo-41173: Copy test results file from ARM worker before uploading#21305

Merged
zooba merged 3 commits into
python:masterpython/cpython:masterfrom
zooba:bpo-41173zooba/cpython:bpo-41173Copy head branch name to clipboard
Jul 7, 2020
Merged

bpo-41173: Copy test results file from ARM worker before uploading#21305
zooba merged 3 commits into
python:masterpython/cpython:masterfrom
zooba:bpo-41173zooba/cpython:bpo-41173Copy head branch name to clipboard

Conversation

@zooba

@zooba zooba commented Jul 3, 2020

Copy link
Copy Markdown
Member

@zooba zooba added OS-windows skip news 🔨 test-with-buildbots Test PR w/ buildbots; report in status section labels Jul 3, 2020
@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @zooba for commit 01bd341 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Jul 3, 2020
Comment thread Tools/buildbot/test.bat
echo where scp
where scp
echo scp %SSH_SERVER%:"%REMOTE_PYTHON_DIR%test-results.xml" "%PYTHON_SOURCE%\test-results.xml"
scp %SSH_SERVER%:"%REMOTE_PYTHON_DIR%test-results.xml" "%PYTHON_SOURCE%\test-results.xml"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"%PYTHON_SOURCE%\test-results.xml" should be "%PYTHON_SOURCE%test-results.xml"
I don't know if the extra slash will matter to scp or not without testing.

Comment thread Tools/buildbot/test.bat Outdated
@@ -37,7 +37,12 @@ set TEMP_ARGS=--temp %REMOTE_PYTHON_DIR%temp

set rt_args=%rt_opts% %dashU% -rwW --slowest --timeout=1200 --fail-env-changed %regrtest_args% %TEMP_ARGS%
ssh %SSH_SERVER% "set TEMP=%REMOTE_PYTHON_DIR%temp& %REMOTE_PYTHON_DIR%PCbuild\rt.bat" %rt_args%

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe change this to
ssh %SSH_SERVER% "set TEMP=%REMOTE_PYTHON_DIR%temp & cd %REMOTE_PYTHON_DIR% & PCbuild\rt.bat" %rt_args%

@zooba zooba merged commit 10772ec into python:master Jul 7, 2020
@zooba zooba deleted the bpo-41173 branch July 7, 2020 23:24
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @zooba for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 7, 2020
…ythonGH-21305)

(cherry picked from commit 10772ec)

Co-authored-by: Steve Dower <steve.dower@python.org>
@bedevere-bot

Copy link
Copy Markdown

GH-21382 is a backport of this pull request to the 3.9 branch.

miss-islington added a commit that referenced this pull request Jul 7, 2020
…H-21305)

(cherry picked from commit 10772ec)

Co-authored-by: Steve Dower <steve.dower@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.