Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to make doc-only #38938

Closed
mcollina opened this issue Jun 5, 2021 · 10 comments
Closed

Unable to make doc-only #38938

mcollina opened this issue Jun 5, 2021 · 10 comments

Comments

@mcollina
Copy link
Member

mcollina commented Jun 5, 2021

  • Version: master
  • Platform: linux
  • Subsystem: doc

What steps will reproduce the bug?

make doc-only

How often does it reproduce? Is there a required condition?

Always

What is the expected behavior?

docs are build

What do you see instead?

if [ -x /home/matteo/repositories/node/./node ] && [ -e /home/matteo/repositories/node/./node ]; then /home/matteo/repositories/node/./node  tools/doc/generate.js --node-version=v17.0.0 --apilinks=out/doc/apilinks.json doc/api/debugger.md --output-directory=out/doc/api --versions-file=out/previous-doc-versions.json; elif [ -x `command -v node` ] && [ -e `command -v node` ] && [ `command -v node` ]; then `command -v node`  tools/doc/generate.js --node-version=v17.0.0 --apilinks=out/doc/apilinks.json doc/api/debugger.md --output-directory=out/doc/api --versions-file=out/previous-doc-versions.json; else echo "No available node, cannot run \"node  tools/doc/generate.js --node-version=v17.0.0 --apilinks=out/doc/apilinks.json doc/api/debugger.md --output-directory=out/doc/api --versions-file=out/previous-doc-versions.json\""; exit 1; fi;
/home/matteo/repositories/node/tools/doc/node_modules/unified/index.js:469
    throw new Error(
    ^

Error: `processSync` finished async. Use `process` instead
    at assertDone (/home/matteo/repositories/node/tools/doc/node_modules/unified/index.js:469:11)
    at Function.processSync (/home/matteo/repositories/node/tools/doc/node_modules/unified/index.js:401:5)
    at Object.<anonymous> (/home/matteo/repositories/node/tools/doc/html.js:64:4)
    at Module._compile (node:internal/modules/cjs/loader:1109:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1138:10)
    at Module.load (node:internal/modules/cjs/loader:989:32)
    at Function.Module._load (node:internal/modules/cjs/loader:829:14)
    at Module.require (node:internal/modules/cjs/loader:1013:19)
    at require (node:internal/modules/cjs/helpers:93:18)
    at Object.<anonymous> (/home/matteo/repositories/node/tools/doc/generate.js:35:14)
make[1]: *** [Makefile:745: out/doc/api/debugger.html] Error 1
make: *** [Makefile:698: doc-only] Error 2

Additional information

There is likely some old cruft in my checkout that I can't seem to clean up make clean; make distclean don't do the trick.

@Trott
Copy link
Member

Trott commented Jun 5, 2021

I can try to take a look at this later, but my immediate thought is: Does rm -rf tools/doc/node_modules fix it?

@mcollina
Copy link
Member Author

mcollina commented Jun 5, 2021

That did not work :/. For some reason I can't comprehend

.processSync(gtocMD).toString();
would like to be async.

@Trott
Copy link
Member

Trott commented Jun 5, 2021

That means that gtocMD() is supposed to be

That did not work :/. For some reason I can't comprehend

.processSync(gtocMD).toString();

would like to be async.

Yeah, that is odd. Is your git HEAD 78af363 or something else? Does it start working if you first run rm -rf out/doc/api/debugger.* or even rm -rf out/doc?

@mcollina
Copy link
Member Author

mcollina commented Jun 6, 2021

Yes, I'm at 78af363

@Trott
Copy link
Member

Trott commented Jun 6, 2021

Maybe something weird in out/doc/api/debugger.md? Remove out/doc and try again? Or did you do that already?

@mcollina
Copy link
Member Author

mcollina commented Jun 6, 2021

I removed everything several times. Maybe there is some residual in a gitignored dot-folder?

@targos
Copy link
Member

targos commented Jun 6, 2021

Last resort: git clean -fdx will remove everything that's either not tracked or gitignored

@mcollina
Copy link
Member Author

mcollina commented Jun 8, 2021

Unfortunately even that or creating a fresh clone did not do the trick.

I also tried looking at the various steps and I can't seem to find where that becomes async.

@targos
Copy link
Member

targos commented Jun 8, 2021

@mcollina can you try with #38966 ?

@mcollina
Copy link
Member Author

mcollina commented Jun 8, 2021

@mcollina can you try with #38966 ?

That does the trick.

targos added a commit to targos/node that referenced this issue Jun 9, 2021
- Migrated to ESM because some dependencies now require it.
- Did not update `highlight.js` to v11 because it has many breaking
  changes.
- Used non-deprecated `highlight.js` API.

Refs: highlightjs/highlight.js#2277
Fixes: nodejs#38938
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
targos added a commit to targos/node that referenced this issue Jun 11, 2021
- Migrated to ESM because some dependencies now require it.
- Did not update `highlight.js` to v11 because it has many breaking
  changes.
- Used non-deprecated `highlight.js` API.

Refs: highlightjs/highlight.js#2277
Fixes: nodejs#38938
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
danielleadams pushed a commit that referenced this issue Jun 15, 2021
- Migrated to ESM because some dependencies now require it.
- Did not update `highlight.js` to v11 because it has many breaking
  changes.
- Used non-deprecated `highlight.js` API.

Refs: highlightjs/highlight.js#2277
Fixes: #38938
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

PR-URL: #38966
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
danielleadams pushed a commit that referenced this issue Jun 17, 2021
- Migrated to ESM because some dependencies now require it.
- Did not update `highlight.js` to v11 because it has many breaking
  changes.
- Used non-deprecated `highlight.js` API.

Refs: highlightjs/highlight.js#2277
Fixes: #38938
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

PR-URL: #38966
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
richardlau pushed a commit that referenced this issue Jul 19, 2021
- Migrated to ESM because some dependencies now require it.
- Did not update `highlight.js` to v11 because it has many breaking
  changes.
- Used non-deprecated `highlight.js` API.

Refs: highlightjs/highlight.js#2277
Fixes: #38938
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

PR-URL: #38966
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
richardlau pushed a commit that referenced this issue Jul 20, 2021
- Migrated to ESM because some dependencies now require it.
- Did not update `highlight.js` to v11 because it has many breaking
  changes.
- Used non-deprecated `highlight.js` API.

Refs: highlightjs/highlight.js#2277
Fixes: #38938
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

PR-URL: #38966
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
foxxyz pushed a commit to foxxyz/node that referenced this issue Oct 18, 2021
- Migrated to ESM because some dependencies now require it.
- Did not update `highlight.js` to v11 because it has many breaking
  changes.
- Used non-deprecated `highlight.js` API.

Refs: highlightjs/highlight.js#2277
Fixes: nodejs#38938
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

PR-URL: nodejs#38966
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants