Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update test cases for vxWorks #63179

Merged
merged 1 commit into from
Aug 1, 2019
Merged

update test cases for vxWorks #63179

merged 1 commit into from
Aug 1, 2019

Conversation

BaoshanPang
Copy link
Contributor

issue-2214.rs: lgamma is lgamma on vxWorks
ignore process-envs.rs and process-remove-from-env.rs as there is no 'env' on vxWorks

@rust-highfive
Copy link
Collaborator

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 1, 2019
@rust-highfive
Copy link
Collaborator

The job x86_64-gnu-llvm-6.0 of your PR failed (raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.
2019-08-01T00:15:49.4663622Z ##[command]git remote add origin https://github.com/rust-lang/rust
2019-08-01T00:15:49.4842538Z ##[command]git config gc.auto 0
2019-08-01T00:15:49.4927113Z ##[command]git config --get-all http.https://github.com/rust-lang/rust.extraheader
2019-08-01T00:15:49.4999291Z ##[command]git config --get-all http.proxy
2019-08-01T00:15:49.5144161Z ##[command]git -c http.extraheader="AUTHORIZATION: basic ***" fetch --force --tags --prune --progress --no-recurse-submodules --depth=2 origin +refs/heads/*:refs/remotes/origin/* +refs/pull/63179/merge:refs/remotes/pull/63179/merge
---
2019-08-01T00:16:24.8730645Z do so (now or later) by using -b with the checkout command again. Example:
2019-08-01T00:16:24.8732227Z 
2019-08-01T00:16:24.8733707Z   git checkout -b <new-branch-name>
2019-08-01T00:16:24.8734906Z 
2019-08-01T00:16:24.8735989Z HEAD is now at 471b0f1a7 Merge b7bd2ea3a094cfff46a440d6c02561cdac3e0f5a into 8a58268b5ad9c4a240be349a633069d48991eb0c
2019-08-01T00:16:24.8864136Z ##[section]Starting: Collect CPU-usage statistics in the background
2019-08-01T00:16:24.8867245Z ==============================================================================
2019-08-01T00:16:24.8867311Z Task         : Bash
2019-08-01T00:16:24.8867362Z Description  : Run a Bash script on macOS, Linux, or Windows
---
2019-08-01T00:22:44.1228442Z    Compiling serde_json v1.0.40
2019-08-01T00:22:48.5616635Z    Compiling tidy v0.1.0 (/checkout/src/tools/tidy)
2019-08-01T00:22:57.4514876Z     Finished release [optimized] target(s) in 1m 32s
2019-08-01T00:22:57.4589053Z tidy check
2019-08-01T00:22:57.9451258Z tidy error: /checkout/src/test/ui/issues/issue-2214.rs:34: trailing whitespace
2019-08-01T00:22:57.9451695Z tidy error: /checkout/src/test/ui/issues/issue-2214.rs:36: trailing whitespace
2019-08-01T00:22:59.3271986Z some tidy checks failed
2019-08-01T00:22:59.3272155Z 
2019-08-01T00:22:59.3272155Z 
2019-08-01T00:22:59.3279856Z command did not execute successfully: "/checkout/obj/build/x86_64-unknown-linux-gnu/stage0-tools-bin/tidy" "/checkout/src" "/checkout/obj/build/x86_64-unknown-linux-gnu/stage0/bin/cargo" "--no-vendor"
2019-08-01T00:22:59.3280072Z 
2019-08-01T00:22:59.3280102Z 
2019-08-01T00:22:59.3280992Z failed to run: /checkout/obj/build/bootstrap/debug/bootstrap test src/tools/tidy
2019-08-01T00:22:59.3281081Z Build completed unsuccessfully in 0:01:35
2019-08-01T00:22:59.3281081Z Build completed unsuccessfully in 0:01:35
2019-08-01T00:23:00.6209501Z ##[error]Bash exited with code '1'.
2019-08-01T00:23:00.6240443Z ##[section]Starting: Checkout
2019-08-01T00:23:00.6243552Z ==============================================================================
2019-08-01T00:23:00.6243607Z Task         : Get sources
2019-08-01T00:23:00.6243654Z Description  : Get sources from a repository. Supports Git, TfsVC, and SVN repositories.

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

ignore process-envs.rs and process-remove-from-env.rs as there is no 'env' on vxWorks
@alexcrichton
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Aug 1, 2019

📌 Commit 8d93778 has been approved by alexcrichton

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 1, 2019
pietroalbini added a commit to pietroalbini/rust that referenced this pull request Aug 1, 2019
update test cases for vxWorks

issue-2214.rs: lgamma is lgamma on vxWorks
ignore process-envs.rs and process-remove-from-env.rs as there is no 'env' on vxWorks
bors added a commit that referenced this pull request Aug 1, 2019
Rollup of 8 pull requests

Successful merges:

 - #62644 (simplify std::io::Write::write rustdoc)
 - #62971 (Add keywords item into the sidebar)
 - #63122 (Account for `maybe_whole_expr` in range patterns)
 - #63158 (Add test for issue-58951)
 - #63170 (cleanup StringReader fields)
 - #63179 (update test cases for vxWorks)
 - #63188 (Fix typos in release notes.)
 - #63191 (ci: fix toolstate not pushing data for Linux)

Failed merges:

r? @ghost
@bors bors merged commit 8d93778 into rust-lang:master Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants