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

rewrite Makefiles; remove trailing whitespaces #243

Closed
wants to merge 1 commit into from

Conversation

matsievskiysv
Copy link

Полностью переписана система сборки.

  • все настройки latexmk перенесены в единый файл latexmkrc, управляемый при помощи переменных среды
  • Makefile в папках dissertation, synopsis и presentation теперь самодостаточны и идентичны (можно было бы оставить одну версию, а остальные сделать в виде ссылок, но не знаю, как это будет работать на Windows). Сборка каждой части может производиться из её (этой части) папки
  • добавлены файлы unix.mk и windows.mk, которые подгружаются в зависимости от операционной системы. В них перенесён, например, выбор шрифтов
  • замена всех примеров сборок одним examples

Кроме того:

  • убраны пробелы в конце строк
  • код презентации теперь собитается из под xelatex или lualatex
  • добавлен рецепт проверки библиографии при помощи biber

Стоит продумать взаимодействие системы сборки и файла common/setup.tex. В настоящий момент, при использовании make, все настройки файла игнорируются.

Сборка работает из под Debian. Надо проверить под Windows (#240).

@Lenchik
Copy link
Collaborator

Lenchik commented Mar 4, 2019

Вероятно, в этом виде целиком этот PR не пройдёт.
Я сейчас вчитаюсь ещё в код и позадаю там вопросы.
А пока по описанному здесь:

замена всех примеров сборок одним examples

Вот это точно пока не надо. Отдельные куски оказались удобны и для проверок. То есть есть случаи, когда нет смысла делать все примеры.

код презентации теперь собирается из под xelatex или lualatex

А почему pdflatex при этом заброшен?

@Lenchik Lenchik self-assigned this Mar 4, 2019
@Lenchik
Copy link
Collaborator

Lenchik commented Mar 4, 2019

От сборки не из корня мы как раз давно ушли. И это было хорошо. Надо, конечно, поискать прошлые обсуждения в PR и issue. Насколько помню, при сборке из корня весь комплект хорошо ложится в онлайновые редакторы.

@matsievskiysv
Copy link
Author

matsievskiysv commented Mar 4, 2019

@Lenchik Для презентации в общем-то надо использовать те-же настройки шрифтов, что из для остального. Переписал под xelatex, т.к. при сборке как было кириллица в шрифтах не отображалась.

@Lenchik
Copy link
Collaborator

Lenchik commented Mar 4, 2019

Я закоммичу кое-что из этого PR: по пробелам точно.
Презентацию поправлю (в части трёх движков), раз востребовано стало.
Можно в отдельном issue обсудить, что там делать по шрифтам и почему (я пока не согласен, что там те же шрифты должны быть, это же в общем виде не печатный материал).

@Lenchik Lenchik mentioned this pull request Mar 4, 2019
@Lenchik
Copy link
Collaborator

Lenchik commented Mar 4, 2019

В чём преимущество от переименовывания папок?

@matsievskiysv
Copy link
Author

В чём преимущество от переименовывания папок?

Это для автоопределения названия основного файла

TARGET ?= $(notdir $(CURDIR))

@Lenchik
Copy link
Collaborator

Lenchik commented Mar 4, 2019

CONTRIBUTING.md, если и стоит куда-то двигать (он работает, как надо), то в рекомендованные современной инструкцией места: https://help.github.com/en/articles/setting-guidelines-for-repository-contributors

@Lenchik
Copy link
Collaborator

Lenchik commented Mar 4, 2019

Если есть желание по какой-то причине продвигать изменения структуры вроде part1\main, предлагаю завести issue и там пообсуждать сначала.

@matsievskiysv
Copy link
Author

В общем да, получился массивный PR. Надо отдельные части под разными issue обсудить. Но в целом всё зависит от того, как будет реализован Makefile.

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.

None yet

2 participants