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

Adding support for Windows machines #9

Merged
merged 1 commit into from
Apr 4, 2018

Conversation

benjaminprojas
Copy link
Contributor

No description provided.

@getninefiveone
Copy link

Will Windows be consistently supported all the time or was this a limited realease?

@randomstuart
Copy link

randomstuart commented Jan 30, 2018

This fix doesn't really work, it still requires that you use C://Users so if it's on another drive i.e. D:, E:, F: like me, you are still out of luck!

The path restriction needs to be completely removed!

I've managed to get it to partially update but it still adds the C drive to the path, i.e. C:\d\Local Sites\mywebsite\logs\apache this was achieved by commenting out lines 202-210 and 244-256. I will look into it some more tomorrow when i have some time.

@benjaminprojas
Copy link
Contributor Author

@randomstuart correct, that is a restriction. I needed to make that happen and didn't have an opportunity to do that yet.

@randomstuart
Copy link

Hi @benjaminprojas, thanks for the response its much appreciated.

Do you think that you will be able to sort this anytime soon as I'm in a real pickle as I need to launch a website but i cannot access it for final approval?

@benjaminprojas
Copy link
Contributor Author

I don't think this will be fixed anytime soon unfortunately. My work schedule is very full at the moment.

@claygriffiths claygriffiths merged commit 3e3568d into getflywheel:master Apr 4, 2018
@claygriffiths claygriffiths mentioned this pull request Apr 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants