Skip to content

Conversation

robertpanzer
Copy link
Member

Thank you for opening a pull request and contributing to AsciidoctorJ!

Please take a bit of time giving some details about your pull request:

Kind of change

  • Bug fix
  • New non-breaking feature
  • New breaking feature
  • Documentation update
  • Build improvement

Description

What is the goal of this pull request?

Make the CLI set the :mkdirs option.

How does it achieve that?

Similar to the implementation of Asciidoctor it sets the option :mkdirs if the input is not read from stdin.

Are there any alternative ways to implement this?

Are there any implications of this pull request? Anything a user must know?

Issue

Fixes #1241

@robertpanzer
Copy link
Member Author

@mojavelinux This PR should be a minimal solution for adding the mkdirs option as reported in #1241
It would generate all results directly into the target directory though, so it would at most create that target directory, but not any structure below that which would reflect the path of the documents relative to the source directory set with -R.

@mojavelinux
Copy link
Member

That's fine because this is orthogonal to supporting **/*.adoc. Most uses of the CLI are to convert a single document to a specified output file or directory. That is what is needed at a minimum in order for AsciidoctorJ to be used with the Antora PDF extension.

@mojavelinux
Copy link
Member

For the test, I would not recommend testing **/*.adoc. Leave that to the other PR. This PR should just test that the CLI will create the necessary directories to write the output to the specified file (singular).

@robertpanzer robertpanzer merged commit c7aba4f into asciidoctor:v2.5.x Nov 23, 2023
@robertpanzer robertpanzer deleted the fix-1241-mkdirs branch November 23, 2023 13:26
@mojavelinux
Copy link
Member

🎉

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