removeObsoleteCopies fails on windows due to mixed dir separators

contao-community-alliance / composer-plugin

Composer plugin that provide contao integration and installer.

GNU Lesser General Public License v3.0

7 stars 10 forks source link

Closed backbone87 closed 7 years ago

backbone87 commented 10 years ago

das problem hatte ich schon mal vor einer ganzen weile angesprochen und damals auch schon das problem gefunden. habe heute mal wieder meine lokale testumgebung am laufen gehabt und einen quickfix dafür gebaut. folgendes habe ich vor dem array_diff in https://github.com/contao-community-alliance/composer-plugin/blob/develop/src/ContaoCommunityAlliance/Composer/Plugin/CopyInstaller.php#L130 eingefügt: $copies = str_replace(array('/', '\\'), DIRECTORY_SEPARATOR, $copies)

ich weiß das ist sehr quick'n'dirty, aber tut erstmal

tristanlins commented 10 years ago

:+1:

backbone87 commented 10 years ago

wir sollten wirklich den ganzen copy und pfad verarbeitungsprozess nochmal sauber reimplementieren.

tristanlins commented 10 years ago

Wie heißt es so schön "es ist historisch gewachsen, wird Zeit es neu zu machen" ;-)

discordier commented 10 years ago

Wieso bin ich hier bitte assigned? Was hab ich mit Windows am Hut?

tristanlins commented 10 years ago

@discordier nicht mit Windows, aber ggf. mit der Umsetzung eines Refactorings, zusammen mit @backbone87 ;-)

discordier commented 10 years ago

I will be in need of a properly installed windows box to fix this issue.

Requirements: composer must be runable from command line and phpunit must also be working.

backbone87 commented 10 years ago

willst du dir selbst ne VM aufsetzen oder wollen wir screen sharen? die ursache für dieses problem habe ich ja schon beschrieben.

tristanlins commented 10 years ago

Hotfixed by e6775b2aec7ed711442c6fc5ab32c7c895deb56c We still need to step more into and find out why the directory separator is mixed up.

discordier commented 7 years ago

Closing as we imply it as fixed