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

Simplification of vagrant installation #272

Closed
wants to merge 1 commit into from
Closed

Simplification of vagrant installation #272

wants to merge 1 commit into from

Conversation

rugabarbo
Copy link
Contributor

Евгений, здравствуй.

Я хочу сделать несколько улучшений для vagrant-установки:

  • сделать vagrant-версию работоспособной (сейчас она не работает из-за ошибки в конфе nginx)
  • обеспечить возможность настройки:
    • часового пояса
    • имени vagrant-бокса

Также хочу внести некоторые другие улучшения.

Этот коммит лишь первый в серии улучшений. Он позволяет ничего не копировать при запуске под вагрантом. Да и для vagrant-файлов не лучшее место в папке docs.

Что думаешь по этому поводу? Может быть был какой-то особый смысл в хранении этих файлов под папкой docs?

@trntv
Copy link
Collaborator

trntv commented Aug 28, 2015

Добрый вечер.

Может быть был какой-то особый смысл в хранении этих файлов под папкой docs?

Вообще смысл в том, что я практически везде переехал на docker и соответственно перевел на него starter-kit. По этой же причине файлы vagrant переехали в docs - для наглядного примера.
Не хочется нагромождать все все в корень приложения. Но я подумаю как лучше все это уместить в одном месте.
А пока не мог бы ты вернуть файлы, но оставить свои изменения, чтобы я смерджил реквест.

@rugabarbo
Copy link
Contributor Author

Этот PR содержит только перемещения файлов для упрощения запуска под вагрантом. Остальные доработки у меня в отдельных ветках. Я предпочитаю не соединять в одном коммите много разных исправлений (так проще делать ревью).

Давай в пределах этого треда решим, как лучше организовать работу с вагрантом? Я могу реализовать и протестировать вариант, который тебе подойдёт.

В пределах данного коммита я пытался сделать такой интерфейс:

  • файл Vagrantfile сразу лежит в руте как /Vagrantfile
  • все вспомогательные файлы вагранта лежат в папке /vagrant

Это позволяет конечному пользователю сразу же сделать vagrant up в рутовой директории проекта.

Мне кажется, что не удастся более компактно разместить вагрант-файлы. Можно конечно положить /Vagrantfile не в рут, а в папку /vagrant, но тогда пользователю для действия vagrant up придётся каждый раз переходить в папку /vagrant - это не очень удобно.

Может быть всё-таки попробуем мой вариант хранения вагрант-инфраструктуры? Я давно работаю с вагрантом и хорошо знаю Ruby, поэтому мог бы поддерживать в актуальном состоянии его инфраструктуру для kit-starter.

@rugabarbo
Copy link
Contributor Author

Я не думаю, что одна дополнительная папка и файл являются "награмождением" в руте проекта. Зато пользователи, которые используют вагрант, скажут тебе спасибо 😊

@rugabarbo
Copy link
Contributor Author

Евгений, я вижу, что моё предложение поддерживать Vagrant-сборку тебе не очень интересно. В связи с этим, я послал свои улучшения в имеющуюся структуру отдельным PR #283. Если всё-таки надумаешь поддерживать сборку вагранта в более удобном виде - напиши мне в личку. А пока что я закрываю этот тикет за невостребованностью.

@rugabarbo rugabarbo closed this Sep 4, 2015
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