platform/upstream/ninja.git
11 years agodeps log: recover on truncated entry
Evan Martin [Sat, 27 Apr 2013 20:39:04 +0000 (13:39 -0700)]
deps log: recover on truncated entry

If a read fails while reading an entry, truncate the log to the last
successfully read entry.  This prevents corruption when a subsequent
run appends another entry.

11 years agodisable SubprocessTest.SetWithLots on travis-ci
Evan Martin [Fri, 26 Apr 2013 18:25:21 +0000 (11:25 -0700)]
disable SubprocessTest.SetWithLots on travis-ci

We run into fork() limits on their VM.

11 years agoadd a test for truncated deps
Evan Martin [Fri, 26 Apr 2013 18:21:57 +0000 (11:21 -0700)]
add a test for truncated deps

This doesn't yet exhibit the full problems with truncation, but it's
a start.

11 years agomake DepsLogTest actually clean up on teardown
Evan Martin [Fri, 26 Apr 2013 18:02:08 +0000 (11:02 -0700)]
make DepsLogTest actually clean up on teardown

I think I commented this out when tracking down a bug and forgot
to uncomment it.

11 years agotravis: pull in gtest before building
Evan Martin [Mon, 22 Apr 2013 15:12:09 +0000 (08:12 -0700)]
travis: pull in gtest before building

11 years agoadd a .travis.yml, trying out travis-ci.org
Evan Martin [Mon, 22 Apr 2013 15:06:37 +0000 (08:06 -0700)]
add a .travis.yml, trying out travis-ci.org

11 years agoMerge pull request #552 from nico/deps_dryrun
Evan Martin [Mon, 22 Apr 2013 15:05:01 +0000 (08:05 -0700)]
Merge pull request #552 from nico/deps_dryrun

Don't record deps in dry runs.

11 years agokeep dry run more similar to normal run
Nico Weber [Mon, 22 Apr 2013 05:03:20 +0000 (22:03 -0700)]
keep dry run more similar to normal run

11 years agoDon't record deps in dry runs.
Nico Weber [Mon, 22 Apr 2013 00:55:10 +0000 (17:55 -0700)]
Don't record deps in dry runs.

deps_log() is NULL during dry runs, so this fixes a crash. It also
matches ninja 1.2.0's behavior as far as I can tell.

Fixes issue #551.

11 years agoadjust the wording in "multiple rules generate X" warning
Evan Martin [Fri, 19 Apr 2013 21:38:23 +0000 (14:38 -0700)]
adjust the wording in "multiple rules generate X" warning

Ideally we'd detect this at build time and only warn if your build
was affected, but that's hard to do.  (Really we should just abort
when this scenario is detected rather than continuing, but now
users are relying on it.)

Hopefully improves issue #543.

11 years agomake DiskInterfaceTest.StatBadPath quiet
Evan Martin [Fri, 19 Apr 2013 21:32:29 +0000 (14:32 -0700)]
make DiskInterfaceTest.StatBadPath quiet

Add a flag to temporarily suppress error output.

Fixes issue #281.

11 years agodrop std:: qualifiers on more stl datatypes
Evan Martin [Thu, 18 Apr 2013 18:27:20 +0000 (11:27 -0700)]
drop std:: qualifiers on more stl datatypes

We "using namespace std" anywhere we need a std::string or a std::vector.

11 years agoMerge pull request #548 from sgraham/laconic-with-repeat
Nico Weber [Thu, 18 Apr 2013 04:20:19 +0000 (21:20 -0700)]
Merge pull request #548 from sgraham/laconic-with-repeat

reset count in LaconicPrinter for when using --gtest_repeat

11 years agoalso print iteration for gtest_repeat=-1
Scott Graham [Thu, 18 Apr 2013 00:30:06 +0000 (17:30 -0700)]
also print iteration for gtest_repeat=-1

11 years agofix --profile=pprof on newer ubuntus
Evan Martin [Mon, 15 Apr 2013 21:41:10 +0000 (14:41 -0700)]
fix --profile=pprof on newer ubuntus

The --as-needed default for ld would drop -lprofiler.

11 years agoreset count in LaconicPrinter for when using --gtest_repeat
Scott Graham [Wed, 17 Apr 2013 20:05:50 +0000 (13:05 -0700)]
reset count in LaconicPrinter for when using --gtest_repeat

11 years agoMerge pull request #547 from nico/vim_cpo
Nico Weber [Tue, 16 Apr 2013 19:20:49 +0000 (12:20 -0700)]
Merge pull request #547 from nico/vim_cpo

Note that version 1.3 of the vim syntax file has been sent upstream.

