Skip to content
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

Sql-sync fails on when remote is OS X/BSD in some cases #415

Closed
jbrauer opened this issue Jan 29, 2014 · 4 comments
Closed

Sql-sync fails on when remote is OS X/BSD in some cases #415

jbrauer opened this issue Jan 29, 2014 · 4 comments

Comments

@jbrauer
Copy link
Contributor

@jbrauer jbrauer commented Jan 29, 2014

Not sure how I only found this now but in running sql-sync this line https://github.com/drush-ops/drush/blob/master/commands/sql/sync.sql.inc#L344 has the following command:

stat --format="%Y"

This isn't supported on OS X 1.5.8 or 1.9 that I tested on. It produces this error...
$ stat --format
stat: illegal option -- -

The BSD derived stat command on these systems would use:

stat -f "%m"

It doesn't seem that we should have to test for what system things are on but should find a universally workable method.

@jbrauer
Copy link
Contributor Author

@jbrauer jbrauer commented Apr 13, 2014

quick work-around is to use --no-cache with drush 6.2 when the remote is BSD.

@weitzman
Copy link
Member

@weitzman weitzman commented Apr 14, 2014

Another quick workaround is to upgrade to master branch as the sql-syn implementation has been radically simplified.

@weitzman weitzman closed this Apr 14, 2014
@jbrauer
Copy link
Contributor Author

@jbrauer jbrauer commented Apr 14, 2014

@weitzman if I'm not mistaken though that has to be done on both sides? At least it didn't seem possible to be on drush 6 and master at once...

@weitzman
Copy link
Member

@weitzman weitzman commented May 5, 2014

I think the best we can do is to support Drush6 on one side and it will be a bit messy as Drush can't cleanup temp files and such. Probably best to have Drush7 on both sides, yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.