-
Notifications
You must be signed in to change notification settings - Fork 39
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 a run-updates.sh script to run updates from the command line #820
Comments
I like the sound of this. Should the
something like that? |
I think we're just talking about a shell version of |
Does |
Yes, if we can get @greganderson or @populist or others over at Pantheon to install the drush commands for Backdrop. |
Though we'll need to fix Drush so that doesn't break Drupal 7 installations when the Backdrop extension is installed. backdrop-contrib/backdrop-drush-extension#9 |
@quicksketch I'm not sure how that works on pantheon - it may not matter since each site is on it's own? |
The way it works on Pantheon is that each CMS we deploy (Drupal 7, Drupal 8, WordPress, Backdrop) has special options. In the case of Drush, sites based on D7 uses a different version of Drush than D8. Backdrop uses the same version as D7 of course. |
@populist what we/Backdrop actually need is what is currently the But for now the backdrop/drush extension needs the thanks, |
@serundeputy assuming that @greg-1-anderson gets that version running for Drupal 8, we can easily switch Backdrop to use that version instead. |
Currently, Pantheon is running Drush 8 (current master branch) for all Drupal 8, and all new Drupal 7 sites. If you have a very old Drupal 7 site, it will be running Drush 5, but the easy workaround here for Backdrop is to just make a new Pantheon site, so you can be sure that your base Drush is the right version. From there, I'm not sure why the base configuration isn't working (presuming that your Backdrop Drush bootstrap class is appropriately loaded and available), but I posted a suggested workaround at #1682 that could be tried. |
Also, before trying #1682, I should also ask if anyone tried just putting the Backdrop drush commands at PANTHEON_BACKDROP_ROOT/drush. That should be sufficient on a new Pantheon site running Drush 8. |
thanks @populist and @greg-1-anderson for looking at these issues! @greg-1-anderson I've installed a brand new Pantheon Backdrop site. Here is a drush updb attempt:
any ideas? Looks like it is picking up thanks, |
Drush 5 is totally wrong. I was going to try to fix the Drush version for you, but I can't find your site. Did you delete it already? Maybe try making a new Drupal 8 site, remove everything from the repository, and re-add Backdrop. Then commit and install through the normal web UI, and see what version of Drush you get. If you do this, your framework should be set to 'drupal8', and you should as a result always get Drush 8. |
I guess I didn't ask Hubot nicely enough. Found it, set your Drush to 8. See if it runs better now. I'm going to take a look at your framework setting too. |
Oh, Backdrop has its own framework value. Makes sense; also explains why it does not work out of the box. |
different:
|
I'll bet you would really like to have php 5 running on that endpoint too. Made that change as well; sorry I didn't notice that the first time. |
@greg-1-anderson did you change the framework value or something? now it is looking for other versions.
maybe one step closer here. ~Geoff |
The framework value is still "backdrop", but I forced your Drush to version 8, and your PHP to version 5.5. It looks like you are running Drush 8 now, but the backdrop include file is not being loaded by Drush. |
@greg-1-anderson so i'll try the #1682 method now; on this site |
So, your backdrop class actually needs to go in BACKDROP_ROOT/drush/bootstrap. I moved it there, and now we're getting a little farther. Drush is selecting your bootstrap class, but it is now saying that it cannot find a backdrop installation directory. Take a look at my last two commits, and see if you can debug your site detection code. You probably don't need to put all of Drush in the drush folder at this point. |
@serundeputy ^^ |
@greg-1-anderson does this mean it is failing on
I put in some debug code and does not seem to get past |
Well, the good news, at least, is that it appears that the Backdrop bootstrap object is being correctly selected. |
😆 |
Not getting as far as you are; when I run |
@greg-1-anderson
thanks, |
oh; but you got Backdrop installation could not be found; hrmm; |
Same result with cc all. |
As far as I know, Drush is working well now. Further issues can be opened in Drush's issue queue. |
I guess it depends on wether this issues is about getting a client update.php working or about getting update.php to work via Pantheon dashboard; much of the conversation in this issues is about Pantheon dashboard. That being said we could reopen this issues in backdrop-ops/backdrop-pantheon if that is the intention. |
On pantheon the updates can't be run from the dashboard because that depends on drush, and we don't have drush working quite yet. As an interim solution (and since there are other people who may want to run updates from the command line) we should add a run-updates.sh script into the scripts directory in core.
The text was updated successfully, but these errors were encountered: