search

cloudsoft / brooklyn-social-apps

Brooklyn templates for social web site applications
Other
0 stars 6 forks source link

Hardcode MySql version to 5.5.29 #4

Closed aledsage closed 11 years ago

aledsage commented 11 years ago

The live test still fails for me though. But I don't understand how it ever worked. In DrupalSshDriver.install, it does sudo("mkdir /etc/postfix") and then getLocation().copyTo(new ByteArrayInputStream(postfixConfig.getBytes()), "/etc/postfix/main.cf"). But the latter will be run as the drupal user, while the directory was created as root user. So the copyTo command will presumably always fail with insufficient permissions to write to that directory. Or am I missing something?

I haven't tried to fix that in this pull request.

buildhive commented 11 years ago

Cloudsoft » brooklyn-social-apps #2 UNSTABLE Looks like there's a problem with this pull request (what's this?)

pveentjer commented 11 years ago

I think it is related because the wrong os is selected. Currently some of the changes to install drupal, are made specifically with debian in mind. I know this is wrong, but we wanted to get a version out of the door asap.

So have you selected debian to run the stuff on?

poweredman commented 11 years ago

Thanks Peter, I hadn't selected Debian specifically. Will try that next.

On an almost related note, do you think we should add a copyTo util that allows us to copy as though using sudo (eg upload as normal user and then do 'sudo mv...').

Aled

Sent from my iPhone

On 2 Feb 2013, at 06:51, Peter Veentjer notifications@github.com wrote:

I think it is related because the wrong os is selected. Currently some of the changes to install drupal, are made specifically with debian in mind. I know this is wrong, but we wanted to get a version out of the door asap.

So have you selected debian to run the stuff on?

— Reply to this email directly or view it on GitHub.

pveentjer commented 11 years ago

What are we going to do with this pull request?

aledsage commented 11 years ago

When 0.5.0-M2 is available, we won't need this (it will fallback to downloading it from http://downloads.cloudsoftcorp.com/brooklyn/repository/). But if we want it to work in the meantime, then this workaround is necessary.

What do you think about merging it, and then reverting when we upgrade to 0.5.0-M2?