| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
|
| |
|
|\
| |
| | |
Don't use va_start() with reference parameters, it's undefined behavior.
|
| |
| |
| |
| | |
Should fix issue #584.
|
|\ \
| | |
| | | |
do not unconditionally use PATH_MAX with getcwd
|
| |/
| |
| |
| |
| |
| | |
Instead, grow a buffer until getcwd either succeeds or fails with an errno different than ERANGE
(meaning the passed buffer was too short to represent the actual path).
Reset errno before calling getcwd to check it eventually did not fail.
|
|/ |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
I just used `ninja -d stat` and it took me a bit to realize that I missed the
trailing 's'.
While here, move the message printing from printf() to Error(). This makes the
output consistent with other error outputs: The messages are now prefixed with
"ninja: error: " instead of just "ninja: " and they go to stderr instead of
stdout.
|
|\
| |
| | |
Add compdb tool
|
| |
| |
| |
| |
| | |
This tool helps convert Ninja build files to a compilation database
of the form used by Clang tooling.
|
|/
|
| |
./bootstrap.py fails on some platforms without this include
|
|
|
|
| |
This is a FAQ.
|
| |
|
|
|
|
|
| |
I had already broken this earlier, but the refactor of variable handling
broke it completely.
|
|
|
|
|
|
|
|
|
|
|
| |
The return type of CollectTarget() is Node, so we should return NULL in the
failure case instead of false.
src/ninja.cc:188:16: warning: initialization of pointer of type 'Node *' to null from a constant boolean expression [-Wbool-conversion]
return false;
^~~~~
Signed-off-by: Thiago Farina <tfarina@chromium.org>
|
|\
| |
| | |
refactoring: decompose CollectTargetsFromArgs
|
| | |
|
|\ \
| | |
| | | |
Allocate disk_interface near where it's needed.
|
| | |
| | |
| | |
| | |
| | |
| | | |
This avoids allocating disk_interface unnecessarily.
Signed-off-by: Thiago Farina <tfarina@chromium.org>
|
|\ \ \
| |_|/
|/| | |
Minor improvements to ninja : Different exit code if user presses Ctrl-C, add eclipse to .gitignore, Improve an error message
|
| | |
| | |
| | |
| | | |
Change-Id: I7be958e18eb2e434e78afb6e03b332281a651957
|
| |/
|/|
| |
| | |
Signed-off-by: Thiago Farina <tfarina@chromium.org>
|
| |
| |
| |
| |
| |
| |
| | |
getopt_long(3) was reporting "ninja: invalid option -- h" when "ninja -h" was
called.
Regression most probably introduced by 5fdb12ed5cec4e1c853c64026142d088ff5519e1
|
| |
| |
| |
| |
| |
| | |
Note: _WIN32 is used instead of WIN32 to enable builds with MSVC
IDE, Windows SDK non-IDE command line tools, and mingw/mingw-w64
based toolchains
|
|/
|
|
|
|
| |
Remove now-unimplemented 'V' from getopt_long. Remove 'h', since it's
included in the long options. Order switch cases in the same order as in
the getopt_long argument.
|
| |
|
|
|
|
|
|
| |
Now "ninja -t msvc ..." passes the "..." to the msvc helper main.
This drastically simplifies bootstrap and makes ninja a single
binary again.
|
|
|
|
|
| |
Calling bucket_count() works locally with MSVC. I wonder if some
other change in the code fixed this.
|
|
|
|
|
| |
This will be necessary to inline msvc-helper as well as so -t graph
can get depfiles.
|
| |
|
| |
|
|
|
|
| |
Now "ninja -t list" works from any directory.
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
The build log is needed in computing whether an edge is
dirty, so I think it belongs here. (It's a bit weird
that Builder needs to reach into it to record completed
commands, maybe it will become cleaner with more thought.)
|
|
|
|
| |
Reducing use of globals.
|
| |
|
|
|
|
|
| |
(Committing this on top of b56fe80 since they're related,
but I may end up reverting both.)
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Fixes issue #392 (didn't handle creating nested build dirs right).
Moves MakeDir out of util.h; all code should go through
DiskInterface to simplify testing. Moves ownership of the
DiskInterface into the client of the Builder, which also allows
removing some code that reached inside the object as well as
a minor leak.
|
| |
|
| |
|
|
|
|
| |
Filed issue #386 about it too.
|
| |
|