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

Added liblzma-dev dependency on Debian #23248

Merged
merged 2 commits into from Apr 24, 2019

Conversation

@asajeffrey
Copy link
Member

asajeffrey commented Apr 23, 2019

This fixes a link error on Debian Linux

  = note: /usr/bin/ld.gold: error: cannot find -llzma
          collect2: error: ld returned 1 exit status

  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes do not require tests because it's a README

This change is Reviewable

@jdm
Copy link
Member

jdm commented Apr 23, 2019

Could you update

# Please keep these in sync with the packages in README.md
as well?

@asajeffrey
Copy link
Member Author

asajeffrey commented Apr 23, 2019

Done.

@jdm
Copy link
Member

jdm commented Apr 23, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Apr 23, 2019

📌 Commit 61f7f12 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Apr 23, 2019

Testing commit 61f7f12 with merge d8ab129...

bors-servo added a commit that referenced this pull request Apr 23, 2019
Added liblzma-dev dependency on Debian

<!-- Please describe your changes on the following line: -->

This fixes a link error on Debian Linux
```
  = note: /usr/bin/ld.gold: error: cannot find -llzma
          collect2: error: ld returned 1 exit status
```

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes do not require tests because it's a README

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/23248)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Apr 24, 2019

💔 Test failed - status-taskcluster

@asajeffrey
Copy link
Member Author

asajeffrey commented Apr 24, 2019

One of the taskcluster Mac runs timed out:

[taskcluster:error] Aborting task...
[taskcluster 2019-04-24T01:35:16.891Z] === Task Finished ===
[taskcluster 2019-04-24T01:35:16.892Z] Task Duration: 1h0m0.03685367s
[taskcluster 2019-04-24T01:35:17.743Z] Uploading artifact public/test-wpt-mp.log from file repo/test-wpt-mp.log with content encoding "gzip", mime type "text/plain" and expiry 2020-04-22T23:55:09.914Z
[taskcluster 2019-04-24T01:35:18.741Z] Uploading artifact public/wpt-mp-errorsummary.log from file repo/wpt-mp-errorsummary.log with content encoding "gzip", mime type "text/plain" and expiry 2020-04-22T23:55:09.914Z
[taskcluster 2019-04-24T01:35:19.120Z] Uploading artifact public/test-wpt.log from file repo/test-wpt.log with content encoding "gzip", mime type "text/plain" and expiry 2020-04-22T23:55:09.914Z
[taskcluster 2019-04-24T01:35:27.915Z] Uploading artifact public/wpt-errorsummary.log from file repo/wpt-errorsummary.log with content encoding "gzip", mime type "text/plain" and expiry 2020-04-22T23:55:09.914Z
[taskcluster 2019-04-24T01:35:28.360Z] Uploading artifact public/wpt-errorsummary.log from file repo/wpt-errorsummary.log with content encoding "gzip", mime type "text/plain" and expiry 2020-04-22T23:55:09.914Z
[taskcluster:error] Uploading error artifact public/intermittents.log from file repo/intermittents.log with message "Could not read file '/Users/worker/tasks/task_1556066072/repo/intermittents.log'", reason "file-missing-on-worker" and expiry 2020-04-22T23:55:09.914Z
[taskcluster:error] TASK FAILURE during artifact upload: file-missing-on-worker: Could not read file '/Users/worker/tasks/task_1556066072/repo/intermittents.log'
[taskcluster:error] Uploading error artifact public/filtered-wpt-errorsummary.log from file repo/filtered-wpt-errorsummary.log with message "Could not read file '/Users/worker/tasks/task_1556066072/repo/filtered-wpt-errorsummary.log'", reason "file-missing-on-worker" and expiry 2020-04-22T23:55:09.914Z
[taskcluster:error] TASK FAILURE during artifact upload: file-missing-on-worker: Could not read file '/Users/worker/tasks/task_1556066072/repo/filtered-wpt-errorsummary.log'
[taskcluster 2019-04-24T01:35:29.628Z] Uploading redirect artifact public/logs/live.log to URL https://queue.taskcluster.net/v1/task/Mhgo87xbTwqDJTz2i6dc7g/runs/0/artifacts/public/logs/live_backing.log with mime type "text/plain; charset=utf-8" and expiry 2020-04-22T23:55:09.914Z
[taskcluster:error] Task aborted - max run time exceeded

@bors-servo retry

@bors-servo
Copy link
Contributor

bors-servo commented Apr 24, 2019

💣 Failed to start rebuilding: Unknown error

@bors-servo
Copy link
Contributor

bors-servo commented Apr 24, 2019

Testing commit 61f7f12 with merge ec680de...

bors-servo added a commit that referenced this pull request Apr 24, 2019
Added liblzma-dev dependency on Debian

<!-- Please describe your changes on the following line: -->

This fixes a link error on Debian Linux
```
  = note: /usr/bin/ld.gold: error: cannot find -llzma
          collect2: error: ld returned 1 exit status
```

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes do not require tests because it's a README

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/23248)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Apr 24, 2019

💔 Test failed - mac-rel-wpt3

@CYBAI
Copy link
Collaborator

CYBAI commented Apr 24, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Apr 24, 2019

@bors-servo
Copy link
Contributor

bors-servo commented Apr 24, 2019

@bors-servo bors-servo merged commit 61f7f12 into servo:master Apr 24, 2019
4 checks passed
4 checks passed
Taskcluster (pull_request) TaskGroup: success
Details
Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.