Skip to content

Declare PANDOC_DATA_DIR and PANDOC_TEMPLATES_DIR as ENV variables #278

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

Merged
merged 1 commit into from
May 14, 2025

Conversation

daamien
Copy link
Collaborator

@daamien daamien commented May 9, 2025

closes #274

Copy link
Collaborator

@alerque alerque left a comment

Choose a reason for hiding this comment

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

This looks like a good improvement to me, with the caveat that it feels like this might be a breaking change for somebody downstream and a little difficult to figure out why it broke. I don't have a way to test this and haven't reasoned out exactly what could happen, this is just a gut feeling that even if the change is for the best it might leave somebody hanging that depended on some override of the previous status quo.

@daamien
Copy link
Collaborator Author

daamien commented May 14, 2025

This looks like a good improvement to me, with the caveat that it feels like this might be a breaking change for somebody downstream and a little difficult to figure out why it broke. I don't have a way to test this and haven't reasoned out exactly what could happen, this is just a gut feeling that even if the change is for the best it might leave somebody hanging that depended on some override of the previous status quo.

From what I understand this will affect people who want to modifiy the pandoc data and template directories at build time, with like

docker build . [ ... ] --build-arg PANDOC_DATA_HOME=/opt/pandoc

I really doubt that anyone is actually doing that :)

Copy link
Collaborator

@alerque alerque left a comment

Choose a reason for hiding this comment

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

Fair enough. I say good to go on that concern.

Now just pending the syntax issue @tarleb spotted and merge conflict resolution.

@daamien daamien force-pushed the 274_declare_pandoc_env_vars branch from 7ec946c to a89e999 Compare May 14, 2025 14:20
@daamien daamien merged commit 85be113 into main May 14, 2025
1 check passed
@alerque alerque deleted the 274_declare_pandoc_env_vars branch May 15, 2025 07:50
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.

Declare PANDOC_DATA_DIR and TEMPLATE_DIR as ENV variables
3 participants