This been already broken

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dww’s picture

Status: Needs review » Needs work

Yup, that's worth doing. However, my good friend grep also found this in the 6.x branch:

themes/engines/phptemplate/phptemplate.engine:// $Id$

Rolling a patch, one sec...

TR’s picture

Title: Remove broken and outdated CVS $Id $ from unicode.entities.inc » Remove CVS $Id$ tags from core
Category: task » bug
Status: Needs work » Needs review
FileSize
517 bytes

Found another one. Let's take care of both at the same time.

dww’s picture

Title: Remove CVS $Id$ tags from core » Remove broken and outdated CVS $Id $ from unicode.entities.inc
Category: bug » task
FileSize
517 bytes

Also, I grepped the 7.x and 8.x branches and those are already clean. Only 6.x has these, so the version here is right.

dww’s picture

Title: Remove broken and outdated CVS $Id $ from unicode.entities.inc » Remove CVS $Id$ tags from core
Category: task » bug
Priority: Normal » Minor
Status: Needs review » Reviewed & tested by the community

Sorry, cross post (and identical patch, nice). But, while we're at it, let's be honest -- this is a minor bug (if it's a bug at all). ;)

TR’s picture

That is interesting, the patches are ===, not just functionally identical.

There have been several issues recently which have addressed this in Drupal 7.x and 8.x, where these tags keep cropping up as new commits are made. I suspect we'll be chasing these for a while still.

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, committed.

Status: Fixed » Closed (fixed)

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