search

panique / mini

Just an extremely simple naked PHP application, useful for small projects and quick prototypes. Some might call it a micro framework :)
1.35k stars 479 forks source link

Replaced redundant code #214

Closed Wruczek closed 8 years ago

Wruczek commented 8 years ago

To fix #199, you can just simply add "-p" argument, which "makes parent directories as needed".

panique commented 8 years ago

Thanks for the commit, but there's a tricky issue:

What you posted is totally correct afaik and exactly like it was back in 2014 when the project started until I realized that the mkdir-command doesn't work sometimes (only sometimes!) when written in one command instead of 3. It's weird and i dont have a real explanaition for it, but after running this bootstrap.sh like 100 times I saw it failing at the mkdir part several times.

My suggestion is that it has to do with folder permissions / ownership when having these folders synced with the host environment (under Windows 7 / 10) and using Virtualbox! Maybe virtualbox / vagrant needs every folder level created step after step and gets confused by creating deeper folders, maybe it's a bug in virtualbox / windows / vagrant / ubuntu or maybe it's just my lacking understanding of how linux works :)

Anyway, to work around this reproduceable issue, it might be the best thing to keep the installer using the 3-lines solution, even if it's redundant for now, maybe this saves somebody's day :)

cheers, chris

panique commented 8 years ago

Ah, little update: this was indeed an issue, described here in this ticket: https://github.com/panique/mini/issues/199