Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Commit

Permalink
build: Regenerate docs for tarball and releases
Browse files Browse the repository at this point in the history 
Related: https://twitter.com/kapeli/status/222477400880070658
  • Loading branch information
@isaacs
isaacs committed Jul 10, 2012
1 parent 71078f9 commit fbfb2f5
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions Makefile
View file
Expand Up @@ -130,7 +130,7 @@ website_files = \
out/doc/changelog.html \
$(doc_images)

doc: program $(apidoc_dirs) $(website_files) $(apiassets) $(apidocs) tools/doc/ blog
doc: $(apidoc_dirs) $(website_files) $(apiassets) $(apidocs) tools/doc/ blog node

blogclean:
rm -rf out/blog
Expand All @@ -144,19 +144,19 @@ $(apidoc_dirs):
out/doc/api/assets/%: doc/api_assets/% out/doc/api/assets/
cp $< $@

out/doc/changelog.html: ChangeLog doc/changelog-head.html doc/changelog-foot.html tools/build-changelog.sh
out/doc/changelog.html: ChangeLog doc/changelog-head.html doc/changelog-foot.html tools/build-changelog.sh node

This comment has been minimized.

Sign in to view

Copy link
@bnoordhuis

bnoordhuis Jul 10, 2012

Member

Why does this depend on node?
bash tools/build-changelog.sh

out/doc/%.html: doc/%.html
out/doc/%.html: doc/%.html node

This comment has been minimized.

Sign in to view

Copy link
@bnoordhuis

bnoordhuis Jul 10, 2012

Member

And this?

This comment has been minimized.

Sign in to view

Copy link
@isaacs

isaacs Jul 10, 2012

Author

Because when the version number changes, it needs to re-generate.

I guess the smarter thing might be to have it depend on src/node_version.cc?

This comment has been minimized.

Sign in to view

Copy link
@bnoordhuis

bnoordhuis Jul 10, 2012

Member

If you mean src/node_version.h, then I'd say yes. :-) Depending on the node binary forces a rebuild of the docs after every recompile.
cat $< | sed -e 's|__VERSION__|'$(VERSION)'|g' > $@

out/doc/%: doc/%
cp -r $< $@

out/doc/api/%.json: doc/api/%.markdown
out/doc/api/%.json: doc/api/%.markdown node
out/Release/node tools/doc/generate.js --format=json $< > $@

out/doc/api/%.html: doc/api/%.markdown
out/doc/api/%.html: doc/api/%.markdown node
out/Release/node tools/doc/generate.js --format=html --template=doc/template.html $< > $@

email.md: ChangeLog tools/email-footer.md
Expand Down Expand Up @@ -216,7 +216,7 @@ $(PKG):
--doc tools/osx-pkg.pmdoc \
--out $(PKG)

$(TARBALL): node out/doc
$(TARBALL): node doc
@if [ "$(shell git status --porcelain | egrep -v '^\?\? ')" = "" ]; then \
exit 0 ; \
else \
Expand Down Expand Up @@ -269,4 +269,4 @@ cpplint:

lint: jslint cpplint

.PHONY: lint cpplint jslint bench clean docopen docclean doc dist distclean check uninstall install install-includes install-bin all program staticlib dynamiclib test test-all website-upload pkg blog blogclean
.PHONY: lint cpplint jslint bench clean docopen docclean doc dist distclean check uninstall install install-includes install-bin all staticlib dynamiclib test test-all website-upload pkg blog blogclean

1 comment on commit fbfb2f5

@bnoordhuis
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haven't tested it (on my ipad) but it looks okay to me.

Please sign in to comment.