As described in this issue in the Apps queue, installing Panopoly 1.17 via 'drush si' was failing to download the panopoly_demo module:

#2430915: Drush not installing 'default apps': "Downloading updates failed:Failed to download from "

The patch attached there fixes the problem. Let's get this included in Panopoly!

Comments

  • dsnopek committed 5ad446e on 7.x-1.x 
    Update Panopoly Core for Issue #2430921: Failing to download '...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Component: Miscellaneous » Core
Status: Active » Fixed
Related issues: +#2430925: Test allowing Apps to pull in the panopoly_demo module when running via Drush

Committed!

However, we really need a test for this so we can make sure we don't break it again: #2430925: Test allowing Apps to pull in the panopoly_demo module when running via Drush

Status: Fixed » Closed (fixed)

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

pmackay’s picture

pmackay commented

Could I ask, if using Panopoly as a base distro, how could the download of panopoly_demo be disabled in the site install process if its not desired?

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented

In the main Panopoly distribution, there is this bit of code in panopoly_install_tasks() in panopoly.profile:

  // Add the Panopoly app selection to the installation process
  $panopoly_server = array(
    'machine name' => 'panopoly',
    'default apps' => array('panopoly_demo'),
    'default content callback' => 'panopoly_default_content',
  );
  require_once(drupal_get_path('module', 'apps') . '/apps.profile.inc');
  $tasks = $tasks + apps_profile_install_tasks($install_state, $panopoly_server);

Simply don't include this in your child distribution! Or instead include a different variation which has the panopoly app server, but doesn't include panopoly_demo as one of the default apps.

pmackay’s picture

pmackay commented

Many thanks @dsnopek!