Me and a colleague discovered a "bug" of sorts in MediaRSS.info.

If you download version 6.x-2.9 via CVS you can see this:

name = Media RSS
description = Creates a media field for image nodes on RSS feeds
package = Views

dependencies[]=views

core = 6.x
; Information added by drupal.org packaging script on 2008-09-23
version = "6.x-1.0"
core = "6.x"
project = "mediarss"
datestamp = "1222207534"

But if you download the tar-file and look in that MediaRSS.info it says:

name = Media RSS
description = Creates a media field for image nodes on RSS feeds
package = Views

dependencies[]=views

core = 6.x
; Information added by drupal.org packaging script on 2008-09-23
version = "6.x-1.0"
core = "6.x"
project = "mediarss"
datestamp = "1222207534"

; Information added by drupal.org packaging script on 2009-05-14
version = "6.x-2.9"
core = "6.x"
project = "mediarss"
datestamp = "1242293461"

Some info comes twice and with different values. The module seems to be working OK, but the Drupal installation says that there's a newer version.

I took a quick glance at the code, didn't use diff, and it seem to be the same. Maybe it's only a bug in the packaging script, but it's definitely worth checking out.

Best regards,
Andreas Rydberg

Comments

Tarnaurion’s picture

Tarnaurion CreditAttribution: Tarnaurion commented

One other thing; when i cat the file mediarss/CVS/Tag it says "NDRUPAL-6--2-9".

Microbe’s picture

Microbe CreditAttribution: Microbe commented

The information in the pre-tar MediaRSS.info doesn't matter because when packed into the tar the new release information is added and so variables take a new value. I have had problems with the Drupal update status script before to do with the MediaRSS module, but not using this latest release. I believe this to be due to the update script as apposed to the MediaRSS module as you may notice other modules also have previous packing data in there .info file which doesn't seem to affect the update status for that module.

I'm not sure why there is an 'N' in the cvs tag. This module is the only time i have used cvs so I don't really know what I'm doing using it. Looking at http://cvs.drupal.org I cannot see the "NDRUPAL" tag though. I will remember to be careful next time i assign a new tag though.

Thanks for the pointers towards possible errors and I will try to remember not to include old packing data in the .info files in newer releases just incase.
Peter

Tarnaurion’s picture

Tarnaurion CreditAttribution: Tarnaurion commented

The "N" in the CVS-tag is there in every Tag-file, like [module-catalog]/CVS/Tag. So you can disregard the "N", I Just copy-pasted exactly what cat printed out.

I posted the CVS-tag to ensure that it was the latest version (2.9).

But it seem to be a problem with the packaging script. Just wanted to give you a heads up.

// Andreas Rydberg