From: Iwan V. <iv...@la...> - 2007-01-22 15:42:43
|
Hi there, we're using spyce in a multithreaded process and are getting an exception in taglib.py: ValueError: list.remove(x): x not in list This happens on line 55 in taglib.py: 54: sys.path.reverse() 55: sys.path.remove(os.path.dirname(self.stack[-1]._lib.__file__)) 56: sys.path.reverse() It would seem from our logs that several threads try to manipulate sys.path, resulting in the exception. And looking at the code, it will certainly break in such a situation... Do you agree? Is this a bug? -i |
From: Jonathan E. <jon...@ca...> - 2007-01-22 18:00:46
|
yeah, that is totally broken I can't think of anything that would need that kind of path manipulation, either. (And if it does you can always add it to sys.path permanently in your config.) I'd say just rip it out (and the corresponding append in tagPush) and see if anything breaks :) -Jonathan On Mon, 22 Jan 2007 15:56:20 +0200, "Iwan Vosloo" <iv...@la...> said: > Hi there, > > we're using spyce in a multithreaded process and are getting an > exception in taglib.py: > > ValueError: list.remove(x): x not in list > > This happens on line 55 in taglib.py: > > 54: sys.path.reverse() > 55: sys.path.remove(os.path.dirname(self.stack[-1]._lib.__file__)) > 56: sys.path.reverse() > > It would seem from our logs that several threads try to manipulate > sys.path, resulting in the exception. > > And looking at the code, it will certainly break in such a situation... > > Do you agree? Is this a bug? > > -i > > > ------------------------------------------------------------------------- > 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 > _______________________________________________ > Spyce-users mailing list > Spy...@li... > https://lists.sourceforge.net/lists/listinfo/spyce-users |
From: Iwan V. <iv...@la...> - 2007-01-24 09:31:10
|
Hi Jonathan, we removed the path manipulations in taglib.py as you suggest. Our tests all seem to be running still... So I suppose you can nuke that code... Thanks -i On Mon, 2007-01-22 at 09:59 -0800, Jonathan Ellis wrote: > yeah, that is totally broken > > I can't think of anything that would need that kind of path > manipulation, either. (And if it does you can always add it to sys.path > permanently in your config.) I'd say just rip it out (and the > corresponding append in tagPush) and see if anything breaks :) > > -Jonathan > > On Mon, 22 Jan 2007 15:56:20 +0200, "Iwan Vosloo" <iv...@la...> said: > > Hi there, > > > > we're using spyce in a multithreaded process and are getting an > > exception in taglib.py: > > > > ValueError: list.remove(x): x not in list > > > > This happens on line 55 in taglib.py: > > > > 54: sys.path.reverse() > > 55: sys.path.remove(os.path.dirname(self.stack[-1]._lib.__file__)) > > 56: sys.path.reverse() > > > > It would seem from our logs that several threads try to manipulate > > sys.path, resulting in the exception. > > > > And looking at the code, it will certainly break in such a situation... > > > > Do you agree? Is this a bug? > > > > -i > > > > > > ------------------------------------------------------------------------- > > 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 > > _______________________________________________ > > Spyce-users mailing list > > Spy...@li... > > https://lists.sourceforge.net/lists/listinfo/spyce-users > > ------------------------------------------------------------------------- > 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 > _______________________________________________ > Spyce-users mailing list > Spy...@li... > https://lists.sourceforge.net/lists/listinfo/spyce-users |
From: Iwan V. <iv...@la...> - 2007-02-15 13:41:18
|
Hi Jonathan, as I said previously, you suggested fix below works fine. Are you going to also rip out the said path manipulation in the sources? (Just need to know if I should flag the change when we upgrade again) Tx -i On Mon, 2007-01-22 at 09:59 -0800, Jonathan Ellis wrote: > yeah, that is totally broken > > I can't think of anything that would need that kind of path > manipulation, either. (And if it does you can always add it to sys.path > permanently in your config.) I'd say just rip it out (and the > corresponding append in tagPush) and see if anything breaks :) > > -Jonathan > > On Mon, 22 Jan 2007 15:56:20 +0200, "Iwan Vosloo" <iv...@la...> said: > > Hi there, > > > > we're using spyce in a multithreaded process and are getting an > > exception in taglib.py: > > > > ValueError: list.remove(x): x not in list > > > > This happens on line 55 in taglib.py: > > > > 54: sys.path.reverse() > > 55: sys.path.remove(os.path.dirname(self.stack[-1]._lib.__file__)) > > 56: sys.path.reverse() > > > > It would seem from our logs that several threads try to manipulate > > sys.path, resulting in the exception. > > > > And looking at the code, it will certainly break in such a situation... > > > > Do you agree? Is this a bug? > > > > -i > > > > > > ------------------------------------------------------------------------- > > 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 > > _______________________________________________ > > Spyce-users mailing list > > Spy...@li... > > https://lists.sourceforge.net/lists/listinfo/spyce-users |
From: Jonathan E. <jon...@ca...> - 2007-02-15 14:59:15
|
Yes, this (and a couple other similar problems) are gone in the 2.2 branch. On Thu, 15 Feb 2007 15:41:01 +0200, "Iwan Vosloo" <iv...@la...> said: > Hi Jonathan, > > as I said previously, you suggested fix below works fine. Are you going > to also rip out the said path manipulation in the sources? (Just need > to know if I should flag the change when we upgrade again) |