Download & Extend

Remove Drush Make dependency definitions (rules.make)

Project:Rules
Version:7.x-2.x-dev
Component:Rules Core
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

This will help with building dependency chains via Drush Make.

rules.make

; Drush Make (http://drupal.org/project/drush_make)
api = 2
core = 7.x

; Dependencies
projects[entity] = 1

Comments

#1

Component:Rules Core» Rules Engine

How does that interfere with possible existing make files (e.g. of distributions) that already include the entity api?

#2

It doesn't break anything. Just sees it, and won't download it twice.

#3

Status:needs review» fixed

ok, then let's do it. Thanks, committed.

#4

Status:fixed» closed (fixed)

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

#5

Status:closed (fixed)» active

If you have a makefile that already downloads Entity API and places it in a specific subdir, this dependency check breaks and the Entity API module is downloaded again. This is particularly problematic if you're initial makefile is pulling a specific tag of the Entity API module - and then the makefile in Rules pulls down a different version.

I know that this is technically a bug in Drush Make - but it's gumming up things here.

#6

Hmm, that's a problem. Do you think we should move this issue over to Drush Make itself?

#7

Yeah. Sorry. In the meantime, selfishly, I'd love to see the Makefile come out of Rules so that it doesn't break the distro I'm working on...

I'll see Jonathan Hedstrom, the Drush Make maintainer, in just a few hours in Denver. I'll ping him on it.

#8

The problems you're experiencing are probably related to #1016924: Dealing with different versions of the same module in recursive makefiles.

#9

I've run into this issue recently where the make file for my install profile downloads both rules and entity into a "contrib" subdirectory. As a result, when rules is downloaded it runs rules.make and I end up with an additional download of entity. I would like to see the make file removed until the issue with drush make can be resolved.

On a side not, the version number for entity in rules.make (1) is not valid.

#10

Status:active» needs review

Attached a patch to remove the make file. If you add this to your make file it will be applied before drush looks for a recursive make file and avoid downloading entity again.

AttachmentSizeStatusTest resultOperations
remove-make-file-1329346-10.patch246 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 258 pass(es).View details

#11

Title:Add Drush Make dependency definitions (rules.make)» Remove Drush Make dependency definitions (rules.make)
Status:needs review» fixed

thanks, I've committed #10 as sadly #5 really is a dealbreaker.

#12

Status:fixed» closed (fixed)

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

#13

Good to see this removed. I think a kind of standard is to provide a .make.example file instead. But I dont really see that as needed here.

nobody click here