11 years agoNote that version 1.3 of the vim syntax file has been sent upstream.
Nico Weber [Tue, 16 Apr 2013 19:18:39 +0000 (12:18 -0700)]
Note that version 1.3 of the vim syntax file has been sent upstream.

Also fix an issue noticed by Bram during integration: Make sure to set "cpo"
correctly. This matches other vim syntax files and is required to make sure
that the backslash continuation works with all .vimrc files
(see `:help line-continuation`).

11 years agoMerge pull request #541 from sgraham/fix-vs2012
Evan Martin [Sat, 13 Apr 2013 19:37:16 +0000 (12:37 -0700)]
Merge pull request #541 from sgraham/fix-vs2012

try again on vs2012 build fixes

11 years agobootstrap: make --windows usable for cl.exe-on-wine
Evan Martin [Thu, 11 Apr 2013 21:05:13 +0000 (14:05 -0700)]
bootstrap: make --windows usable for cl.exe-on-wine

11 years agoadd HACKING note on cl.exe-via-wine
Evan Martin [Thu, 11 Apr 2013 20:50:36 +0000 (13:50 -0700)]
add HACKING note on cl.exe-via-wine

11 years agowindows: always extract dependencies, even on compile failure
Evan Martin [Thu, 11 Apr 2013 19:46:34 +0000 (12:46 -0700)]
windows: always extract dependencies, even on compile failure

We always want to filter /showIncludes output.

11 years agofix test
Evan Martin [Thu, 11 Apr 2013 18:28:28 +0000 (11:28 -0700)]
fix test

I intentionally changed the semantics of the code that affected this
test, but didn't update the test.  Oops.

11 years agowindows build fix
Evan Martin [Thu, 11 Apr 2013 17:03:48 +0000 (10:03 -0700)]
windows build fix

How did this ever work?

11 years agotry again on vs2012 build fixes
Scott Graham [Thu, 11 Apr 2013 03:16:24 +0000 (20:16 -0700)]
try again on vs2012 build fixes

11 years agobuild ninja itself in deps mode
Evan Martin [Wed, 10 Apr 2013 19:51:04 +0000 (12:51 -0700)]
build ninja itself in deps mode

Hopefully will help flush out bugs.

11 years agoalways use output timestamp for deps
Evan Martin [Wed, 10 Apr 2013 19:49:23 +0000 (12:49 -0700)]
always use output timestamp for deps

Using the timestamp of the .d file was wrong.  It can be written
at a different time than the output, and it is immediately deleted
after parsing; only the output remains for subsequent timestamp
comparison purposes, so always use the output's timestamp.  (This
is how the code worked on Windows already.)

11 years agoinclude mtimes in deplog explain
Evan Martin [Wed, 10 Apr 2013 19:10:10 +0000 (12:10 -0700)]
include mtimes in deplog explain

11 years agoslightly smaller subsections
Evan Martin [Wed, 10 Apr 2013 17:08:51 +0000 (10:08 -0700)]
slightly smaller subsections

11 years agomove pool docs down under "more details"
Evan Martin [Wed, 10 Apr 2013 17:03:55 +0000 (10:03 -0700)]
move pool docs down under "more details"

The earlier section is a tutorial-style overview.  The latter section
is for side features like phony rules and header dependencies.  Pools
fit in with the latter.

11 years agomore minor manual touchups
Evan Martin [Wed, 10 Apr 2013 17:02:27 +0000 (10:02 -0700)]
more minor manual touchups

11 years agomake unittest match module name
Evan Martin [Wed, 10 Apr 2013 16:43:25 +0000 (09:43 -0700)]
make unittest match module name

11 years agomention mtimes
Evan Martin [Wed, 10 Apr 2013 16:43:07 +0000 (09:43 -0700)]
mention mtimes

11 years agomonospace flags
Evan Martin [Wed, 10 Apr 2013 16:34:34 +0000 (09:34 -0700)]
monospace flags

11 years agomore windows doc updates
Evan Martin [Wed, 10 Apr 2013 16:31:23 +0000 (09:31 -0700)]
more windows doc updates

11 years agoupdate some windows references in the docs
Evan Martin [Tue, 9 Apr 2013 17:15:18 +0000 (10:15 -0700)]
update some windows references in the docs

Drop the timing info, it's too variable to commit to a manual.

11 years agobigger manual
Evan Martin [Tue, 9 Apr 2013 17:07:17 +0000 (10:07 -0700)]
bigger manual

11 years agoMerge pull request #539 from nico/typo
Nico Weber [Tue, 9 Apr 2013 19:45:46 +0000 (12:45 -0700)]
Merge pull request #539 from nico/typo

