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

Fix Android CI failing to download SDKs #57857

Merged
merged 1 commit into from
Jan 23, 2019

Conversation

pietroalbini
Copy link
Member

@pietroalbini pietroalbini commented Jan 23, 2019

A component of the Android SDK now requires an additional license (full license text) to be accepted before it's possible to use it. The license is dated January 16th 2019, so it's recent.

The weird thing about the license is that it doesn't prompt you to accept it during sdkmanager --licenses like all the other ones, but during sdkmanager platform-tools emulator ..., and we didn't pipe yes to it before this PR.

The PR changes the SDK installation script to accept all the licenses even on the sdkmanager platform-tools emulator command.

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 23, 2019
@pietroalbini
Copy link
Member Author

@bors p=1000 try

@bors
Copy link
Contributor

bors commented Jan 23, 2019

⌛ Trying commit 3f8bc7a with merge 8a1076e...

bors added a commit that referenced this pull request Jan 23, 2019
[WIP] Fix Android CI failing to download SDKs

TODO
@pietroalbini
Copy link
Member Author

Ok, the fix is working on the try build (the SDK was downloaded without a problem). I'll look around if there is any public announcement from Google about the license change.

r? @Mark-Simulacrum
cc @rust-lang/infra

@pietroalbini pietroalbini changed the title [WIP] Fix Android CI failing to download SDKs Fix Android CI failing to download SDKs Jan 23, 2019
@pietroalbini pietroalbini added the T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. label Jan 23, 2019
@pietroalbini
Copy link
Member Author

Nope, doesn't seem like there is a public announcement about the change. Those seems to be the standard SDK license terms though, so I guess they just added the license prompt to a component that didn't have the prompt before.

@aidanhs
Copy link
Member

aidanhs commented Jan 23, 2019

@bors r+

@bors
Copy link
Contributor

bors commented Jan 23, 2019

📌 Commit 91f328f has been approved by aidanhs

@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 Jan 23, 2019
@bors
Copy link
Contributor

bors commented Jan 23, 2019

⌛ Testing commit 91f328f with merge 23409a9f53daadac3e5851bfee6981777c8898e3...

@pietroalbini
Copy link
Member Author

Bug on Google's issue tracker: https://issuetracker.google.com/issues/123054726

@RalfJung
Copy link
Member

Bug on Google's issue tracker: issuetracker.google.com/issues/123054726

Is there a public version of this that's not behind a "pay-with-your-data" wall?

@pietroalbini
Copy link
Member Author

Is there a public version of this that's not behind a "pay-with-your-data" wall?

