whatwg / build.whatwg.org

Build server for running whatwg/wattsi
Creative Commons Zero v1.0 Universal
12 stars 12 forks source link

Removing errant markup reports "Binary file...matches" #9

Closed kenrussell closed 6 years ago

kenrussell commented 6 years ago

Attempting to remove an incorrect 'rowspan="2"' annotation from a <tr> element in this commit:

https://github.com/whatwg/html/pull/3971/commits/6a4aaba3470d5fc3974a96401d1b780c4fe17414

and attempting to run build.sh, the wattsi-server reported:

Local wattsi is not present; trying the build server...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 14.1M    0 6474k  100 7994k   351k   433k  0:00:18  0:00:18 --:--:--     0
Binary file /Users/kbr/src/kenrussell/whatwg/html-build/.temp/wattsi-output.txt matches

This happened even after removing the .temp directory.

domenic commented 6 years ago

This should only occur with out of date (i.e., day old) versions of the html-build repository. Did the update check not kick in? We moved servers and the headers switching to lowercase broke some logic in the build.sh script, but I updated it...

kenrussell commented 6 years ago

Yes, confirmed that updating my html-build repo to ToT fixed this issue.