CRM-6736 drush civicrm install command should install in sites/all not sites/default

    Details

    • Type: Patch
    • Status: Done/Fixed
    • Priority: Major
    • Resolution: Fixed/Completed
    • Affects Version/s: 3.2.2
    • Fix Version/s: 3.3.alpha
    • Labels:
      None

      Description

      Right now drush civicrm-install determines the location of the modules directory using:

      drush_get_context('DRUSH_DRUPAL_SITE_ROOT', FALSE);

      That will return the directory with the Drupal settings.php file in it.

      Therefor, drush will install civicrm either in the default directory or a directory named after the domain of the site.

      civicrm, however, seems to want to be installed in sites/all/modules (as I learned from civicrm.settings.php - which sometimes fails if civicrm is in the default directory).

      I've attached a patch that hard codes this behavior. I'm not convinced that's the right way to do it either... but am a bit stumped. Or - should I submit a patch for civicrm.settings.php ?

        Attachments

          Activity

          [CRM-6736] drush civicrm install command should install in sites/all not sites/default
          Sushant Paste added a comment -

          Tested r29880.

          Deepak Srivastava added a comment -

          Its the default installer behavior that should be fixed, similar to as done by UI installer. And user need not specify the install path as a requirement. Installing to a separate path specified by user, could be an optional thing.

          Sushant Paste added a comment -

          Tested r30016.

            People

            • Assignee:
              Sushant Paste
              Reporter:
              Jamie McClelland

              Dates

              • Created:
                Updated:
                Resolved: