Skip to content
This repository has been archived by the owner on Nov 6, 2018. It is now read-only.

Add travis-ci build #25

Merged
merged 1 commit into from
Aug 7, 2016
Merged

Add travis-ci build #25

merged 1 commit into from
Aug 7, 2016

Conversation

lamtev
Copy link
Contributor

@lamtev lamtev commented Aug 6, 2016

Add travis-ci build of bachelor_thesis, coursework, report, presentation

@lamtev
Copy link
Contributor Author

lamtev commented Aug 6, 2016

Link to travis

@ejiektpobehuk
Copy link
Owner

Сборка бака не совсем корректна (не хватает генераци номенклатуры).
makeindex thesis.nlo -s nomencl.ist -o thesis.nls

В сборке шаблонов меня чутка пугают условия(факт их наличия). Есть #23 , предлагающий решить данную проблему.

Я сейчас быстренько всем накидаю make файлы, И проблемой останется только захардкоженность списка тестируемых шаблонов. Думаю. не критично, но решаемо.

ps. трэвис только на амазон артефакты кидает?

@ejiektpobehuk
Copy link
Owner

добавил make файлы в 85c0062.

@ejiektpobehuk
Copy link
Owner

P.s. Какую роль играет трэвис для даннной репы? Проверяет, что шаблон ещё собирается?

@lamtev
Copy link
Contributor Author

lamtev commented Aug 7, 2016

За исключением команды gs, инструкции по сборке были взяты отсюда.

Сборка тревисом настроена на данный момент чисто для тестирования. Если сборка упала, значит, внесенные изменения некорректны. Можно будет сразу смотреть, падает ли сборка при пулл реквесте. Без деплоя пока.

@ejiektpobehuk
Copy link
Owner

Jenkins я давно не трогал =
Поэтому актуальность в нём относительная.

Переделай travis_build.sh на make и будет отлично!

Шаблоны r_tables и gnuplot не собираешь из-за нехватки сред?

@ghost
Copy link

ghost commented Aug 7, 2016

Антон, не переделывайте на make файлы, они слишком убогие, их надо удалить.
Владислав, вы внесли их после того, как получили реквест, надо было принять реквест, а потом улучшать как хотите.

@ejiektpobehuk
Copy link
Owner

@vilegzhanina, Travis требует вмешательства в репозиторий, чем отличается от дженкинса, способного существовать со своими конфигами отдельно.
Так вот зачем вносить в репозиторий куда-то вшитый башскрипт сборки, когда проблему можно и нужно решать в другом месте.

В рамках одного пулл-реквеста, рассуждения о пост-фактум внеснных изменениях логичны. Однако, мне кажется, что проще развернуть один пулл реквест и принять следующий, чем принять первый и потом заводить тикет на тему модификации, что, конечно же, может и не быть правдой.

Make файлы справляются со своей задачей и удобны для автоматизации, если есть предложения интереснее, прошу вносить их в #23.

P.s. всем этим никак не хочу задеть Антона. @lamtev, ты большой молодец, и я с удовольствием приму твой следующий пулл реквест (с корректировкой на make файлы), хотя и этот крайне хорош и полезен!

@ghost
Copy link

ghost commented Aug 7, 2016

Вы придумали какую-то проблему (какую?), и хотите, чтобы кто-то ее за вас решил.
Что нелогичного принять value в репозиторий, а потом улучшать скрипты как ВАМ хочется?

Могу напомнить, что вы являетесь владельцем репозитория не потому, что это была ваша идея, не потому, что вы внесли большой вклад в разработку шаблонов, не потому что вы наиболее компетентны, и на ваш репозиторий ссылаются как на основной, не потому, что это ваша заслуга и авторитет.
Вы владеете этим репозиторием потому, что я надеялась, что вы справитесь с процессом непрерывных улучшений шаблонов сообществом студентов. Но пока что вы практикуете ненужную бюрократию, блокируете улучшения, ставите условием принятия реквестов ваше личное видение архитектуры, и ведете себя как будто вы здесь гуру и хозяин и как все то, что было перечислено в первом списке НЕ.
Это неприемлемо.

@ejiektpobehuk
Copy link
Owner

@vilegzhanina проблема данного пулл реквеста - некорректная сборка шаблона бака. У данной проблемы есть корни: отсутствие каких-либо инструкций по сборке шаблонов.
Просить автора пулл реквеста о модицикациях может быть вполне здравой идеей, так как он может быть более компетентен в вопросе их внесения и может выразить сомнение в их необходимости.

@ejiektpobehuk
Copy link
Owner

Данный пулл реквест принимаю, так как вопрос о принадлежности недостатков процесса сборки имеет спорное к нему тоношение

@ejiektpobehuk ejiektpobehuk merged commit 26e408e into ejiektpobehuk:master Aug 7, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants