Bugs item #1298350, was opened at 2005-09-22 07:25
Message generated for change (Comment added) made by duncanwebb
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=446895&aid=1298350&group_id=46652

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: plugins
Group: None
>Status: Pending
>Resolution: Fixed
Priority: 5
Submitted By: Al Crowley (albert_crowley)
Assigned to: Nobody/Anonymous (nobody)
Summary: problem detecting Mame version (with possible fix)

Initial Comment:
Hi,
I've got a newish Fedora Core 4 system with Freevo
installed via RPM freevo-1.5.3-1_fc4.  I've also
installed xmame via RPM.  When I tried to open the MAME
menu item (under Games) I was getting a crash with error:
local variable 'result' referenced vefore assignment. 
(line 279)

I think the regexp in that function is wrong for my
xmame install.  (I'm not a python programmer though, so
I can't be sure)  When I call xmame --version, the
version comes out:
xmame (x11) versoin 0.99 (Aug 16 2005)

The regex looks like it expects three sets of digits
and it's only getting two for my xmame install.

I think I was able to fix this by replacing the regexp
on line 272 with:
".*version\s+(\d+)\.(\d+)\.?(\d?)\s+"
(note: I changed the last two + signs to ? marks)

I didn't test this much, but the fix seems to be
working for me and thought I would share it.
-Al

----------------------------------------------------------------------

>Comment By: Duncan Webb (duncanwebb)
Date: 2006-09-23 16:46

Message:
Logged In: YES 
user_id=104395

New test is:
this_re = re.compile( ".*version\s+(\d+)\.(\d+)\s+" )

Since only group(2) is used there is no need for a third group

----------------------------------------------------------------------

Comment By: Duncan Webb (duncanwebb)
Date: 2006-09-23 16:46

Message:
Logged In: YES 
user_id=104395

Patch applied to branches/rel-1-5/freevo at:
svn://svn.freevo.org/freevo/branches/rel-1-5/freevo

Please test

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=446895&aid=1298350&group_id=46652

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel

Reply via email to