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

doctrine na pasta ./bin #80

Open
jackmakiyama opened this issue Aug 8, 2014 · 7 comments
Open

doctrine na pasta ./bin #80

jackmakiyama opened this issue Aug 8, 2014 · 7 comments

Comments

@jackmakiyama
Copy link
Contributor

Os arquivos doctrine dessa pasta não deve ser versionado e também o composer não está indicando a pasta como bin-dir também.

A não ser que ele tenha sido montado, que pode ser o caso.

@eminetto pode confirmar isso?

Se ele não foi escrito por vocês eu vou remover e indicar a pasta bin-dir no composer.
Se foi vou só mudar o include bootstrap.php para include cli-config.php, para que ele funcione corretamente.

@eminetto
Copy link
Contributor

eminetto commented Aug 8, 2014

Pior q eu não tenho certeza disso. Tem como você fazer um teste? Tentar remover o arquivo e fazer um composer update? Para ver se ele cria novamente e como cria?

@jackmakiyama
Copy link
Contributor Author

Não cria, no composer.json não está sendo indicado o diretório e esse ./bin/doctrine inclui o bootstrap da aplicação de vocês não o autoload do composer indicando que foi feito e não gerado.

Se ele não tiver nenhuma ação a mais que o ./vendor/bin/doctrine acho que seria melhor só indicar o bin-dir para o composer.

O que me diz?

@vitornp
Copy link
Contributor

vitornp commented Aug 8, 2014

Na verdade esse arquivo não é o gerado pelo composer e sim um Symfony Console.

Com a tarefa de migrations esse arquivo será alterado para console e ainda está em discussão tem um PR #76 se vc quiser acompanhar e ajduar

@jackmakiyama
Copy link
Contributor Author

@vitornp estou por dentro da PR, vamos esperar ela desenrolar então antes de executar esse aqui.
Não conheço o Symfony Console, mas ainda acho que esse arquivo não deve ser versionado, ainda mais se ele tem a mesma função que o ./vendor/bin/doctrine que é o indicado no README para ser usado. E até penso que ele nem precisa existir já que o indicado é usar o que está dentro da pasta vendor.

@vitornp
Copy link
Contributor

vitornp commented Aug 8, 2014

@jackmakiyama ok, vi que fez um comentário lá eu dei uma breve explicação sobre ele. Se você quiser mais informação pode dar uma olhada na documenteção dele http://symfony.com/doc/current/components/console/introduction.html

@jackmakiyama
Copy link
Contributor Author

Certo, antes ele tinha a mesma função que o gerado pelo composer o tornando redundante. Agora faz mais sentido se renomear como citou lá. Vamos aguardar o andamento para ver como ele vai funcionar e fechar aqui caso necessário.

@vitornp
Copy link
Contributor

vitornp commented Aug 8, 2014

ok

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants