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.

Sign up for GitHub

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

Jump to bottom

Install xindy to support PDF with non-ascii indexes #98

Merged
merged 1 commit into from Mar 19, 2019
Merged

Install xindy to support PDF with non-ascii indexes #98

merged 1 commit into from Mar 19, 2019

Conversation

humitos
Copy link
Member

@humitos humitos commented Mar 6, 2019

@humitos humitos added the Needed: design decision A core team decision is required label Mar 6, 2019
@humitos humitos mentioned this pull request Mar 7, 2019
Closed
agjohnson
agjohnson requested changes Mar 8, 2019
View reviewed changes
Copy link
Contributor

@agjohnson agjohnson left a comment

Choose a reason for hiding this comment

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

We don't need an extra layer just for this package, I'd say it can be compressed into the previous layer. I don't see a problem installing the package either way.

@humitos humitos mentioned this pull request Mar 18, 2019
Open
@humitos humitos removed the Needed: design decision A core team decision is required label Mar 18, 2019
@humitos
Copy link
Member Author

humitos commented Mar 18, 2019

I put together with the previous layer where we install packages related to LaTeX. I left the comment above the RUN. Once the docker image builds I will merge it.

@humitos humitos merged commit 0845c28 into master Mar 19, 2019
@humitos humitos deleted the humitos/install-xindy branch March 19, 2019 14:17
@ax3l ax3l mentioned this pull request Apr 8, 2020
Closed
@jmcnamara jmcnamara mentioned this pull request Oct 27, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@agjohnson agjohnson agjohnson requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@humitos @agjohnson