Unfortunately no, the Android issue tracker requires auth :(

@bors
Copy link
Contributor

bors commented Jan 23, 2019

💔 Test failed - status-appveyor

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 23, 2019
@pietroalbini
Copy link
Member Author

@bors retry

I'm sure I didn't break cargo on windows with this PR...

@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 Jan 23, 2019
@bors
Copy link
Contributor

bors commented Jan 23, 2019

⌛ Testing commit 91f328f with merge 5d0e5bd...

bors added a commit that referenced this pull request Jan 23, 2019
Fix Android CI failing to download SDKs

A component of the Android SDK now requires an additional license ([full license text](https://gist.github.com/pietroalbini/28b46a6fed0921d129de58e7aef29f11)) to be accepted before it's possible to use it. The license is dated January 16th 2019, so it's recent.

The weird thing about the license is that it doesn't prompt you to accept it during `sdkmanager --licenses` like all the other ones, but during `sdkmanager platform-tools emulator ...`, and we didn't pipe `yes` to it before this PR.

The PR changes the SDK installation script to accept all the licenses even on the `sdkmanager platform-tools emulator` command.
@pietroalbini pietroalbini added beta-nominated Nominated for backporting to the compiler in the beta channel. beta-accepted Accepted for backporting to the compiler in the beta channel. stable-nominated Nominated for backporting to the compiler in the stable channel. stable-accepted Accepted for backporting to the compiler in the stable channel. labels Jan 23, 2019
@pietroalbini
Copy link
Member Author

Accepted for backport everywhere to make sure the fix is also applied to our other branches.

@bors
Copy link
Contributor

bors commented Jan 23, 2019

💔 Test failed - checks-travis

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 23, 2019
@rust-highfive
Copy link
Collaborator

The job dist-x86_64-linux of your PR failed on Travis (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.
[00:00:57] 
[00:00:57] Total download size: 4.9 M
[00:00:57] Downloading Packages:
[00:01:00] --------------------------------------------------------------------------------
[00:01:00] Total                                           1.6 MB/s | 4.9 MB     00:03     
[00:01:00] warning: rpmts_HdrFromFdno: Header V3 DSA signature: NOKEY, key ID e8562897
[00:01:00] Importing GPG key 0xE8562897 "CentOS-5 Key (CentOS 5 Official Signing Key) <[email protected]>" from /etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-5
[00:01:01] Running Transaction Test
[00:01:01] Finished Transaction Test
[00:01:01] Transaction Test Succeeded
[00:01:01] Running Transaction
---
[00:03:44] + hide_output make install
[00:03:44] + set +x
[00:04:05] shared.sh: line 1:   352 Terminated              bash -c "while true; do sleep 30; echo \$(date) - building ...; done"
[00:04:05] + cd ..
[00:04:05] + rm -rf openssl-1.0.2k
[00:04:05] ./build-openssl.sh: line 16:  4115 Terminated              bash -c "while true; do sleep 30; echo \$(date) - building ...; done"  (wd: /tmp/openssl-1.0.2k)
[00:04:05] + ln -nsf /etc/pki/tls/cert.pem /rustroot/ssl/
[00:04:06]  ---> 394da2f4bf6c
[00:04:06] Step 14/41 : COPY dist-x86_64-linux/build-curl.sh /tmp/
[00:04:06]  ---> 347feff5f43d
[00:04:06] Step 15/41 : RUN ./build-curl.sh
[00:04:06] Step 15/41 : RUN ./build-curl.sh
[00:04:06]  ---> Running in 5b854b24d2de
[00:04:07] + source shared.sh
[00:04:07] + VERSION=7.51.0
[00:04:07] + curl http://cool.haxx.se/download/curl-7.51.0.tar.bz2
[00:04:08]   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
[00:04:08]                                  Dload  Upload   Total   Spent    Left  Speed
[00:04:10] 
  0 2509k    0 14215    0     0   9199      0  0:04:39  0:00:01  0:04:38  9199
  0 2509k    0 14215    0     0   9199      0  0:04:39  0:00:01  0:04:38  9199
  4 2509k    4  114k    0     0  61306      0  0:00:41  0:00:01  0:00:40  271k
 94 2509k   94 2360k    0     0   838k      0  0:00:02  0:00:02 --:--:-- 1846k
100 2509k  100 2509k    0     0   885k      0  0:00:02  0:00:02 --:--:-- 1936k
[00:04:10] + mkdir curl-build
[00:04:10] + cd curl-build
[00:04:10] + hide_output ../curl-7.51.0/configure --prefix=/rustroot --with-ssl=/rustroot --disable-sspi --disable-gopher --disable-smtp --disable-smb --disable-imap --disable-pop3 --disable-tftp --disable-telnet --disable-manual --disable-dict --disable-rtsp --disable-ldaps --disable-ldap
[00:04:34] + hide_output make -j10
[00:04:34] + set +x
[00:04:47] shared.sh: line 1:    13 Terminated              bash -c "while true; do sleep 30; echo \$(date) - building ...; done"
[00:04:47] + hide_output make install
---
 96 82.1M   96 79.3M    0     0  2736k      0  0:00:30  0:00:29  0:00:01 2503k
 99 82.1M   99 81.6M    0     0  2721k      0  0:00:30  0:00:30 --:--:-- 2565k
100 82.1M  100 82.1M    0     0  2720k      0  0:00:30  0:00:30 --:--:-- 2660k
[00:09:02] + cd gcc-4.8.5
[00:09:02] + sed -i 's|ftp://gcc\.gnu\.org/|http://gcc.gnu.org/|g' ./contrib/download_prerequisites
[00:09:02] --2019-01-23 13:03:04--  http://gcc.gnu.org/pub/gcc/infrastructure/mpfr-2.4.2.tar.bz2
[00:09:02] Resolving gcc.gnu.org... 209.132.180.131
[00:09:02] Connecting to gcc.gnu.org|209.132.180.131|:80... connected.
[00:09:02] HTTP request sent, awaiting response... 200 OK
---
[00:45:19]  ---> 1439f877b702
[00:45:19] Step 25/41 : RUN ./build-clang.sh
[00:45:19]  ---> Running in e86e242ca106
[00:45:19] + source shared.sh
[00:45:19] + LLVM=032b00a5404865765cda7db3039f39d54964d8b0
[00:45:19] + LLD=3e4aa4e8671523321af51449e0569f455ef3ad43
[00:45:19] + CLANG=a6b9739069763243020f4ea6fe586bc135fde1f9
[00:45:19] + mkdir clang
[00:45:19] + cd clang
[00:45:19] + tar xzf - --strip-components=1
[00:45:19] + curl -L https://github.com/llvm-mirror/llvm/archive/032b00a5404865765cda7db3039f39d54964d8b0.tar.gz
[00:45:19]                                  Dload  Upload   Total   Spent    Left  Speed
[00:45:19] 
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100   158    0   158    0     0    806      0 --:--:-- --:--:-- --:--:--   844
---
100 9504k    0 9504k    0     0  4535k      0 --:--:--  0:00:02 --:--:-- 5466k
100 14.2M    0 14.2M    0     0  4691k      0 --:--:--  0:00:03 --:--:-- 5242k
100 17.1M    0 17.1M    0     0  4968k      0 --:--:--  0:00:03 --:--:-- 5548k
[00:45:32] + mkdir -p tools/lld
[00:45:32] + curl -L https://github.com/llvm-mirror/lld/archive/3e4aa4e8671523321af51449e0569f455ef3ad43.tar.gz
[00:45:32] + tar zxf - --strip-components=1 -C tools/lld
[00:45:32]                                  Dload  Upload   Total   Spent    Left  Speed
[00:45:32] 
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100   157    0   157    0     0    834      0 --:--:-- --:--:-- --:--:--   848
---
[00:45:33] + cd clang-build
[00:45:33] + INC=/rustroot/include
[00:45:33] + INC=/rustroot/include:/rustroot/lib/gcc/x86_64-unknown-linux-gnu/4.8.5/include-fixed
[00:45:33] + INC=/rustroot/include:/rustroot/lib/gcc/x86_64-unknown-linux-gnu/4.8.5/include-fixed:/usr/include
[00:45:33] + hide_output cmake .. -DCMAKE_C_COMPILER=/rustroot/bin/gcc -DCMAKE_CXX_COMPILER=/rustroot/bin/g++ -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/rustroot -DLLVM_TARGETS_TO_BUILD=X86 -DC_INCLUDE_DIRS=/rustroot/include:/rustroot/lib/gcc/x86_64-unknown-linux-gnu/4.8.5/include-fixed:/usr/include
[00:46:03] Wed Jan 23 13:40:04 UTC 2019 - building ...
[00:46:08] + hide_output make -j10
[00:46:08] + set +x
[00:46:38] Wed Jan 23 13:40:40 UTC 2019 - building ...
---
[01:50:35]  ---> 9d169687f58e
[01:50:35] Step 32/41 : RUN ./build-perl.sh
[01:50:35]  ---> Running in e79c2f7c164f
[01:50:36] + source shared.sh
[01:50:36] + curl https://www.cpan.org/src/5.0/perl-5.28.0.tar.gz
[01:50:36]   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
[01:50:36]                                  Dload  Upload   Total   Spent    Left  Speed
[01:50:37] 
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
---
[02:58:41]    Compiling quote v0.6.10
[02:58:42]    Compiling syn v0.15.22
[02:58:48] [RUSTC-TIMING] serde test:false 11.601
[02:58:48]    Compiling serde_json v1.0.33
The job exceeded the maximum time limit for jobs, and has been terminated.

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)

@pietroalbini
Copy link
Member Author

@bors retry please

@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 Jan 23, 2019
@bors
Copy link
Contributor

bors commented Jan 23, 2019

⌛ Testing commit 91f328f with merge 19f8958...

bors added a commit that referenced this pull request Jan 23, 2019
Fix Android CI failing to download SDKs

A component of the Android SDK now requires an additional license ([full license text](https://gist.github.com/pietroalbini/28b46a6fed0921d129de58e7aef29f11)) to be accepted before it's possible to use it. The license is dated January 16th 2019, so it's recent.

The weird thing about the license is that it doesn't prompt you to accept it during `sdkmanager --licenses` like all the other ones, but during `sdkmanager platform-tools emulator ...`, and we didn't pipe `yes` to it before this PR.

The PR changes the SDK installation script to accept all the licenses even on the `sdkmanager platform-tools emulator` command.
@nagisa
Copy link
Member

nagisa commented Jan 23, 2019

@bors
Copy link
Contributor

bors commented Jan 23, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: aidanhs
Pushing 19f8958 to master...

@bors bors merged commit 91f328f into rust-lang:master Jan 23, 2019
@pietroalbini
Copy link
Member Author

🎉

@bors treeclosed-

@pietroalbini pietroalbini removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jan 24, 2019
@pietroalbini pietroalbini deleted the fix-android-ci branch January 24, 2019 16:12
bors added a commit that referenced this pull request Jan 24, 2019
[beta] Rollup backports

* #57857: Fix Android CI failing to download SDKs
* #57840: Fix issue 57762
* #57698: Fix typo bug in DepGraph::try_mark_green().

r? @ghost
@pietroalbini pietroalbini removed stable-accepted Accepted for backporting to the compiler in the stable channel. labels Feb 20, 2019
@Mark-Simulacrum Mark-Simulacrum removed the stable-nominated Nominated for backporting to the compiler in the stable channel. label Feb 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants