[development] version = "$Name$" considered harmful in .info files

[Khalid Baheyeldin]

On 6/13/07, Derek Wright <drupal at dwwright.net> wrote:
>
> Hello world,
>
> It was easy to miss this in my last message[1] to the "Announcing
> cvs_deploy.module" thread, so I'm starting a new one...
>
> The only reason we ever recommended people add:
>
> version = "$Name$"
>
> to the .info files they checked into CVS was for the convenience of
> the power-users who deploy from CVS.  These people should just use
> the new cvs_deploy.module[2] instead.
>
> This line has caused lots of trouble and confusion for lots of
> people.  I'm sorry I ever suggested it, and wish I had just written
> and released the cvs_deploy.module right when we were adding the
> version information to the modules page in D5.  Oh well, live and
> learn...
>
> I've edited the 4.7.x -> 5.x upgrade docs[3] and the handbook page
> about writing .info files[4] to reflect the new best practice:
>
> > version (Optional)
> > The version string will ordinarily be added by drupal.org when a
> > release is created and a tarball packaged. However, if your module
> > is not being hosted on the drupal.org infrastructure, you can give
> > your module whatever version string makes sense.
> > Users getting their modules directly from CVS will not have a
> > version string, since the .info files checked into CVS do not
> > define a version. These users are encouraged to use the CVS deploy
> > [2] module to provide accurate version strings for the admin/build/
> > modules page for modules in directories checked out directly from CVS.
> >
> > In the past (before the CVS deploy module existed), it was
> > recommended to use this:
> >
> > version = "$Name$"
> > However, this led to much confusion and is no longer recommended.
> > If you are a module developer, and you already have the above line
> > in your .info files checked into the Drupal CVS repository, you
> > should remove that line.
> Please let me know if you have any questions or complaints.
>
> Thanks,
> -Derek (dww)
>
> [1] http://lists.drupal.org/pipermail/development/2007-June/024698.html
> [2] http://drupal.org/project/cvs_deploy
> [3] http://drupal.org/node/64279
> [4] http://drupal.org/node/101009
>


Should we do a mass checkout and remove this line from all packages?
This way we start with a clean slate now, and not wait for every developer
to catch on.

We have done that in the past for several things.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.drupal.org/pipermail/development/attachments/20070613/ce3971fe/attachment.htm