Builder linux-rel-css Build #13575

Results:

Build successful

SourceStamp:

BuildSlave:

Reason:

The AnyBranchScheduler scheduler named 'servo-auto' triggered this build

Steps and Logfiles:

1. git update ( 3 secs )
   1. - no logs -
2. generic generic ( 0 secs )
   1. - no logs -
3. generic_1 generic_1 ( 0 secs )
   1. - no logs -
4. shell 'pkill -x ...' ( 0 secs )
   1. stdio
5. shell__1 './mach clean-nightlies ...' ( 15 secs )
   1. stdio
6. shell__2 './mach clean-cargo-cache ...' ( 0 secs )
   1. stdio
7. shell__3 './etc/ci/clean_build_artifacts.sh' ( 0 secs )
   1. stdio
8. compile ran build --release --with-debug-assertions -p servo ( 12 mins, 53 secs )
   1. - no logs -
9. test ran test-wpt --release --processes 24 --total-chunks 2 --this-chunk 2 --always-succeed ( 11 mins, 57 secs )
   1. - no logs -
10. shell__4 './mach filter-intermittents ...' ( 6 secs )
    1. stdio
    2. filtered-wpt-errorsummary.log
11. test__1 ran test-wpt --release --processes 24 --always-succeed tests/wpt/webgl/tests/conformance ( 2 mins, 18 secs )
    1. webgl-errorsummary.log
12. shell__5 './mach filter-intermittents ...' ( 1 secs )
    1. stdio
    2. webgl-intermittents.log
    3. filtered-webgl-errorsummary.log
13. shell__6 'bash ./etc/ci/lockfile_changed.sh' ( 0 secs )
    1. stdio
14. shell__7 './etc/ci/clean_build_artifacts.sh' ( 0 secs )
    1. stdio

Build Properties:

Forced Build Properties:

Responsible Users:

1. bors-servo

   lbergstrom+borsohnoyoudont@mozilla.com

Timing:

All Changes:

:

1. Change #124164

   Category	None
   Changed by	bors-servo <lbergstrom+borsohnoyoudont@mozilla.com>
   Changed at	Wed 02 Oct 2019 17:03:11
   Repository	https://github.com/servo/servo
   Project	servo/servo
   Branch	auto
   Revision	22e3797fb620b8b24b28abbce781ac0c0aa6ccda

   Comments

   Auto merge of #24343 - Manishearth:pkg-config-gst-uwp, r=jdm
   
   Isolate UWP builds from external gstreamers
   
   Fixes https://github.com/servo/servo/issues/24327
   
   This further seals the UWP build from the non-cross regular environment and makes pkg-config look at the UWP package.
   
   This makes sure external gstreamer stuff doesn't sneak in. We already kinda do this by setting the LIB environment variable, but PKG_CONFIG_PATH also sneaks in and causes problems. Thing is, it turns out that the pkgconfig in gstreamer-uwp isn't enough for a full servo build, since we don't package gstreamer-webrtc, and the gstreamer-webrtc crate requires it to be around, even if we won't end up loading the library at runtime. Stuff has succeeded so far because people have gstreamer installations whose PKG_CONFIG_PATH is pulled in, despite us using a different set of DLLs, which somehow works but sometimes doesn't (I still don't know why).
   
   I've added a fake gstreamer-webrtc-1.0.pc file to both targets in the gstreamer-uwp package with the following contents. It doesn't do anything the other pc files don't, so it doesn't end up pulling in additional libraries, it just exists to convince pkgconfig that we *have* this library (even though we don't), so that the build may succeed (we'll fail at runtime when we try to open WebRTC connections, but those are disabled anyway).
   
   <details>
   
   ```pkgconfig
   
   prefix=c:/gstreamer/1.0/arm64.uwp-release
   exec_prefix=${prefix}
   libdir=${prefix}/lib
   includedir=${prefix}/include/gstreamer-1.0
   datarootdir=${prefix}/share
   datadir=${datarootdir}
   girdir=${datadir}/gir-1.0
   typelibdir=${libdir}/girepository-1.0
   
   Name: Fake GStreamer WebRTC library
   Description: Hacky stand-in for gstreamer-webrtc, does nothing but stops the Rust gstreamer-webrtc crate from failing during build
   Requires: gstreamer-1.0 gstreamer-base-1.0
   Version: 1.16.0
   Libs: -L${libdir}
   Cflags: -I${includedir}
   
   ```
   </details>
   
   r? @jdm
   
   Perhaps don't merge yet, I can't test this until tomorrow (and ideally would like to test this from scratch tomorrow evening)
   
   <!-- 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/24343)
   <!-- Reviewable:end -->

   Changed files

   • python/servo/build_commands.py
   • python/servo/packages.py