Patch (to be ported)
Project:
Features
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
22 Sep 2011 at 14:38 UTC
Updated:
12 Mar 2015 at 19:05 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
tim.plunkettSee attached.
Comment #2
febbraro commentedInteresting. The packaging script on d.o DO use the double quotes though.
Unstoppable force meets immovable object?
Comment #3
tim.plunkettI'll patch the packaging script next :) Open source++
Comment #4
Grayside commentedHeh, oddly enough when I saw features do that the bits of C++ in my background grabbed ahold of it and I use the doublequotes universally now.
Comment #5
hefox commentedGoing to be annoying change for updating existing features cause every feature will have changed and need to be feature-updated (the git diff u.u!), but that's not anything unusual. Other than that, looks fine.
Comment #6
Grayside commentedWell, it won't require anything because there is no functional difference in .info parsing. The minor annoyance of the git diff when features are updated, will just have to be choked on.
Comment #7
neochief commentedLooks fine for me either. Those who care and have a lot of features can re-generate them pretty easily with "drush features-update-all", so this shouldn't be a big deal.
Comment #8
mpotter commentedSeems good. People are not forced to update their Features because they won't be marked as overridden or anything bad. So this has a low impact and just improves consistency.
Committed to 264a557.
Comment #9
hefox commented