II think it would be appropriate to have the Comment RSS module listed under a 'Syndication' section of the modules listing (admin/build/modules). I've included my patch to the .info file.

CommentFileSizeAuthor
commentrss.info.patch528 bytesdave reid

Comments

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Active » Postponed (maintainer needs more info)

What other modules are in this category?

add1sun’s picture

add1sun commented

Hm, honestly the module package should only be used for modules that are actually grouped together (e.g. OG, Views or eCommerce.) I'm not sure that making up a "package" for Syndication is appropriate so I'd vote for won't fixing this.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

I don' think it is bad form to make up module categories like this, but I am not sure there are any other feed *generator* modules in this category, or that this should be there as well.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

I have my FeedBurner module listed under 'Syndication' in a site's module list since it seemed appropriate. Somewhere in the drupal module developer docs I read that you're supposed to list under package such as your primary module category on the project page. Should I not be listing it under that package?

add1sun’s picture

add1sun commented

Well I have been going by the .info file docs in the handbook (http://drupal.org/node/231036) which say:

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

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Postponed (maintainer needs more info) » Closed (won't fix)

Alright, that seems like it makes pretty good sense. I'm going to remove the 'Syndication' package from the FeedBurner module and also mark this issue as won't fix if no one else minds.