When getting autosave from CVS, autosave.info has old packaging info in it from 6.x-2.2. This makes update status constantly tell me my version is out of date. This info should not be committed into CVS.

CommentFileSizeAuthor
autosave-info.patch560 bytesJody Lynn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented
Status: Needs review » Needs work

Should not have "Package" either.

liquidcms’s picture

liquidcms CreditAttribution: liquidcms commented

package is Other; which is correct.

If left off it will just end up in Other - but no reason it can't be explicitly set.

liquidcms’s picture

liquidcms CreditAttribution: liquidcms commented

unless someone wants to declare a new "standard" package like "Editor".. :)

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented

According to Writing .info files (Drupal 6.x):

package
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.

jcisio’s picture

jcisio CreditAttribution: jcisio commented

#1 is correct as per #4. The different is that if "Package = Other" it is lost in a site that doesn't use English (if package is undefined, it will be t('Other')).