The ID tag in the code looks hand-written, which is unnecessary. Is there a particular reason why the maintainer does this instead of including the $Id$ CVS tag?

Comments

SamAMac’s picture

SamAMac commented
Assigned: Unassigned » SamAMac

It is a pain changing the ID tags myself on each commit, but I have enough problems with people using the wrong version of gsitemap with their Drupal version that I don't want to have people asking me why "gsitemap 4.6" has v1.38.2.15 at the top.

I must admit a fair amount of ignorance of how CVS works, so if there's a good reason beyond my own convenience for using $Id$, I'll change it.

SamAMac’s picture

SamAMac commented
Status: Active » Fixed

Yeah, it's getting obnoxious to change it every time. I have changed this in HEAD and the Drupal 5 branch, in favor of an additional comment indicating core compatibility.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)