Re: Pycheckmate.py throws IndexError in Python bundle
On 21 Aug 2009, at 17:13, Jay Soffian wrote:
> On Wed, Aug 19, 2009 at 9:43 AM, Allan Odgaard<mailinglist <at> textmate.org
> > wrote:
>> On 18 Aug 2009, at 21:46, J W-d wrote:
>>> [...] Line 26 reads:
>>> __version__ = "$Revision$"
>>> When it should read:
>>> __version__ = "$Revision 11734 $"
>> The revision is put into that line by svn if you cat/export/checkout
>> the file.
>> I see however that the git mirror does not have this revision, so I
>> assuming you updated the bundle from git (maybe via GetBundles).
>> We’ll need to address that (I didn’t know about it until now).
>>> Though I must say... What's the use of the revision, exactly?? It
>>> that it has only served to break the script and not much else. :)
>> The idea with the revision is that when a user reports that “I used
>> ⌃⇧V and it gives me this error” we can ask the user what
>> of the script he has, so we can see if it has since been fixed :)
> Ugh, I can fix this, but:
> a) I need to pull up your message on how write access to the bundles
> work, since it's been a while since I touched anything (or I may just
> send a patch to the list...)
Write bundles(at)textmate dot org for stuff related to svn access.
> b) I'm not sure what the best alternative is for the svn tag, it may
> just be burning a revision into the source file on checkin and
> remembering to update it manually on each commit.
I am not sure what a good fix is either, and this command is not the
only one which use svn keywords. I sort of feel this should be fixed
in the “script” that does the git mirroring, so if I were you,
I’d leave it up to me to find a more universal fix for this problem.
textmate mailing list
textmate <at> lists.macromates.com