summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* No need to convert log_path to c-string. Two places.Thiago Farina2012-05-061-4/+3
| | | | Signed-off-by: Thiago Farina <tfarina@chromium.org>
* First check for string end, then dereference. (5ms more expensive, heh.)Nico Weber2012-05-041-2/+2
|
* Don't walk path components twice. Speeds up CanonicalizePath() 115ms (285ms ↵Nico Weber2012-05-041-6/+2
| | | | -> 170ms).
* Reorder a few lines, no functionality (or perf) change.Nico Weber2012-05-041-5/+4
|
* Skip single '/' characters earlier in the loop. 300ms -> 285ms.Nico Weber2012-05-041-8/+11
|
* Speed up CanonicalizePath() 6.8% (322ms -> 300ms for chrome empty build).Nico Weber2012-05-042-8/+35
|
* Merge pull request #290 from nico/metricsEvan Martin2012-05-021-0/+2
|\ | | | | Add .ninja_log load time to metrics.
| * Add .ninja_log load time to metrics.Nico Weber2012-05-011-0/+2
| | | | | | | | On my system, it takes 22% of the empty build time for chrome.
* | Merge pull request #284 from nico/fixwinbuildEvan Martin2012-05-022-1/+4
|\ \ | | | | | | Fix Windows build.
| * | Fix Windows build.Nico Weber2012-04-302-1/+4
| |/
* | Merge pull request #283 from tfarina/string-ampersandEvan Martin2012-05-026-7/+7
|\ \ | |/ |/| Put & operator with the type rather than the variable name.
| * Put & operator with the type rather than the variable name.Thiago Farina2012-04-286-7/+7
| | | | | | | | Signed-off-by: Thiago Farina <tfarina@chromium.org>
* | Merge pull request #274 from polrop/max_load_averageEvan Martin2012-04-285-4/+51
|\ \ | | | | | | Max load average
| * | Add -l N option to limit the load average.Nicolas Despres2012-04-195-4/+51
| | | | | | | | | | | | | | | | | | | | | This is similar to GNU make -l/--load-average option. It limits the number of job started if the load average exceed the given value. It can be very useful when running ninja on a continuous integration server where we want to use parallelism as much as possible without overloading the server.
| * | Remove trailing white spaces.Nicolas Despres2012-04-191-6/+6
| | |
* | | suggest ninja -h when someone runs 'ninja help'Evan Martin2012-04-261-0/+2
| |/ |/|
* | allow UTF-8 in rule descriptionsEvan Martin2012-04-263-5/+12
| | | | | | | | | | The lexer already mostly allowed this, except that chars >127 were being interpreted as negative indexes into the lexer table.
* | reduce custom ninja status patch in minor waysEvan Martin2012-04-263-128/+53
| | | | | | | | | | Elsewhere in the code I avoid sstream and manual buffer management, so switch this code to behave similarly. Sorry for being OCD.
* | Merge branch 'custom_ninja_status' of git://github.com/polrop/ninjaEvan Martin2012-04-263-51/+199
|\ \
| * | Refactor and test progress status formatting.Nicolas Despres2012-04-243-69/+140
| | |
| * | Introduce NINJA_STATUS env var for customization.Nicolas Despres2012-04-241-2/+79
| | | | | | | | | | | | | | | | | | Some people may want to display different relevant information about the progress of the build. Also it can be usefull to debug jobs allocation.
| * | Remove trailing white spaces.Nicolas Despres2012-04-242-13/+13
| |/
* | Merge pull request #272 from nico/statstweakEvan Martin2012-04-261-3/+3
|\ \ | | | | | | Switch the order of total and avg columns in -d stats output.
| * | Switch the order of total and avg columns in -d stats output.Nico Weber2012-04-171-3/+3
| |/
* | Merge pull request #269 from sgraham/explainEvan Martin2012-04-266-10/+72
|\ \ | | | | | | add support for -d explain to help debug why rules are running
| * | add support for -d explain to help debug why rules are runningScott Graham2012-04-136-10/+72
| |/
* | Merge pull request #228 from pcc/lazily-create-commandrunnerEvan Martin2012-04-261-4/+8
|\ \ | | | | | | Lazily create the CommandRunner in Builder::Build
| * | Lazily create the CommandRunner in Builder::BuildPeter Collingbourne2012-02-271-4/+8
| | | | | | | | | | | | | | | This allows the user to interrupt the build (i.e. using Ctrl-C) while the build plan is being computed.
* | | Merge branch 'dont-clean-phony' of git://github.com/pcc/ninjaEvan Martin2012-04-262-9/+36
|\ \ \ | |_|/ |/| | | | | | | | Conflicts: src/clean.cc
| * | Don't delete phony targets when cleaning a specified targetPeter Collingbourne2012-02-282-7/+34
| |/ | | | | | | Also, modify Cleaner::CleanAll to use Edge::is_phony.
* | Merge pull request #252 from qhuo/remove-depfile-when-cleaningEvan Martin2012-04-112-0/+34
|\ \ | | | | | | Remove depfiles when running "ninja -t clean <target>" or "ninja -t clean -r <rule>"
| * | Remove depfiles when running when running "ninja -t clean <target>"Qingning Huo2012-03-142-0/+34
| | | | | | | | | | | | or "ninja -t clean -r <rule>"
* | | Merge pull request #253 from qhuo/include-io.hEvan Martin2012-04-101-0/+1
|\ \ \ | | | | | | | | Fix windows build (add #include <io.h>)
| * | | Fix windows build (add #include <io.h>)Qingning Huo2012-03-151-0/+1
| |/ /
* | | Make |log_| variable private to BuildLog.Evan Martin2012-04-052-4/+5
| | | | | | | | | | | | | | | | | | This fix the TODO in build_log.h file. Signed-off-by: Thiago Farina <tfarina@chromium.org>
* | | Merge pull request #263 from mathstuf/dev/gcc-470-build-fixesEvan Martin2012-04-053-0/+3
|\ \ \ | | | | | | | | Include unistd.h where needed
| * | | Include unistd.h where neededBen Boeckel2012-03-283-0/+3
| | | |
* | | | Merge pull request #256 from iwadon/fix-sign-compareEvan Martin2012-04-053-10/+10
|\ \ \ \ | |/ / / |/| | | fix warning: "comparison between signed and unsigned integer expressions"
| * | | fix warning: "comparison between signed and unsigned integer expressions"Hiroyuki Iwatsuki2012-03-163-10/+10
| |/ /
* | | Depfile parser accept tilde.Nicolas Despres2012-03-143-10/+21
|/ /
* | Merge pull request #250 from qhuo/SetCloseOnExec-for-WindowsEvan Martin2012-03-131-3/+4
|\ \ | | | | | | Add a Windows version of SetCloseOnExec()
| * | Add a Windows version of SetCloseOnExec()Qingning Huo2012-03-131-3/+4
| | |
* | | Merge pull request #249 from qhuo/StatBadPath-Win7Evan Martin2012-03-131-2/+5
|\ \ \ | | | | | | | | Fix StatBadPath for Windows 7
| * | | Fix StatBadPath for Windows 7Qingning Huo2012-03-131-2/+5
| |/ / | | | | | | | | | | | | | | | | | | The StatBadPath test expects both Windows and Linux to reject a path name with 512 characters. However, it seems that such path is actually acceptable to Windows 7. The change constructs a different path name that is invalid on Windows.
* | | Fix resources leaksPierre Schweitzer2012-03-132-1/+3
|/ /
* | Issue #241 - handle depfiles generated by older versions of GCCOleksandr Usov2012-03-123-8/+41
| | | | | | | | | | | | | | | | | | | | Older versions of GCC would produce broken depfiles when -MT or -MQ is used gcc43 -MT foo.o -MMD -MF foo.o.d -o foo.o -c foo.c will result in the following depfile foo.o foo.o: <dependencies> Parse multiple outputs unifying duplicates and correctly report errors if they are different.
* | Merge pull request #235 from fischman/ppollEvan Martin2012-03-083-7/+86
|\ \ | | | | | | pselect->ppoll on linux to raise the process limit roof
| * | addressed pcc commentsAmi Fischman2012-03-042-12/+19
| | |
| * | pselect->ppoll on linux to raise the process limit roofAmi Fischman2012-03-043-7/+79
| | |
* | | Merge pull request #239 from pcc/enotdirEvan Martin2012-03-082-13/+15
|\ \ \ | | | | | | | | Treat paths of the form "existing-file/something" as non-existent