fix typo in comment

11 years agofix typo in comment
Nico Weber [Tue, 9 Apr 2013 19:44:31 +0000 (12:44 -0700)]
fix typo in comment

11 years agoMerge pull request #537 from yannicklm/master
Evan Martin [Tue, 9 Apr 2013 19:39:09 +0000 (12:39 -0700)]
Merge pull request #537 from yannicklm/master

    ninja.vim: add deps as keyword

11 years agoninja.vim: add deps as keyword
yannicklm [Tue, 9 Apr 2013 18:13:48 +0000 (20:13 +0200)]
ninja.vim: add deps as keyword

new in ninja 1.3

11 years agoMerge pull request #535 from sgraham/fix-windows-depslog
Evan Martin [Tue, 9 Apr 2013 18:02:53 +0000 (11:02 -0700)]
Merge pull request #535 from sgraham/fix-windows-depslog

fix windows build after depslog

11 years agofix test
Scott Graham [Tue, 9 Apr 2013 17:44:54 +0000 (10:44 -0700)]
fix test

11 years agoMerge pull request #534 from nico/invalid_log
Evan Martin [Tue, 9 Apr 2013 17:35:24 +0000 (10:35 -0700)]
Merge pull request #534 from nico/invalid_log

On invalid depslog header, restart build instead of showing an error.

11 years agoadd ifdef around console_
Scott Graham [Tue, 9 Apr 2013 17:05:23 +0000 (10:05 -0700)]
add ifdef around console_

11 years agofix windows build after depslog
Scott Graham [Tue, 9 Apr 2013 17:03:46 +0000 (10:03 -0700)]
fix windows build after depslog

11 years agoOn invalid depslog header, restart build instead of showing an error.
Nico Weber [Tue, 9 Apr 2013 16:58:54 +0000 (09:58 -0700)]
On invalid depslog header, restart build instead of showing an error.

Also add tests for invalid headers.

11 years agodocument deps
Evan Martin [Tue, 9 Apr 2013 16:33:35 +0000 (09:33 -0700)]
document deps

11 years agoMerge branch 'dep-pipeless'
Evan Martin [Tue, 9 Apr 2013 15:57:02 +0000 (08:57 -0700)]
Merge branch 'dep-pipeless'

This merges a new mechanism for tracking "depfile" dependencies that
is faster on all platforms but dramatically so on Windows.

11 years agoMerge pull request #532 from nico/sshhh_gtest
Evan Martin [Tue, 9 Apr 2013 04:57:45 +0000 (21:57 -0700)]
Merge pull request #532 from nico/sshhh_gtest

Make test runner less chatty.

11 years agoTry to fix build on Linux more.
Nico Weber [Tue, 9 Apr 2013 04:53:25 +0000 (21:53 -0700)]
Try to fix build on Linux more.

11 years agoTry to fix build on Linux.
Nico Weber [Tue, 9 Apr 2013 04:48:51 +0000 (21:48 -0700)]
Try to fix build on Linux.

11 years agoadd a straightforward deps log test, fix the other one
Evan Martin [Tue, 9 Apr 2013 04:43:49 +0000 (21:43 -0700)]
add a straightforward deps log test, fix the other one

The first test I wrote was wrong; write a simpler test that exercises
the "no failures" code paths, then fix the second test and the bugs it
exposed.

11 years agoMake ninja_test output nicer.
Nico Weber [Tue, 9 Apr 2013 04:41:50 +0000 (21:41 -0700)]
Make ninja_test output nicer.

11 years agoUse LinePrinter in test runner.
Nico Weber [Tue, 9 Apr 2013 04:28:12 +0000 (21:28 -0700)]
Use LinePrinter in test runner.

11 years agoMake LinePrinter members private, add comments.
Nico Weber [Tue, 9 Apr 2013 04:19:27 +0000 (21:19 -0700)]
Make LinePrinter members private, add comments.

11 years agomove printing of new lines into LinePrinter.
Nico Weber [Tue, 9 Apr 2013 04:14:29 +0000 (21:14 -0700)]
move printing of new lines into LinePrinter.

11 years agoWrite the depslog version in binary instead of text.
Nico Weber [Tue, 9 Apr 2013 00:18:30 +0000 (17:18 -0700)]
Write the depslog version in binary instead of text.

This way, it doubles as a byte-order marker. The header is now exactly one
line in a hex editor, and it's still relatively easy to look at the version
in a text editor.

