Skip to content

A fix to inclusion of posttitle.tex#6

Merged
eteq merged 2 commits intoeteq:masterfrom
mattpitkin:master
Dec 22, 2015
Merged

A fix to inclusion of posttitle.tex#6
eteq merged 2 commits intoeteq:masterfrom
mattpitkin:master

Conversation

@mattpitkin
Copy link
Contributor

I've got a fix to checking whether the 'posttitle.tex' file exists - previously if localdir and buildir were different, and --relative-links was set then the check for whether this file exists would fail even if it did exist in localdir.

Also, I've swapped where posttitle.tex is included to be before the abstract.

@mattpitkin
Copy link
Contributor Author

I've just seen that the second point I make would be fixed by the suggestion from #5 in commit 29757fb

@mattpitkin
Copy link
Contributor Author

I've just added another patch (29757fb) that fixes how figures and captions are included in the case that locadir and builddir are different and --relative-links are used. Before this these were not included, or not included properly.

@eteq eteq merged commit 2b8c9c7 into eteq:master Dec 22, 2015
eteq added a commit that referenced this pull request Dec 22, 2015
A fix to inclusion of posttitle.tex
@eteq
Copy link
Owner

eteq commented Dec 22, 2015

Alright, I finally got to this - there was a subtle bug due to figdir getting overridden and one other place that needed the "absolute" path instead of the relative one. But I managed to fix those and merge this into master. So thanks, @mattpitkin !

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