I just tried to install the Media module, after installing File entity 7.x-2.x-dev, but I am not allowed to activate the module, because of incompatibility.

It seems like both Media and File Entity were just updated. Is this something that was overlooked? Or is this a File Entity issue?

CommentFileSizeAuthor
#7 media_dependency-1304490-7.patch306 bytesidflood
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Postponed (maintainer needs more info)

Try re-downloading File entity in about 12 hours once the dev release has time to regenerate. If it's still a problem mark this back to active.

MustangGB’s picture

MustangGB CreditAttribution: MustangGB commented
Project: D7 Media » File Entity (fieldable files)
Status: Postponed (maintainer needs more info) » Active

The issue is that the file_entity packing script adds version = "7.x-2.x-dev" after it is faked

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Project: File Entity (fieldable files) » D7 Media
Assigned: Unassigned » Dave Reid

Gah, the real issue here is our version comparing doesn't allow us to use (2.x) in a dependency with a module with version 7.x-2.x-dev. I'm going to remove the version-specific line from media.info.

Stretsh’s picture

Stretsh CreditAttribution: Stretsh commented

I'll be looking out for an update. Just removed both and tried downloading again, but still the same...

Juan C’s picture

Juan C CreditAttribution: Juan C commented

subscribe

Stretsh’s picture

Stretsh CreditAttribution: Stretsh commented

So... Basically, only the dependency is the problem huh? I removed the (2.x) after File Entity in the media.info and it works.

I guess you'll let me know if there is something else that needs to be put there...

idflood’s picture

idflood CreditAttribution: idflood commented
Status: Active » Needs review
FileSize
media_dependency-1304490-7.patch306 bytes

The issue is still present in the 7.x-2.x branch. Here is a simple patch to fix this. Since the dependency doesn't check for specific version the module homepage homepage should be changed to something like this.

Required modules for Media 7.x-2.x

  • CTools
  • File Entity 2.x
  • Views
klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented

Confirming that #7 above fixes things.

Does anyone know if there is any issue filed against core for requiring specific versions of modules in .info files?

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs review » Fixed

I'm committed dependencies[] = file_entity (>1.99) which seems to work although it's a little ugly.
http://drupalcode.org/project/media.git/commit/72386e1

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
CreditAttribution: klonos commented

Yep, that works too. Thanx Dave ;)

Status: Fixed » Closed (fixed)

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