The package versioning information was added to the committed version of the file in CVS. This information is suppose to be added in packaging and caused me some issues with drush and update status knowing the right version. Patch attached.

CommentFileSizeAuthor
search404_nopackage.patch556 bytespopulist

Comments

grendzy’s picture

This is also present in the DRUPAL-6--1-1 tag. It's incorrectly marked as version = "6.x-1.0", which causes update_status to report the wrong version.

zyxware’s picture

Assigned: Unassigned » zyxware

So I just need to apply this patch and commit back to CVS?

Regards
Zyxware

zyxware’s picture

Status: Needs review » Fixed

I have removed the packaging information from the current release made today.

It should reflect in the Latest Dev snapshot after the nightly builds

Regards
Zyxware

grendzy’s picture

Status: Fixed » Active

Thanks! Can you also remove version = VERSION from the .info file?

BTW thanks for your work on this module. It works so well I'm now usually installing search404 instead of customerror.

zyxware’s picture

Status: Active » Needs review

Hello Grendzy

We have removed the "version = VERSION" from the info file and it should be there in our latest release.

Do check it out and let us know and Thank You for using our module.

Regards
Zyxware

grendzy’s picture

Status: Needs review » Fixed

Looks great, Thanks!

zyxware’s picture

Status: Fixed » Closed (fixed)

Ok then I am closing this issue :)

Thanks
Zyxware

Narno’s picture

Version: 5.x-1.x-dev » 6.x-1.4
Status: Closed (fixed) » Active

This is a same with "package = Other", you can delete it: it's not necessary because "Other" is de native/default package in modules list page for module without package name.

bartezz’s picture

+1 @ Narno.

Most of my optional modules are placed under 'Other'.
Search404 has this defined in the .info file therefore creating a whole new 'Other' list with just Search404 sitting in it....

Cheers

zyxware’s picture

Status: Active » Needs review

Hi Narno

We have removed the packaging info from 6.x version and commited in the dev,

Let us know if this is enough.

Regards
Zyxware

zyxware’s picture

Status: Needs review » Fixed

Marking this as fixed there there has been no activity.

Feel free to open the issue in case it is not resolved.

Regards
Santhosh Raju

Narno’s picture

Thank you zyxware (I know I'm late) ! :-)

Status: Fixed » Closed (fixed)

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