Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hello,
I just update media module to the Media 7.x-1.0-rc1 on my site. The Media Gallery have a strict dependence with Media 7.x-1.0-beta5. I remove the Media module version string from .info file dependencies section, and have a quick look at modules interaction.
Currently, everything seems to work well with Media 7.x-1.0-rc1 and Media Gallery 7.x-1.0-beta6, but I did not test it thoroughly.
May be it would be fine remove version restriction from dependencies of Media module.
Comment | File | Size | Author |
---|---|---|---|
#15 | media_gallery-media-compat.patch | 864 bytes | effulgentsia |
#14 | media_gallery-media-compat.patch | 446 bytes | effulgentsia |
Comments
Comment #1
did1979 CreditAttribution: did1979 commentedHello,
I also have this issue.
I have not tried to delete the dependency with media gallery 7.x.
I follow the issue.
bye.
did
Comment #2
David_Rothstein CreditAttribution: David_Rothstein commentedRight, so it seems like the next version of Media Gallery should definitely be made compatible with the latest Media 1.x release, whatever it turns out to be. Good to hear it's mostly there already ;)
We put the restrictions in the info file since both modules were in flux, and people were flooding the issue queue with support requests when they couldn't get random versions of each module to work with each other.... As both modules become more stable, though, we can probably consider removing that restriction.
Comment #3
fenstratWe're also using Media Gallery with Media 7.x-1.0-rc1, removed the dependency and things seem fine.
Comment #4
tamsoftware CreditAttribution: tamsoftware commentedworked indeed here too.
Changed
dependencies[] = media (1.0-beta5)
to
dependencies[] = media
at line 8 of sites/all/modules/media_gallery/media_gallery.info
and it allowed me to update fine. Thanks !
I now have a cosmetic question: I have searched all my modules dependancies for such a version specification with
grep -r --include "*.info" -h dependencies *
but this is the only module that uses it.
What would be the syntax to force media version 1.x (e.g. not version 2) but not specify the ".x" in question?
Cheers and thanks a lot for this great module
Franck Horlaville
TAM Software
Comment #5
Balbo CreditAttribution: Balbo commented+1
Comment #6
Carlos Miranda Levy CreditAttribution: Carlos Miranda Levy commentedsubscribe
Comment #7
brodiebrodie CreditAttribution: brodiebrodie commentedsubscribe
Comment #8
AlanO CreditAttribution: AlanO commentedsubscribe
Comment #9
Jeffrey C. CreditAttribution: Jeffrey C. commentedOne more success attempt! I simply edited the dependencies and 1.0-rc1 seems just fine!
Comment #10
Jeffrey C. CreditAttribution: Jeffrey C. commentedClosing this issue. Don't know if I have the priority to do this, but everything just seems fine and so far no bugs are reported. Please reopen this issue if necessary.
Comment #11
fenstrat@legendm33066 Nothing has been done here, you may have fixed it manually by removing the dependency in the .info file but that needs to make it back into the modules code via a patch. An issue is marked as fixed once a patch has been applied to the code base.
Comment #12
Jeffrey C. CreditAttribution: Jeffrey C. commentedOh sorry! I'll try to make a patch!
Comment #13
adam_b CreditAttribution: adam_b commentedsubscribe
Comment #14
effulgentsia CreditAttribution: effulgentsia commentedMedia Gallery 1.x-dev does not list dependencies on particular versions, because we presume that if you're running a dev snapshot, it's your responsibility to evaluate compatibility.
However, as per #2, for Media Gallery beta6, we listed a firm dependency on Media beta5 and Multiform beta2, because we knew it to not be compatible with earlier versions, and given some flux during beta releases, suspected it wouldn't be compatible with later versions either.
Now that Media is in RC phase, this patch relaxes the dependency on Media to beta5 or later, but only within 1.x, because it very likely is not yet compatible with Media 2.x, which is itself still in its unstable phase.
If someone can verify that this patch has the desired effect of flagging incompatibility with Media 1.x-beta4 and Media 2.x-unstable1, but not flagging incompatibility for Media 1.x-beta5 or Media 1.x-rc1, please set this issue to RTBC. However, this patch should only be committed immediately before releasing Media Gallery beta7, and then immediately reverted after cutting the beta7 tag, so that dev snapshots can continue to not require firm version dependencies.
Comment #15
effulgentsia CreditAttribution: effulgentsia commentedThis also changes a line in the README.txt.
Comment #16
effulgentsia CreditAttribution: effulgentsia commentedI committed http://drupalcode.org/project/media_gallery.git/commitdiff/99a5b17466e6a... which differs from #15 in that it allows all Media 1.x versions, including 1.x-dev and pre-beta5. I'm hoping that very few people now are on old versions of Media and trying to use new versions of Media Gallery, but there is a contingent using dev snapshots of Media, so I opted to make things smoother for the latter group than the former one.
Getting version dependencies in .info files right is tricky. See #1013302: Versioned dependencies fail with dev versions and git clones.
Anyway, Media 1.0-rc2 and Media Gallery 1.0-beta7 are now released and available on the respective project pages.
Comment #17.0
(not verified) CreditAttribution: commentedtypos