11 years agorefactor; no intended functionality change
Nico Weber [Tue, 9 Apr 2013 04:11:34 +0000 (21:11 -0700)]
refactor; no intended functionality change

11 years agomove single-line printing to new class
Nico Weber [Tue, 9 Apr 2013 03:54:28 +0000 (20:54 -0700)]
move single-line printing to new class

11 years agoMake deps=gcc without depfile an error.
Nico Weber [Tue, 9 Apr 2013 04:08:19 +0000 (21:08 -0700)]
Make deps=gcc without depfile an error.

When I first played with depslog, I accidentally removed the depfile attribute
on my cc edges, which had the effect of ninja silently ignoring all depfiles.
Instead, let ninja complain about edges that have deps set to gcc and depfile
set to nothing.

This is done at edge build time, instead of at mainfest load time, because
adding this check to ParseEdge() regressed empty build time for target 'chrome'
by 30ms. The check is only useful for generator authors, regular users should
never see this.

11 years agoadd a test for the "deps out of date" case
Evan Martin [Mon, 8 Apr 2013 17:20:58 +0000 (10:20 -0700)]
add a test for the "deps out of date" case

It touched the various remaining XXXes in the code, hooray.

11 years agoMake gtest output more silent, ninja issue #528.
Nico Weber [Tue, 9 Apr 2013 01:02:05 +0000 (18:02 -0700)]
Make gtest output more silent, ninja issue #528.

This is just a proof-of-concept. The terminal printing logic should be
extracted from src/build.cc and then reused here.

11 years agoadd a test verifying build failure on bad deps
Evan Martin [Fri, 5 Apr 2013 21:08:41 +0000 (14:08 -0700)]
add a test verifying build failure on bad deps

11 years agomake it an error for now to have multiple outputs with depslog
Evan Martin [Fri, 5 Apr 2013 17:27:22 +0000 (10:27 -0700)]
make it an error for now to have multiple outputs with depslog

11 years agoonly write deps log if edge is in deps log mode
Evan Martin [Tue, 19 Feb 2013 18:00:05 +0000 (10:00 -0800)]
only write deps log if edge is in deps log mode

11 years agouse logged deps mtime in dirty calculation
Evan Martin [Sun, 17 Feb 2013 23:13:35 +0000 (15:13 -0800)]
use logged deps mtime in dirty calculation

The idea here is that it's possible for a build to complete (writing
its output) but then for Ninja to get interrupted before writing out
the updated dependency information.  In that case the mtime stored in
the deps log (if any) will match the previous output, and we'll know
we need to rebuild the output just to get the deps updated.

11 years agodon't call .front() on an empty vector
Evan Martin [Sun, 17 Feb 2013 21:19:53 +0000 (13:19 -0800)]
don't call .front() on an empty vector

Fixes a gcc debug-mode assertion.

11 years agohook up depslog writing into build process
Evan Martin [Sun, 17 Feb 2013 20:53:40 +0000 (12:53 -0800)]
hook up depslog writing into build process

11 years agorename "special" to "deps"
Evan Martin [Sun, 17 Feb 2013 20:09:54 +0000 (12:09 -0800)]
rename "special" to "deps"

11 years agomissing header
Evan Martin [Sat, 16 Feb 2013 22:34:49 +0000 (14:34 -0800)]
missing header

11 years agoremove depfiles files as they're parsed
Evan Martin [Tue, 15 Jan 2013 02:00:52 +0000 (18:00 -0800)]
remove depfiles files as they're parsed

11 years agodepslog: track dead record count
Evan Martin [Wed, 9 Jan 2013 01:52:44 +0000 (17:52 -0800)]
depslog: track dead record count

11 years agoadd recompaction to depslog
Evan Martin [Tue, 8 Jan 2013 16:43:39 +0000 (08:43 -0800)]
add recompaction to depslog

Not done automatically yet, just an implementation and a test.

11 years agoclarify depslog overview
Evan Martin [Tue, 8 Jan 2013 16:20:56 +0000 (08:20 -0800)]
clarify depslog overview

11 years agomake old deps format migration actually work
Evan Martin [Tue, 8 Jan 2013 16:17:12 +0000 (08:17 -0800)]
make old deps format migration actually work

11 years agorecord and check depslog file version
Evan Martin [Mon, 7 Jan 2013 19:04:24 +0000 (11:04 -0800)]
record and check depslog file version

Future-proofing against some change we may need to make later.

11 years agodon't write out deps entries if nothing changed
Evan Martin [Mon, 7 Jan 2013 18:59:27 +0000 (10:59 -0800)]
don't write out deps entries if nothing changed

Shortcuts a common case.

