New issue
Advanced search Search tips

Issue 876139 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 22
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 871418



Sign in to add a comment

win_upload_clang try bot fails test suite with "tail command not found" errors

Project Member Reported by r...@chromium.org, Aug 21

Issue description

https://ci.chromium.org/buildbot/tryserver.chromium.win/win_upload_clang/374

Errors look like:
$ ":" "RUN: at line 3"
$ "tail" "-n" "+11" "C:\b\rr\tmpikhea3\w\src\third_party\llvm-bootstrap\tools\clang\test\Analysis\Output\retain-release-path-notes.m.tmp"
# command stderr:
'tail': command not found
error: command failed with exit status: 127

The bot is configured with a two machine pool, and I suspect that one of the bots has the right tools installed and the other does not. 

Most builds use build4-m4 and succeed, but my build was with build6-m4, and failed.
 
Both bots should pull gnuwin-7 from http://commondatastorage.googleapis.com/chromium-browser-clang/index.html?path=tools/ . It does not contain `tail`.  Not sure why it works on build4-m4.

I'm guessing the dependency of retain-release-path-notes on tail is new?
Owner: h...@chromium.org
Status: Assigned (was: Untriaged)
Adding tail to our gnuwin zip should be easy, I'll take a look.
The new failures appear to be due to http://llvm.org/viewvc/llvm-project?rev=339475&view=rev
Blockedon: 871418
Blocking: 871418
Blockedon: -871418
Project Member

Comment 8 by bugdroid1@chromium.org, Aug 22

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b58a10e16c9e233e91c632209188ade2714bff8a

commit b58a10e16c9e233e91c632209188ade2714bff8a
Author: Hans Wennborg <hans@chromium.org>
Date: Wed Aug 22 00:24:31 2018

Clang update script: use gnuwin-8 package for building on Windows

It contains 'tail' in addition to the contents of gnuwin-7, to be able
to run some tests that were updated in r339475.

NOTRY=true

Bug:  876139 
Change-Id: I447d79591b01de309b258a54a37cb5603599201b
Reviewed-on: https://chromium-review.googlesource.com/1183909
Commit-Queue: Hans Wennborg <hans@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584932}
[modify] https://crrev.com/b58a10e16c9e233e91c632209188ade2714bff8a/tools/clang/scripts/update.py

Status: Fixed (was: Started)

Sign in to add a comment