github/textmate
1
1
Fork 0
mirror of https://github.com/textmate/textmate.git synced 2026-01-22 13:17:55 -05:00
Code Issues Packages Projects Releases Wiki Activity

Remove ‘after_bump’ phony rule

Browse Source 
Turns out a phony rule in ninja isn’t exactly the same as an alias. When the file the phony rule is supposed to be an alias of already exist when calling ‘ninja’ then rules depending on the alias are not considered out-of-date even if the aliased file is of newer date.

Build rules that previously depended on ‘after_bump’ should instead depend on ‘builddir/revision.$APP_REVISION’.
This commit is contained in:
Allan Odgaard
2013-02-16 10:31:54 +01:00
parent c7763d5bcc
commit eb613bf563
1 changed files with 0 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
bin/gen_build
View File

@@ -854,7 +854,6 @@ rule bump_revision
build $builddir/revision.next: bump_revision $builddir/build.ninja
build bump: phony $builddir/revision.next
build $builddir/revision.$APP_REVISION: touch
build after_bump: phony $builddir/revision.$APP_REVISION
rule clean
command = rm -r '$path'