-
-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add an asset solution for Windows users #22
Comments
Yeah I'm really not sure what to do here. I just don't use Windows on a regular basis, so I don't have a test environment. I'd very happily accept pull requests for ensuring that Craft-Scripts works on Windows as well, though! |
Yeah, I didn’t think you’d have one. Just hoping some Windows user who tries to use it takes this on. ;) |
Hola! I'm actually about to start work on porting these for Windows. I'm a full time student so it may take some time, but I'll do a PR as soon as I'm done :) |
Ah very cool @alexstojda ! |
@alexstojda Have you made any progress on this? I'd be interested in working on it with you - I'm a Windows user too and I'd really like to use these scripts. 😄 |
Not yet! Looks like it's going to be a summer project to be honest. School has taken most of my time sadly.
…________________________________
From: Katie Fritz <notifications@github.com>
Sent: Thursday, March 29, 2018 5:13:39 PM
To: nystudio107/craft-scripts
Cc: Alex; Mention
Subject: Re: [nystudio107/craft-scripts] Add an asset solution for Windows users (#22)
@alexstojda<https://github.com/alexstojda> Have you made any progress on this? I'd be interested in working on it with you - I'm a Windows user too and I'd really like to use these scripts. 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#22 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ABSn9HNB7fdwIlW7Mddpg_tEnUCx0sGfks5tjU6DgaJpZM4P3wje>.
|
@alexstojda I got things working using cygwin. I'll try to write something up eventually, but in the meantime, let me know if you want any troubleshooting help. |
@KatieMFritz I'd love to hear the pertinent details of what you had to do to get it working. It's either that or I might just end up going back to Laravel/Homestead for my pc boxes. |
@onebrightlight Oh boy. 😆 Okay, here's what I did, minus the going in circles:
To run scripts:
|
@KatieMFritz you are a genius! I knew about Cygwin but completely forgot about it |
@khalwat I think this is an acceptable enough solution to include in the readme and close the issue! |
@KatieMFritz while "" will probably work, the default setting should work as well:
Do you mind if I steal your writeup and add it to the docs? |
@khalwat Do it! |
PS - when uploading |
FYI, you don't need to use Cygwin if you're on Windows 10; Windows 10 has the Windows Subsystem for Linux (aka WSL) which allows native use of Linux shells via a container. The whole thing is pretty seamless. I'll write some docs. |
PR me @adrienne ! |
So this is obviously not something that affects many people, I know most of us are on Macs, but one thing I came across is that Rsync does not exist for Windows users. There are a few options out there for us to use, but many of them don't have support for all the flags your script in
./pull_assets.sh
uses.It's not an urgent issue at all, but if anyone has a solution for getting around this requirement, it'd be rad. It's the only stumbling block I have in getting this environment up and running in a near 1:1 setup with my Macs.
The text was updated successfully, but these errors were encountered: