When deploying a new CiviCRM platform using the example drush makefile in 7.x-3.x-dev, the resulting sites/all/modules/civicrm directory only contains the files and directories from the l10n package. No errors are reported during the platform creation process.

core = 7.x
api = 2
projects[] = drupal
translations[] = fr
defaults[projects][l10n_url] = http://ftp.drupal.org/files/translations/l10n_server.xml

projects[drupal][patch][] = "file:///var/aegir/patches/letsencrypt_challenge.patch"

projects[front][subdir] = contrib
projects[logintoboggan][subdir] = contrib
projects[views][subdir] = contrib
projects[ctools][subdir] = contrib

projects[civicrm][type] = "module"
projects[civicrm][directory_name] = "civicrm"
projects[civicrm][download][type] = "get"
projects[civicrm][download][url] = "http://downloads.sourceforge.net/project/civicrm/civicrm-stable/4.6.14/civicrm-4.6.14-drupal.tar.gz"

libraries[civicrm_l10n][destination] = "modules"
libraries[civicrm_l10n][directory_name] = "civicrm"
libraries[civicrm_l10n][download][type] = "get"
libraries[civicrm_l10n][download][url] = "http://downloads.sourceforge.net/project/civicrm/civicrm-stable/4.6.14/civicrm-4.6.14-l10n.tar.gz"
libraries[civicrm_l10n][overwrite] = TRUE
CommentFileSizeAuthor
#8 example_makefile-2699211-8.patch2.23 KBhelmo

Comments

lavamind created an issue. See original summary.

lavamind’s picture

lavamind commented
Issue summary: View changes
helmo’s picture

helmo commented

I can confirm this, with drush 8.0.5

I tried removing the '[overwrite] = TRUE' option... no luck.

The --help info from drush make mentions 'Overwrite existing directories. Default is to merge' but it does not apear to be doing that.

  • bgm committed 3547e9d on 7.x-3.x 
    Issue #2699211: update civicrm.make example.
bgm’s picture

bgm commented
Status: Active » Postponed (maintainer needs more info)

I updated the example makefile for CiviCRM 4.7.6 (it was really outdated!), however, initially I ran with "drush make civicrm.make --debug", and saw that tar was running out of disk space in /tmp.

The l10n tar is pretty big, and sometimes it's common practice to have a pretty small /tmp (100-200 MB).

Can you run with --debug and see what the error was?

helmo’s picture

helmo commented

Just tried the updated makefile in drush 8.0.5 and drush 8.1... Same.

Last part of the verbose log:

ctools-7.x-1.9 downloaded.                                                                                                                                                                                                                                                      [ok]
/home/helmo/.drush/cache/download/https---download.civicrm.org-civicrm-4.7.6-drupal.tar.gz retrieved from cache.                                                                                                                                                                   [notice]
civicrm downloaded from https://download.civicrm.org/civicrm-4.7.6-drupal.tar.gz.                                                                                                                                                                                               [ok]
Command dispatch complete                                                                                                                                                                                                                                                          [notice]

Undefined property: DrushMakeProject_Library::$type make.project.inc:154                                                                                                                                                                                                           [notice]
/home/helmo/.drush/cache/download/https---download.civicrm.org-civicrm-4.7.6-l10n.tar.gz retrieved from cache.                                                                                                                                                                     [notice]
civicrm_l10n downloaded from https://download.civicrm.org/civicrm-4.7.6-l10n.tar.gz.                                                                                                                                                                                            [ok]
Executing: tar  -C /tmp/drush_tmp_1461696284_571fb71cb914e -xzf civicrm-4.7.6-l10n.tar.gz
Command dispatch complete

The Undefined property can be fixed by adding libraries[civicrm_l10n][type] = "libraries"

ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented

Here's a more-or-less stock makefile for a CiviCRM platform: https://github.com/nditech/NDIplatforms/blob/0.4.x/makefiles/stock/civic.... It uses the new .make.yml format, which is recommended now.

helmo’s picture

helmo commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new example_makefile-2699211-8.patch2.23 KB

Here's a patch that moves us to a make.yml file based on @ergonlogic's example.

bgm’s picture

bgm commented
Status: Needs review » Reviewed & tested by the community

Seems good, please merge!

  • helmo committed 143ce56 on 7.x-3.x 
    Issue #2699211 by helmo: Example makefile doesn't work
helmo’s picture

helmo commented
Status: Reviewed & tested by the community » Fixed
ergonlogic’s picture

ergonlogic
he/him
Primary language English
Location Montréal, Québec 🇨🇦
commented

I just updated the release version included in the makefile, and added some explanatory docs: http://cgit.drupalcode.org/hosting_civicrm/commit/?id=f2bb3b82b091118f6c...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.