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.

from http://drupal.org/node/231036

CommentFileSizeAuthor
#9 seo_checklist-package-name-581466-9.patch739 bytestraviscarden
#2 581466-seochecklist-info-D6-2.patch608 bytesdave reid

Comments

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

If you decide to leave it, then also consider

It should not use punctuation...
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 581466-seochecklist-info-D6-2.patch608 bytes

Patch attached for 6.x-2.x.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs review
nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS. Thanks!

Status: Fixed » Closed (fixed)

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

lpalgarvio’s picture

lpalgarvio commented
Title: Please remove "package" from .info » package group
Component: Documentation » User interface
Category: bug » task
Priority: Minor » Normal
Status: Closed (fixed) » Active

please add "SEO" package group to the info file (package = SEO)

this is not entirely correct as per Drupal standards, but please think of the admins ;)
it's really annoying to have 50-100 modules in the "Other" fieldset/tab.

many modules follow similar approach (other administration, permission and caching/performance modules) to help admins

some SEO modules are using that package, while for others it has been suggested.

lpalgarvio’s picture

lpalgarvio commented
Title: package group » SEO Checklist » package name
Issue tags: +package name
traviscarden’s picture

traviscarden commented
Title: SEO Checklist » package name » Set "package" in .info to "SEO"
Version: 6.x-2.0 » 7.x-3.x-dev
Status: Active » Needs review
StatusFileSize
new seo_checklist-package-name-581466-9.patch739 bytes

This seems sensible to me. I was able to quickly find four other modules using "SEO" as their package name (Drupal SEO Essentials, Drupal SEO Friend, Drupal SEO Tools, and Page Title). Here's a patch.

traviscarden’s picture

traviscarden commented
Status: Needs review » Fixed

Thanks, all. This is fixed in the new 4.x release. Please try it out and give your feedback in the queue!

traviscarden’s picture

traviscarden commented
Version: 7.x-3.x-dev » 7.x-4.0-beta1

Status: Fixed » Closed (fixed)
Issue tags: -package name

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

Issue tags: +package name

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.