Closed (fixed)
Project:
Media Gallery
Version:
7.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
26 Sep 2011 at 14:06 UTC
Updated:
4 Jan 2014 at 01:11 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
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 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 commentedworked indeed here too.
Changed
dependencies[] = media (1.0-beta5)to
dependencies[] = mediaat 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 commented+1
Comment #6
Carlos Miranda Levy commentedsubscribe
Comment #7
brodiebrodie commentedsubscribe
Comment #8
AlanO commentedsubscribe
Comment #9
Jeffrey C. commentedOne more success attempt! I simply edited the dependencies and 1.0-rc1 seems just fine!
Comment #10
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. commentedOh sorry! I'll try to make a patch!
Comment #13
adam_b commentedsubscribe
Comment #14
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 commentedThis also changes a line in the README.txt.
Comment #16
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: Composer metadata on dev versions doesn't work in update.php.
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) commentedtypos