11 years agowindows: drop use of msvc helper in build
Evan Martin [Sun, 6 Jan 2013 23:33:46 +0000 (15:33 -0800)]
windows: drop use of msvc helper in build

11 years agouse special=anything to trigger loading from depslog
Evan Martin [Sun, 6 Jan 2013 23:26:20 +0000 (15:26 -0800)]
use special=anything to trigger loading from depslog

11 years agowindows: use CLParser to extract deps during build
Evan Martin [Sun, 6 Jan 2013 23:21:07 +0000 (15:21 -0800)]
windows: use CLParser to extract deps during build

11 years agorefactor build-time deps-extraction
Evan Martin [Sun, 6 Jan 2013 19:42:40 +0000 (11:42 -0800)]
refactor build-time deps-extraction

11 years agofactor MSVC parsing out of CLWrapper into CLParser
Evan Martin [Sun, 6 Jan 2013 19:36:17 +0000 (11:36 -0800)]
factor MSVC parsing out of CLWrapper into CLParser

11 years agowindows: add uint16 casts in depslog
Evan Martin [Sat, 5 Jan 2013 18:49:48 +0000 (10:49 -0800)]
windows: add uint16 casts in depslog

11 years agoadd "special=gcc" attribute, use to load depslog
Evan Martin [Fri, 4 Jan 2013 17:39:01 +0000 (09:39 -0800)]
add "special=gcc" attribute, use to load depslog

11 years agotrack deps log load time in metrics
Evan Martin [Sun, 30 Dec 2012 20:50:43 +0000 (12:50 -0800)]
track deps log load time in metrics

11 years agoexpand DepsLog test, fix two bugs it revealed
Evan Martin [Sun, 30 Dec 2012 20:48:44 +0000 (12:48 -0800)]
expand DepsLog test, fix two bugs it revealed

11 years agoplumb DepsLog load through Builder
Evan Martin [Sun, 30 Dec 2012 18:15:49 +0000 (10:15 -0800)]
plumb DepsLog load through Builder

11 years agono error if deps log doesn't exist
Evan Martin [Sun, 30 Dec 2012 18:11:13 +0000 (10:11 -0800)]
no error if deps log doesn't exist

11 years agoload deps log at startup
Evan Martin [Sun, 30 Dec 2012 18:10:03 +0000 (10:10 -0800)]
load deps log at startup

11 years agofactor out creation of build directory
Evan Martin [Sun, 30 Dec 2012 17:53:57 +0000 (09:53 -0800)]
factor out creation of build directory

11 years agouse DepsLog in loading dependencies
Evan Martin [Sat, 29 Dec 2012 19:55:52 +0000 (11:55 -0800)]
use DepsLog in loading dependencies

WIP

11 years agofactor out implicit dep loading
Evan Martin [Sat, 29 Dec 2012 19:24:03 +0000 (11:24 -0800)]
factor out implicit dep loading

11 years agopass command results via a struct
Evan Martin [Sat, 29 Dec 2012 18:53:59 +0000 (10:53 -0800)]
pass command results via a struct

WaitForCommand now passes all command output via a struct.
This will allow adding more output in a future change.

11 years agoadd DepsLog, a new data structure for dependency information
Evan Martin [Mon, 17 Dec 2012 17:08:15 +0000 (09:08 -0800)]
add DepsLog, a new data structure for dependency information

DepsLog is a compact serialization of dependency information.
It can be used to replace depfiles for faster loading.

11 years agosync version number in master with release
Evan Martin [Mon, 8 Apr 2013 21:44:30 +0000 (14:44 -0700)]
sync version number in master with release

Update RELEASING with notes on how to not screw this up again.

11 years agosplit FakeCommandRunner out of BuildTest
Evan Martin [Mon, 8 Apr 2013 21:15:36 +0000 (14:15 -0700)]
split FakeCommandRunner out of BuildTest

Separating concerns to make a subsequent refactoring easier.

11 years agomove test virtual time "now_" into VirtualFileSystem
Evan Martin [Mon, 8 Apr 2013 21:05:27 +0000 (14:05 -0700)]
move test virtual time "now_" into VirtualFileSystem

It's the only piece that cares about the current time.

11 years agoMerge pull request #527 from wang-bin/compile_fix
Evan Martin [Mon, 8 Apr 2013 21:17:27 +0000 (14:17 -0700)]
Merge pull request #527 from wang-bin/compile_fix

fix compile error about noreturn on windows

11 years agofix compile error about noreturn on windows
wang-bin [Sun, 7 Apr 2013 03:14:16 +0000 (11:14 +0800)]
fix compile error about noreturn on windows