This been already broken

Files: 
CommentFileSizeAuthor
#3 1176524-2.no-more-CVS-Ids.D6.patch517 bytesdww
#2 1176524-remove-cvs-tags.patch517 bytesTR
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]
d6-unicode.patch227 bytesandypost
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]

Comments

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

Title:Remove broken and outdated CVS $Id $ from unicode.entities.incRemove CVS $Id$ tags from core
Category:task» bug
Status:Needs work» Needs review
StatusFileSize
new517 bytes
PASSED: [[SimpleTest]]: [MySQL] 190 pass(es).
[ View ]

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

Title:Remove CVS $Id$ tags from coreRemove broken and outdated CVS $Id $ from unicode.entities.inc
Category:bug» task
StatusFileSize
new517 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.

Title:Remove broken and outdated CVS $Id $ from unicode.entities.incRemove 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). ;)

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.

Status:Reviewed & tested by the community» Fixed

Thanks, committed.

Status:Fixed» Closed (fixed)

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