Needs review
Project:
Features
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
10 Jun 2014 at 14:43 UTC
Updated:
24 Nov 2015 at 21:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
geberele commentedComment #2
geberele commentedComment #4
geberele commented1: features-drush_features_components_d6-2283617-0.patch queued for re-testing.
Comment #5
geberele commentedNot sure why the test is failing, the patch works fine on branch 6.x-1.x
Comment #7
geberele commentedComment #8
geberele commentedComment #10
geberele commentedComment #11
geberele commentedComment #12
geberele commentedComment #14
geberele commentedComment #15
geberele commentedComment #17
geberele commentedThe patch drush_features_components_d6-2283617-3.patch is fine, it fails because there are exceptions not related to this patch.
Comment #18
geberele commentedComment #20
joachim commentedI've applied the patch and I can confirm it works great! I've sent it for a re-test; if that's green I'll RTBC it.
Very glad to see this, as it's become a part of my workflow on D7 sites and I was missing it when working on old D6 sites!
Thank you geberele for the backport! :)
Comment #22
jwilson3The patch no longer applied against 6.x-1.x-dev (6.x-1.2+19-dev) because the module already contains a function called
_drush_features_component_listwhich is used by the features-add command.So I've rerolled this and renamed all of the functions to use the plural
_drush_features_components_list().