Builder linux32_sde Build 8924327179749454528

Results:

Success

Trigger Info:

Project	boringssl
Revision	de3c1f69cc8422f94e4415b34398f3be29fa1f83
Got Revision	de3c1f69cc8422f94e4415b34398f3be29fa1f83

Execution:

• Source: Task 4266bb6a7fb87810
• Bot: gce-trusty-e833d7b0-us-west1-c-pq1h
• Recipe: boringssl

Steps and Logfiles:

Show: Expanded Default Non-Green

1. ( 140 ms ) setup_build

   running recipe: "boringssl"
   • stdout
   • run_recipe
2. ( 7 ms ) uname
   • stdout
3. ( 37 secs ) bot_update

   [18GB/295GB used (6%)]
   • stdout
   • json.output
4. ( 13 secs ) gclient runhooks
   • stdout
5. ( 35 ms ) clean
   • stdout
6. ( 35 ms ) mkdir
   • stdout
7. ( 7 secs ) cmake
   • stdout
8. ( 29 secs ) ninja
   • stdout
9. ( 248 ms ) check filenames
   • stdout
10. ( 13 mins 57 secs ) unit tests
    • stdout
    • test_utils.test_results
11. ( 0 ) recipe result
    • result

Build Properties:

Name	Value	Source
$recipe_engine/path	{"cache_dir": "/b/s/w/ir/cache", "temp_dir": "/b/s/w/ir/tmp/rt"}	setup_build
$recipe_engine/runtime	{"is_experimental": false, "is_luci": true}	setup_build
bot_id	"gce-trusty-e833d7b0-us-west1-c-pq1h"	setup_build
branch	"refs/heads/master"	setup_build
buildbucket	{"build": {"bucket": "luci.boringssl.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1547495399674235, "id": "8924327179749454528", "project": "boringssl", "tags": ["builder:linux32_sde", "buildset:commit/git/de3c1f69cc8422f94e4415b34398f3be29fa1f83", "buildset:commit/gitiles/boringssl.googlesource.com/boringssl/+/de3c1f69cc8422f94e4415b34398f3be29fa1f83", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9089757242052741120", "scheduler_job_id:boringssl/linux32_sde", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"}	setup_build
buildername	"linux32_sde"	setup_build
path_config	"generic"	setup_build
recipe	"boringssl"	setup_build
repository	"https://boringssl.googlesource.com/boringssl.git"	setup_build
revision	"de3c1f69cc8422f94e4415b34398f3be29fa1f83"	setup_build
got_revision	"de3c1f69cc8422f94e4415b34398f3be29fa1f83"	bot_update

Blamelist:

1. Tom Tan (Tom.Tanohnoyoudont@microsoft.com)

Timing:

Create	Monday, 14-Jan-19 19:49:59 UTC
Start	Monday, 14-Jan-19 19:50:07 UTC
End	Monday, 14-Jan-19 20:05:45 UTC
Pending	7 secs
Execution	15 mins 37 secs

All Changes:

1. Fix header file for _byteswap_ulong and _byteswap_uint64 from MSVC CRT

   Changed by	Tom Tan - Tom.Tanohnoyoudont@microsoft.com
   Changed at	Monday, 14-Jan-19 19:49:39 UTC
   Repository	https://boringssl.googlesource.com/boringssl
   Branch
   Revision	de3c1f69cc8422f94e4415b34398f3be29fa1f83

   Comments

   Fix header file for _byteswap_ulong and _byteswap_uint64 from MSVC CRT
   
   _byteswap_ulong and _byteswap_uint64 are documented (see below link) as coming from stdlib.h.
    On some build configurations stdlib.h is pulled in by intrin.h but that is not guaranteed. In particular,
   this assumption causes build breaks when building Chromium for Windows ARM64 with clang-cl. This
    change switches the #include to use the documented header file, thus fixing Windows ARM64 with clang-cl.
   
   
   https://docs.microsoft.com/en-us/cpp/c-runtime-library/reference/byteswap-uint64-byteswap-ulong-byteswap-ushort
   
   Bug: chromium:893460
   Change-Id: I738c7227a9e156c894c2be62b52228a5bbd88414
   Reviewed-on: https://boringssl-review.googlesource.com/c/34244
   Reviewed-by: David Benjamin <davidben@google.com>
   Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
   Commit-Queue: David Benjamin <davidben@google.com>
   

   Changed files

   • crypto/internal.h