Closed (fixed)
Project:
Twitter
Version:
7.x-6.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Apr 2013 at 14:38 UTC
Updated:
22 Aug 2015 at 13:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
damienmckennaNot sure what can be done about this because the OAuth project was poorly named. Maybe this might help?
Comment #2
damienmckennaTriggering the testbot.
Comment #3
damienmckennaTriggering the testbot.
Comment #4
damienmckennaClosed a duplicate: #2355833: Twitter incorrectly requires oauth_common
Comment #5
steinmb commentedI suggest we do nothing. This module is not the only one seeing this problem.
Had a look into https://github.com/drush-ops/drush/blob/master/commands/pm/pm.drush.inc
Comment #6
damienmckennaIt's more of a core Drupal limitation than a problem with Drush: #2205271: Project namespace for dependencies
Comment #7
steinmb commentedAh thanks :) Should we put this issue to 'postponed' until there is working D7 backported?
Comment #8
damienmckennaI'm not sure the issue will be resolved for D7, I think we could do a workaround until then. Would you mind giving the patch a try? Thanks.
Comment #9
steinmb commentedNo big difference running from drush.
Comment #10
damienmckennaI finally tested this, and yes, it doesn't work. The reason for this is that hook_requirements() is ran after the info file's dependencies are tested, so we could either remove the line from the info file and use the patch or just deal with it :-\
This patch updates the instructions in README.txt to note the module name conflict.
Comment #12
steinmb commentedWhat does this sentence actually mean?
Why do you not think the core issue will be fixed for D7? It is tagged with a "needs backport" and found nothing in the comments it that indicate that I will not be back ported.
Comment #16
damienmckennaCommitted. I removed the line from the 7.x branches and rewrote it for the 6.x branch so it actually made sense.