As per comments in #1833726: Does panopoly wysiwyg have to depend on panopoly core? and #1833692: Does panopoly theme have to depend on panopoly core?, let's try to reduce reliance on panopoly core in the info files.

CommentFileSizeAuthor
#4 panopoly-reduce_reliance_on_panopoly_core-1840880-4.patch350 bytesbarraponto

Comments

populist’s picture

populist commented
Status: Active » Fixed

This is now all done

populist’s picture

populist commented

Here is a caveat though... the modules that are required sometimes require patches or dev versions which is going to cause problems in the future.

Status: Fixed » Closed (fixed)

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

barraponto’s picture

barraponto
he / him
commented
Status: Closed (fixed) » Needs review
StatusFileSize
new panopoly-reduce_reliance_on_panopoly_core-1840880-4.patch350 bytes

I really don't understand what's the point of moving the recommended dependencies out of panopoly's makefile. UUID and Devel are both dependencies in panopoly.info, but they're only provided by the panopoly_core makefile. I'd say the next step is to remove them from panopoly.info and leave them disabled by default.

populist’s picture

populist commented
Status: Needs review » Closed (fixed)

The goal of this issue was to make sure that those modules are not hardcoded in. They are on by default since folks like using devel, but you can turn them off easily enough!