Skip to content

Commit 5577dbd

Browse files
mi-acCommit bot
authored andcommitted
Remove version generation approach.
Revert "Partially reland Auto-generate v8 version based on tags." This reverts commit 0707afc. Revert "Ensure tags are fetched when generating the V8 version." This reverts commit ea6831e. Revert "Restrict tag-update for version generation to cached git repos." This reverts commit c6641e1. BUG=chromium:446166 LOG=n Review URL: https://codereview.chromium.org/866263005 Cr-Commit-Position: refs/heads/master@{#26244}
1 parent 8a67083 commit 5577dbd

4 files changed

Lines changed: 7 additions & 170 deletions

File tree

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ shell_g
4646
/out
4747
/perf.data
4848
/perf.data.old
49-
/src/version_gen.cc
5049
/test/benchmarks/CHECKED_OUT_*
5150
/test/benchmarks/downloaded_*
5251
/test/benchmarks/kraken

DEPS

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -87,11 +87,6 @@ hooks = [
8787
'pattern': '.',
8888
'action': ['python', 'v8/tools/clang/scripts/update.py', '--if-needed'],
8989
},
90-
{
91-
# Generate v8 version based on the last git tag.
92-
"pattern": ".",
93-
"action": ["python", "v8/build/generate_version.py"],
94-
},
9590
{
9691
# A change to a .gyp, .gypi, or to GYP itself should run the generator.
9792
"pattern": ".",

Makefile

Lines changed: 7 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -266,7 +266,7 @@ NACL_CHECKS = $(addsuffix .check,$(NACL_BUILDS))
266266
# File where previously used GYPFLAGS are stored.
267267
ENVFILE = $(OUTDIR)/environment
268268

269-
.PHONY: all check clean builddeps dependencies $(ENVFILE).new native version \
269+
.PHONY: all check clean builddeps dependencies $(ENVFILE).new native \
270270
qc quickcheck $(QUICKCHECKS) turbocheck \
271271
$(addsuffix .quickcheck,$(MODES)) $(addsuffix .quickcheck,$(ARCHES)) \
272272
$(ARCHES) $(MODES) $(BUILDS) $(CHECKS) $(addsuffix .clean,$(ARCHES)) \
@@ -279,13 +279,9 @@ ENVFILE = $(OUTDIR)/environment
279279
# Target definitions. "all" is the default.
280280
all: $(DEFAULT_MODES)
281281

282-
# Target for generating the v8 version file from git tags.
283-
version:
284-
build/generate_version.py
285-
286282
# Special target for the buildbots to use. Depends on $(OUTDIR)/Makefile
287283
# having been created before.
288-
buildbot: version
284+
buildbot:
289285
$(MAKE) -C "$(OUTDIR)" BUILDTYPE=$(BUILDTYPE) \
290286
builddir="$(abspath $(OUTDIR))/$(BUILDTYPE)"
291287

@@ -296,23 +292,22 @@ $(MODES): $(addsuffix .$$@,$(DEFAULT_ARCHES))
296292
$(ARCHES): $(addprefix $$@.,$(DEFAULT_MODES))
297293

298294
# Defines how to build a particular target (e.g. ia32.release).
299-
$(BUILDS): $(OUTDIR)/Makefile.$$@ version
295+
$(BUILDS): $(OUTDIR)/Makefile.$$@
300296
@$(MAKE) -C "$(OUTDIR)" -f Makefile.$@ \
301297
BUILDTYPE=$(shell echo $(subst .,,$(suffix $@)) | \
302298
python -c "print \
303299
raw_input().replace('opt', '').capitalize()") \
304300
builddir="$(shell pwd)/$(OUTDIR)/$@"
305301

306-
native: $(OUTDIR)/Makefile.native version
302+
native: $(OUTDIR)/Makefile.native
307303
@$(MAKE) -C "$(OUTDIR)" -f Makefile.native \
308304
BUILDTYPE=Release \
309305
builddir="$(shell pwd)/$(OUTDIR)/$@"
310306

311307
$(ANDROID_ARCHES): $(addprefix $$@.,$(MODES))
312308

313309
$(ANDROID_BUILDS): $(GYPFILES) $(ENVFILE) build/android.gypi \
314-
must-set-ANDROID_NDK_ROOT_OR_TOOLCHAIN Makefile.android \
315-
version
310+
must-set-ANDROID_NDK_ROOT_OR_TOOLCHAIN Makefile.android
316311
@$(MAKE) -f Makefile.android $@ \
317312
ARCH="$(basename $@)" \
318313
MODE="$(subst .,,$(suffix $@))" \
@@ -322,7 +317,7 @@ $(ANDROID_BUILDS): $(GYPFILES) $(ENVFILE) build/android.gypi \
322317
$(NACL_ARCHES): $(addprefix $$@.,$(MODES))
323318

324319
$(NACL_BUILDS): $(GYPFILES) $(ENVFILE) \
325-
Makefile.nacl must-set-NACL_SDK_ROOT version
320+
Makefile.nacl must-set-NACL_SDK_ROOT
326321
@$(MAKE) -f Makefile.nacl $@ \
327322
ARCH="$(basename $@)" \
328323
MODE="$(subst .,,$(suffix $@))" \
@@ -422,10 +417,7 @@ native.clean:
422417
rm -rf $(OUTDIR)/native
423418
find $(OUTDIR) -regex '.*\(host\|target\)\.native\.mk' -delete
424419

425-
version.clean:
426-
rm -f src/version_gen.cc
427-
428-
clean: $(addsuffix .clean, $(ARCHES) $(ANDROID_ARCHES) $(NACL_ARCHES)) native.clean gtags.clean version.clean
420+
clean: $(addsuffix .clean, $(ARCHES) $(ANDROID_ARCHES) $(NACL_ARCHES)) native.clean gtags.clean
429421

430422
# GYP file generation targets.
431423
OUT_MAKEFILES = $(addprefix $(OUTDIR)/Makefile.,$(BUILDS))

build/generate_version.py

Lines changed: 0 additions & 149 deletions
This file was deleted.

0 commit comments

Comments
 (0)