Skip to content

Remove PHP Warning on mkdir()#128

Merged
Mark-H merged 3 commits intomodmore:masterfrom
krismas:bug-init-direxists
Sep 15, 2015
Merged

Remove PHP Warning on mkdir()#128
Mark-H merged 3 commits intomodmore:masterfrom
krismas:bug-init-direxists

Conversation

@krismas
Copy link
Contributor

@krismas krismas commented Aug 21, 2015

Remove PHP Warning: mkdir(): File exists when overwriting previous Gitify installation with the same data/backup directories.

… systems (cf. Docker/Alpine). The invalid path is the first one : `modx-2.3.5-pl/./`, this path cause an error while unzipping : `unzip: can't create directory 'modx-2.3.5-pl/./': File exists`
…Gitify installation with the same data/backup directories.
@Mark-H
Copy link
Member

Mark-H commented Sep 4, 2015

Hey @krismas, thanks for your contributions - I will be getting Gitify up to date and merge stuff in soon.

Could you make sure to branch each PR off the primary (master) branch again, rather than your own updated branch? As you can see, this PR contains all of the other PR commits as well making it a bit harder to review. I'll just merge them in the way you submitted them to be able of properly reviewing them individually, but if you could keep an eye on that in the future that'd be great :)

@Mark-H Mark-H merged commit e902c03 into modmore:master Sep 15, 2015
@Mark-H
Copy link
Member

Mark-H commented Sep 15, 2015

Thanks @krismas, all your PRs are now merged.

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.

2 participants