Skip to content

main local/partial build setup#6553

Merged
djencks merged 1 commit into
apache:mainfrom
djencks:main-local-docs-build
Dec 19, 2021
Merged

main local/partial build setup#6553
djencks merged 1 commit into
apache:mainfrom
djencks:main-local-docs-build

Conversation

@djencks
Copy link
Copy Markdown
Contributor

@djencks djencks commented Dec 17, 2021

This works now although there are some spurious errors from camel-spring-boot branches we don't want.

Once I upgrade the camel-website dependencies (apache/camel-website#727) the errors will be gone.

@djencks djencks force-pushed the main-local-docs-build branch from b915ffd to a6ec99f Compare December 18, 2021 01:37
@djencks
Copy link
Copy Markdown
Contributor Author

djencks commented Dec 18, 2021

It now works with the following problem for me. I'd like feedback from others on both linux and macOS whether they have the same problem: it's a bug in Antora AFAICT.

I get an error like this:

[17:36:35.262] ERROR (@djencks/antora-source-watch): EEXIST: file already exists, mkdir '/Users/david/projects/camel/camel-website/documentation/components/next/eips/_attachments/Hohpe_EIP_camel_20150622.zip'
Cause: Error (no stacktrace)

Deleting the source zips "solves" the problem. I haven't been able to make a small example project, but it occurs when I modify the playbook to use my local camel clone (just like source-map does, but hardcoded).

To delete the files,

rm ../core/camel-core-engine/src/main/docs/modules/eips/attachments/Hohpe_EIP_camel_*

in the docs directory where you are running local-build.sh.

@djencks djencks force-pushed the main-local-docs-build branch from a6ec99f to 9770362 Compare December 18, 2021 03:16
@djencks
Copy link
Copy Markdown
Contributor Author

djencks commented Dec 18, 2021

For the versions relying on symlinks rather than copying (main, 3.14.x, 3.13.x, and 3.12.x), the file watcher will not detect changes to the original files under components, but at least in Intellij IDEA you can edit the symlinked "copy" under docs/components and the file watcher will detect a change. I don't know how this works :-)

@davsclaus
Copy link
Copy Markdown
Contributor

That is fine David, we really only work on "main" for new doc content, and for released versions its only smaller backports of bugs etc so often very limited doc changes there.

About the .zip file, then maybe we can drop it - we can instead just put it somewhere else in a git repo and point people there. Its a zip file with the EIP icons in Visio (a digram tool).

@djencks djencks merged commit 48f92b0 into apache:main Dec 19, 2021
@djencks
Copy link
Copy Markdown
Contributor Author

djencks commented Dec 19, 2021

The zip error seems to be intermittent now, and it never occurs with the main build using the git repo. If it turns out to be unfixable and a big nuisance we could put the zips in the camel-website static dir, but I'd rather not.

I'd like to merge the changes for the other branches as well, I work on all the branches :-) and having them all work the same way makes it much easier for me.

@djencks djencks deleted the main-local-docs-build branch December 20, 2021 17:20
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

Successfully merging this pull request may close these issues.

2 participants