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

Add FreeBSD to `host_triple` #11626

Merged
merged 1 commit into from Jun 5, 2016
Merged

Conversation

@dlrobertson
Copy link
Contributor

dlrobertson commented Jun 5, 2016

Add freebsd to host_triple so that the correct nightly compiler will be downloaded. There are still other crates that need patching to compile, but this is a start.

  • ./mach build -d does not report any errors
    • Other crates need patches
  • ./mach test-tidy does not report any errors
  • These changes do not require tests because: the correct compiler is not downloaded without this commit

Part of: #11625


This change is Reviewable

@highfive
Copy link

highfive commented Jun 5, 2016

Heads up! This PR modifies the following files:

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Jun 5, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Jun 5, 2016

📌 Commit 517023e has been approved by larsbergstrom

@highfive highfive assigned larsbergstrom and unassigned Manishearth Jun 5, 2016
@bors-servo
Copy link
Contributor

bors-servo commented Jun 5, 2016

Testing commit 517023e with merge 4a15406...

bors-servo added a commit that referenced this pull request Jun 5, 2016
…trom

Add FreeBSD to `host_triple`

Add freebsd to `host_triple` so that the correct nightly compiler will be downloaded. There are still other crates that need patching to compile, but this is a start.
---
- [ ] `./mach build -d` does not report any errors
  - Other crates need patches
- [X] `./mach test-tidy` does not report any errors
- [X] These changes do not require tests because: the correct compiler is not downloaded without this commit

Part of: #11625

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

bors-servo commented Jun 5, 2016

💔 Test failed - linux-rel

@highfive
Copy link

highfive commented Jun 5, 2016

  ▶ FAIL [expected PASS] /css-transforms-1_dev/html/transform-abspos-007.htm
  └   → /css-transforms-1_dev/html/transform-abspos-007.htm 71f0313eedfbfcce0b0fcc5ae55f34b1daa3b8d8
/css-transforms-1_dev/html/reference/transform-abspos-ref.htm 78d197606924062e8dd2a773c977afcecf8940f8
Testing 71f0313eedfbfcce0b0fcc5ae55f34b1daa3b8d8 == 78d197606924062e8dd2a773c977afcecf8940f8

  ▶ TIMEOUT [expected PASS] /css21_dev/html4/color-009.htm
  │ 
  │ Xlib:  extension &#34;XFree86-VidModeExtension&#34; missing on display &#34;:0&#34;.
  │ Shutting down the Constellation after generating an output file or exit flag specified
  │ Xlib:  extension &#34;XFree86-VidModeExtension&#34; missing on display &#34;:0&#34;.
  │ thread &#39;LayoutWorker worker 2/12&#39; panicked at &#39;called `Result::unwrap()` on an `Err` value: IoError(Error { repr: Os { code: 104, message: &#34;Connection reset by peer&#34; } })&#39;, ../src/libcore/result.rs:785
  │ stack backtrace:
  │    1:     0x7f69612f1230 - std::sys::backtrace::tracing::imp::write::h9fb600083204ae7f
  │    2:     0x7f69612f8e5b - std::panicking::default_hook::_$u7b$$u7b$closure$u7d$$u7d$::hca543c34f11229ac
  │    3:     0x7f69612f8ac3 - std::panicking::default_hook::hc2c969e7453d080c
  │    4:     0x7f6961016be7 - util::panicking::initiate_panic_hook::_$u7b$$u7b$closure$u7d$$u7d$::_$u7b$$u7b$closure$u7d$$u7d$::he2b22674ad1748f3
  │    5:     0x7f69612dee1c - std::panicking::rust_panic_with_hook::hfe203e3083c2b544
  │    6:     0x7f69612f90a1 - std::panicking::begin_panic::h4889569716505182
  │    7:     0x7f69612e087a - std::panicking::begin_panic_fmt::h484cd47786497f03
  │    8:     0x7f69612f903e - rust_begin_unwind
  │    9:     0x7f696132f63f - core::panicking::panic_fmt::h257ceb0aa351d801
  │   10:     0x7f6960c82d03 - core::result::unwrap_failed::h1d94443d858a91df
  │   11:     0x7f6960ca3924 - gfx::font_cache_thread::FontCacheThread::find_font_template::h5b87de1d4383ad19
  │   12:     0x7f6960ca8874 - gfx::font_context::FontContext::layout_font_group_for_style::habc798e1e1f378d1
  │   13:     0x7f695f922d02 - layout::text::font_metrics_for_style::h47d9ef1163e2ab90
  │   14:     0x7f695f922c58 - layout::fragment::Fragment::calculate_line_height::h2fd722b6e5d0ee10
  │   15:     0x7f695f924646 - layout::fragment::Fragment::inline_metrics::hb73a427b21f4f6d6
  │   16:     0x7f695f929210 - layout::inline::LineBreaker::new_block_size_for_line::hb4e8eab4e7d35075
  │   17:     0x7f695f92b796 - _&lt;layout..inline..InlineFlow as layout..flow..Flow&gt;::assign_block_size::hf126b56b43fc68dc
  │   18:     0x7f695fa0e003 - _&lt;layout..traversal..AssignISizes&lt;&#39;a&gt; as layout..parallel..ParallelPreorderFlowTraversal&gt;::run_parallel::hc10616c74c94e0bf
  │   19:     0x7f695fa0e7bf - layout::parallel::assign_inline_sizes::h917782c09b72a5f7
  │   20:     0x7f695f956036 - std::panicking::try::call::ha4fefec3c980f4a5
  │   21:     0x7f696130329b - __rust_try
  │   22:     0x7f696130323e - __rust_maybe_catch_panic
  │   23:     0x7f695f9578c7 - _&lt;F as alloc..boxed..FnBox&lt;A&gt;&gt;::call_box::h82fb0d1a3a0ebea9
  │   24:     0x7f69612f7244 - std::sys::thread::Thread::new::thread_start::h6f266e069bf4ec2b
  │   25:     0x7f695d15d181 - start_thread
  │   26:     0x7f695cc7447c - __clone
  └   27:                0x0 - &lt;unknown&gt;
Add FreeBSD to `host_triple` so that the correct nightly compiler will
be downloaded.
@dlrobertson dlrobertson force-pushed the dlrobertson:freebsd-host-triple branch from 517023e to 5fc1b6c Jun 5, 2016
@highfive
Copy link

highfive commented Jun 5, 2016

New code was committed to pull request.

@dlrobertson
Copy link
Contributor Author

dlrobertson commented Jun 5, 2016

rebased

@mbrubeck
Copy link
Contributor

mbrubeck commented Jun 5, 2016

@mbrubeck
Copy link
Contributor

mbrubeck commented Jun 5, 2016

@bors-servo r=larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Jun 5, 2016

📌 Commit 5fc1b6c has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Jun 5, 2016

Testing commit 5fc1b6c with merge fada22a...

bors-servo added a commit that referenced this pull request Jun 5, 2016
…trom

Add FreeBSD to `host_triple`

Add freebsd to `host_triple` so that the correct nightly compiler will be downloaded. There are still other crates that need patching to compile, but this is a start.
---
- [ ] `./mach build -d` does not report any errors
  - Other crates need patches
- [X] `./mach test-tidy` does not report any errors
- [X] These changes do not require tests because: the correct compiler is not downloaded without this commit

Part of: #11625

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

bors-servo commented Jun 5, 2016

@bors-servo bors-servo merged commit 5fc1b6c into servo:master Jun 5, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@dlrobertson dlrobertson deleted the dlrobertson:freebsd-host-triple branch Jun 5, 2016
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.