Skip to content

Conversation

nellshamrell
Copy link
Contributor

Fixes #73535

Signed-off-by: Nell Shamrell nellshamrell@gmail.com

@rust-highfive
Copy link
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @sfackler (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 Jun 25, 2020
@nellshamrell nellshamrell changed the title diable collectionbenches for android disable collectionbenches for android Jun 25, 2020
Signed-off-by: Nell Shamrell <nellshamrell@gmail.com>
@rust-highfive
Copy link
Contributor

The job mingw-check of your PR failed (pretty log, 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.
##[section]Starting: Linux mingw-check ##[section]Starting: Initialize job Agent name: 'Azure Pipelines 64' Agent machine name: 'fv-az578' Current agent version: '2.171.1' ##[group]Operating System 16.04.6 LTS LTS ##[endgroup] ##[group]Virtual Environment Environment: ubuntu-16.04 Version: 20200621.1 Included Software: https://github.com/actions/virtual-environments/blob/ubuntu16/20200621.1/images/linux/Ubuntu1604-README.md ##[endgroup] Agent running as: 'vsts' Prepare build directory. Set build variables. Download all required tasks. Download all required tasks. Downloading task: Bash (3.171.1) Checking job knob settings. Knob: AgentToolsDirectory = /opt/hostedtoolcache Source: ${AGENT_TOOLSDIRECTORY} Knob: AgentPerflog = /home/vsts/perflog Source: ${VSTS_AGENT_PERFLOG} Start tracking orphan processes. ##[section]Finishing: Initialize job ##[section]Starting: Configure Job Name ============================================================================== --- ========================== Starting Command Output =========================== [command]/bin/bash --noprofile --norc /home/vsts/work/_temp/af36f957-1ecc-40ff-8d2b-8c116a8d186a.sh ##[section]Finishing: Disable git automatic line ending conversion ##[section]Starting: Checkout rust-lang/rust@refs/pull/73729/merge to s Task : Get sources Description : Get sources from a repository. Supports Git, TfsVC, and SVN repositories. Version : 1.0.0 Author : Microsoft --- ##[command]git remote add origin https://github.com/rust-lang/rust ##[command]git config gc.auto 0 ##[command]git config --get-all http.https://github.com/rust-lang/rust.extraheader ##[command]git config --get-all http.proxy ##[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/73729/merge:refs/remotes/pull/73729/merge --- ---> a9ec21d337b3 Step 6/7 : ENV RUN_CHECK_WITH_PARALLEL_QUERIES 1 ---> Using cache ---> 5ff2c13d8dba Step 7/7 : ENV SCRIPT python3 ../x.py test src/tools/expand-yaml-anchors && python3 ../x.py check --target=i686-pc-windows-gnu --host=i686-pc-windows-gnu && python3 ../x.py build --stage 0 src/tools/build-manifest && python3 ../x.py test --stage 0 src/tools/compiletest && python3 ../x.py test src/tools/tidy && python3 ../x.py doc --stage 0 src/libstd && /scripts/validate-toolstate.sh ---> 6b931e755c7e Successfully built 6b931e755c7e Successfully tagged rust-ci:latest Built container sha256:6b931e755c7ea1f69816640eb9df74fafd40a545d0e5aa8341d35009dabb0f3c --- Checking rustc_feature v0.0.0 (/checkout/src/librustc_feature) Checking rustc_parse_format v0.0.0 (/checkout/src/librustc_parse_format) Checking rustc_ast_pretty v0.0.0 (/checkout/src/librustc_ast_pretty) Checking rustc_hir v0.0.0 (/checkout/src/librustc_hir) Checking rustc_query_system v0.0.0 (/checkout/src/librustc_query_system) Checking chalk-engine v0.11.0 Checking rustc_hir_pretty v0.0.0 (/checkout/src/librustc_hir_pretty) Checking rustc_parse v0.0.0 (/checkout/src/librustc_parse) Checking rustc_ast_lowering v0.0.0 (/checkout/src/librustc_ast_lowering) --- configure: build.locked-deps := True configure: rust.dist-src := False configure: build.submodules := False configure: rust.debug-assertions := True configure: build.configure-args := ['--enable-sccache', '--disable-manage-submodu ... configure: writing `config.toml` in current directory configure: configure: run `python /checkout/x.py --help` configure: --- Hugepagesize: 2048 kB DirectMap4k: 133056 kB DirectMap2M: 5109760 kB DirectMap1G: 4194304 kB + python3 ../x.py test src/tools/expand-yaml-anchors Ensuring the YAML anchors in the GitHub Actions config were expanded Ensuring the YAML anchors in the GitHub Actions config were expanded Building stage0 tool expand-yaml-anchors (x86_64-unknown-linux-gnu) --- Compiling linked-hash-map v0.5.2 Compiling lazy_static v1.4.0 Compiling yaml-rust v0.4.3 Compiling quote v1.0.2 Compiling thiserror-impl v1.0.5 Compiling thiserror v1.0.5 Compiling yaml-merge-keys v0.4.0 Compiling expand-yaml-anchors v0.1.0 (/checkout/src/tools/expand-yaml-anchors) Build completed successfully in 0:00:30 + python3 ../x.py check --target=i686-pc-windows-gnu --host=i686-pc-windows-gnu Finished dev [unoptimized] target(s) in 0.18s Checking rustdoc artifacts (x86_64-unknown-linux-gnu -> i686-pc-windows-gnu) --- Checking rustc_feature v0.0.0 (/checkout/src/librustc_feature) Checking rustc_parse_format v0.0.0 (/checkout/src/librustc_parse_format) Checking rustc_ast_pretty v0.0.0 (/checkout/src/librustc_ast_pretty) Checking rustc_hir v0.0.0 (/checkout/src/librustc_hir) Checking rustc_query_system v0.0.0 (/checkout/src/librustc_query_system) Checking chalk-engine v0.11.0 Checking rustc_hir_pretty v0.0.0 (/checkout/src/librustc_hir_pretty) Checking rustc_parse v0.0.0 (/checkout/src/librustc_parse) Checking rustc_ast_lowering v0.0.0 (/checkout/src/librustc_ast_lowering) --- tidy check * 608 error codes * highest error code: E0765 * 291 features tidy error: /checkout/src/liballoc/benches/lib.rs:3: platform-specific cfg: cfg(not(target_os = "android")) Found 504 error codes Found 0 error codes with no tests Done! some tidy checks failed some tidy checks failed 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" failed to run: /checkout/obj/build/bootstrap/debug/bootstrap test src/tools/tidy Build completed unsuccessfully in 0:00:36 Build completed unsuccessfully in 0:00:36 == clock drift check == local time: Thu Jun 25 18:37:29 UTC 2020 network time: Thu, 25 Jun 2020 18:37:29 GMT == end clock drift check == ##[error]Bash exited with code '1'. ##[section]Finishing: Run build ##[section]Starting: Checkout rust-lang/rust@refs/pull/73729/merge to s Task : Get sources Description : Get sources from a repository. Supports Git, TfsVC, and SVN repositories. Version : 1.0.0 Author : Microsoft Author : Microsoft Help : [More Information](https://go.microsoft.com/fwlink/?LinkId=798199) ============================================================================== Cleaning any cached credential from repository: rust-lang/rust (GitHub) ##[section]Finishing: Checkout rust-lang/rust@refs/pull/73729/merge to s Cleaning up task key Start cleaning up orphan processes. Terminate orphan process: pid (4156) (python) ##[section]Finishing: Finalize Job 

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 @rust-lang/infra. (Feature Requests)

@sfackler
Copy link
Member

r=me with the tidy lint fixed

Signed-off-by: Nell Shamrell <nellshamrell@gmail.com>
@sfackler
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jun 25, 2020

📌 Commit 5c88b51 has been approved by sfackler

@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 Jun 25, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 26, 2020
…arth Rollup of 13 pull requests Successful merges: - rust-lang#72620 (Omit DW_AT_linkage_name when it is the same as DW_AT_name) - rust-lang#72967 (Don't move cursor in search box when using arrows to navigate results) - rust-lang#73102 (proc_macro: Stop flattening groups with dummy spans) - rust-lang#73297 (Support configurable deny-warnings for all in-tree crates.) - rust-lang#73507 (Cleanup MinGW LLVM linkage workaround) - rust-lang#73588 (Fix handling of reserved registers for ARM inline asm) - rust-lang#73597 (Record span of `const` kw in GenericParamKind) - rust-lang#73629 (Make AssocOp Copy) - rust-lang#73681 (Update Chalk to 0.14) - rust-lang#73707 (Fix links in `SliceIndex` documentation) - rust-lang#73719 (emitter: column width defaults to 140) - rust-lang#73729 (disable collectionbenches for android) - rust-lang#73748 (Add code block to code in documentation of `List::rebase_onto`) Failed merges: r? @ghost
@bors bors merged commit 91a638d into rust-lang:master Jun 26, 2020
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jul 14, 2020
…arth Rollup of 13 pull requests Successful merges: - rust-lang#72620 (Omit DW_AT_linkage_name when it is the same as DW_AT_name) - rust-lang#72967 (Don't move cursor in search box when using arrows to navigate results) - rust-lang#73102 (proc_macro: Stop flattening groups with dummy spans) - rust-lang#73297 (Support configurable deny-warnings for all in-tree crates.) - rust-lang#73507 (Cleanup MinGW LLVM linkage workaround) - rust-lang#73588 (Fix handling of reserved registers for ARM inline asm) - rust-lang#73597 (Record span of `const` kw in GenericParamKind) - rust-lang#73629 (Make AssocOp Copy) - rust-lang#73681 (Update Chalk to 0.14) - rust-lang#73707 (Fix links in `SliceIndex` documentation) - rust-lang#73719 (emitter: column width defaults to 140) - rust-lang#73729 (disable collectionbenches for android) - rust-lang#73748 (Add code block to code in documentation of `List::rebase_onto`) Failed merges: r? @ghost
@cuviper cuviper added this to the 1.46 milestone May 2, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 27, 2024
…r=<try> Re-enable android tests/benches in alloc This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. r? `@ghost` try-job: arm-android try-job: dist-android
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 27, 2024
…r=<try> Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. r? `@ghost` try-job: arm-android try-job: dist-android
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 28, 2024
…r=<try> Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. r? `@ghost` try-job: arm-android try-job: dist-android
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Aug 29, 2024
…, r=tgross35 Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. And I've validated that they should pass now with a try-build :)
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Aug 29, 2024
…, r=tgross35 Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. And I've validated that they should pass now with a try-build :)
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Aug 29, 2024
…, r=tgross35 Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. And I've validated that they should pass now with a try-build :)
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Aug 30, 2024
…, r=tgross35 Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. And I've validated that they should pass now with a try-build :)
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Aug 31, 2024
…, r=tgross35 Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. And I've validated that they should pass now with a try-build :)
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 31, 2024
…, r=tgross35 Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. And I've validated that they should pass now with a try-build :)
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Aug 31, 2024
…, r=tgross35 Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. And I've validated that they should pass now with a try-build :)
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Aug 31, 2024
Rollup merge of rust-lang#129640 - saethlin:unignore-android-in-alloc, r=tgross35 Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. And I've validated that they should pass now with a try-build :)
github-actions bot pushed a commit to rust-lang/miri that referenced this pull request Sep 1, 2024
Re-enable android tests/benches in alloc/core This is basically a revert of rust-lang/rust#73729. These tests better work on android now; it's been 4 years and we don't use dlmalloc on that target anymore. And I've validated that they should pass now with a try-build :)
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.

6 participants