search

Coderockr / orcamentos

Aplicativo de gerenciamento de Orçamentos
117 stars 71 forks source link

doctrine na pasta ./bin #80

Open jackmakiyama opened 10 years ago

jackmakiyama commented 10 years ago

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 commented 10 years ago

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 commented 10 years ago

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 commented 10 years ago

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 commented 10 years ago

@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 commented 10 years ago

@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 commented 10 years ago

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 commented 10 years ago

ok