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

Build quick fix #141

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

stanclai
Copy link

@stanclai stanclai commented Mar 3, 2017

This set of patches is the result of consequent eliminating of errors and warnings appeared during building the book in all possible formats (make install).

plfLaTeX lamented that longtable.sty can't be found, and toprule control
sequence is undefined. Solution was to add packages "longtable" and
"booktabs".
This fix in a certain way cancels cryptoparty#116, but it's necessary. The fact is
that during making html and epub output all the images are supposed to
be directly in chapter folders. Commands that link or copy images don't
descent into subdirectories, so images in "assets" folder are not copied
to destination. So, there was two ways to fix this: first, to change
logic to recursively walk into all subfolders and find all images, and
second, to place all images in some uniform way (i.e. only in chapter
directories, along with text files). The latter was much more easier to
make.
In output files the list was formed as a solid stream of words, without
line breaks. One solution is to add two spaces in the end of each line.
It's the way to introduce line break when converting from md to other
formats. Another solution is to put some punctuation, which is more
visual for editing source md. I've chosen the latter.
@stanclai
Copy link
Author

stanclai commented Mar 3, 2017

Yuval Adam, I revised my set of patches after rebasing over main tree. These ones are still essential for successfull build.

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.

1 participant