The .info file is missing a core = 6.x line.

CommentFileSizeAuthor
#1 comment_subject-info-fix-472740-1.patch652 byteswebchick

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Active » Reviewed & tested by the community
StatusFileSize
new comment_subject-info-fix-472740-1.patch652 bytes

This is trivial (and absolutely necessary) so I'm marking it RTBC. I'm kind of shocked this has not been reported before?

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

I suspect it hasn't been reported before because the tarball has a working .info file. Must be something screwy with the cvs version. Assuming you're using drush to get it?

Michelle

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Issue tags: +DruBB
arhak’s picture

arhak commented

as Michelle said, tarball its ok,
nevertheless we'll support CVS/drush users and add this fix (altogether another fixes)

PS: what does DruBB tag means?

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

DruBB is a project: http://drupal.org/project/drubb

I wonder if DruBB still needs this, though? I think the functionality is built into nodecomment.

Michelle

arhak’s picture

arhak commented

@#5 Michelle, this module got new co-maintainers, so please, feel free to open feature requests or bug reports
for current 6.x-1.x branch take into account #636280: Release 6.x-1.5
also you can use 6.x-2.x branch which by default provides the same functionality (but that one is in alpha stage)

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

Thanks for the offer but I don't actually use this module. I use Nodecomments.

Michelle

arhak’s picture

arhak commented
Status: Reviewed & tested by the community » Fixed
Issue tags: -DruBB

released 6.x-1.5 with this fixed

arhak’s picture

arhak commented
Status: Fixed » Closed (fixed)

almost two weeks now, I'm closing it to clean up the issue queue

arhak’s picture

arhak commented
Title: Impossible to enable this module in Drupal 6 » Support CVS/drush install (AKA core=6.x)
Version: 6.x-1.4 » 6.x-1.x-dev
Category: bug » support
Status: Closed (fixed) » Active

this has to be re-fixed before next release

sinasalek’s picture

sinasalek commented
Status: Active » Fixed

Fixed in head

arhak’s picture

arhak commented
Status: Fixed » Closed (fixed)

thanks sinasalek