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