1. Remove the package = Spam control line. As per http://drupal.org/node/231036:

    If your module comes with other modules or is meant to be used exclusively with other modules, enter the name of the package here. If left blank, the module will be listed as 'Other'. In general, this field should only be used by large multi-module packages, or by modules meant to extend these packages, such as CCK, Views, E-Commerce, Organic Groups and the like. All other modules should leave this blank. As a guideline, four or more modules that depend on each other (or all on a single module) make a good candidate for a package. Fewer probably do not.

  2. Added core = 6.x. From the above link it's a required value for the module to work.
  3. Removed everything else unnecessary because it will be automatically added by the drupal.org packaging script when you create releases.
CommentFileSizeAuthor
#1 602310-hashcash-cleanup-info-D6.patch896 bytesdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
StatusFileSize
new 602310-hashcash-cleanup-info-D6.patch896 bytes
sdrycroft’s picture

sdrycroft commented
Status: Needs review » Closed (won't fix)

Works for me. Package is in there for a reason, if you don't like it, feel free to remove it.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Closed (won't fix) » Active

Ok I can live with the package line, but take a look at the .info included in the latest release:

; $Id: hashcash.info,v 1.7 2009/09/07 15:04:38 sdrycroft Exp $
name = Hashcash
description = Prevents SPAM from reaching the server by implementing hashcash
package = Spam control
; Information added by drupal.org packaging script on 2008-07-16
version = "5.x-1.4"
project = "hashcash"
datestamp = "1216202421"


core = 6.x
; Information added by drupal.org packaging script on 2009-02-04
version = "6.x-1.5"
core = "6.x"
project = "hashcash"
datestamp = "1233752477"


; Information added by drupal.org packaging script on 2009-10-12
version = "6.x-2.2"
core = "6.x"
project = "hashcash"
datestamp = "1255358732"

Those last five lines are the ones automatically included by the packaging script. Why why why do you have all the other stuff in there? It has no purpose and affects people using the CVS version of the module.

sdrycroft’s picture

sdrycroft commented
Status: Active » Fixed

In CVS, thanks David. Also removed "package" line - I thought it was "EDIT" which most of my packages get tagged with, so given that it isn't, there's no issue removing it.

Status: Fixed » Closed (fixed)

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