Author: duncan
Date: Mon Nov 19 14:07:27 2007
New Revision: 10145

Log:
[ 1834488 ] [PATCH] dont hardcode /tmp in lastfm plugin
Patch from Henne Vogelsang applied


Modified:
   branches/rel-1-7/freevo/ChangeLog
   branches/rel-1-7/freevo/src/audio/plugins/lastfm.py
   branches/rel-1/freevo/ChangeLog
   branches/rel-1/freevo/src/audio/plugins/lastfm.py

Modified: branches/rel-1-7/freevo/ChangeLog
==============================================================================
--- branches/rel-1-7/freevo/ChangeLog   (original)
+++ branches/rel-1-7/freevo/ChangeLog   Mon Nov 19 14:07:27 2007
@@ -21,6 +21,7 @@
  * Updated mplayer to allow the volume events to passed over and shown 
(F#1834486)
  * Renamed plug-in audio.playlist to audio.playlists (B#1834140)
  * Fixed itv plug-in, having "No player for this item found" 
(B#1833018,B#1833829)
+ * Fixed lastfm plug-in path using /tmp to use FREEVO_CACHEDIR (B#1834488)
  * Fixed helper plugins, not allowing tuples as arguments (B#1832837)
 
 == Release 1.7.4 (2007-11-15) ==

Modified: branches/rel-1-7/freevo/src/audio/plugins/lastfm.py
==============================================================================
--- branches/rel-1-7/freevo/src/audio/plugins/lastfm.py (original)
+++ branches/rel-1-7/freevo/src/audio/plugins/lastfm.py Mon Nov 19 14:07:27 2007
@@ -129,9 +129,9 @@
 
     def download_cover(self,pic_url):
         # Download album Cover to /tmp/freevo
-        os.system("rm /tmp/freevo/cover*.jpg")
+        os.system('rm -f %s/lfmcover_*.jpg' % config.FREEVO_CACHEDIR )
         self.covercount +=1
-        savefile = '/tmp/freevo/cover_' + str(time.time()) + '.jpg'
+        savefile = config.FREEVO_CACHEDIR + '/lfmcover_' + str(time.time()) + 
'.jpg'
         pic_file = urllib.urlopen(pic_url).read()
         save = open(savefile, 'w')
         print >> save, pic_file

Modified: branches/rel-1/freevo/ChangeLog
==============================================================================
--- branches/rel-1/freevo/ChangeLog     (original)
+++ branches/rel-1/freevo/ChangeLog     Mon Nov 19 14:07:27 2007
@@ -26,6 +26,7 @@
  * Updated mplayer to allow the volume events to passed over and shown 
(F#1834486)
  * Renamed plug-in audio.playlist to audio.playlists (B#1834140)
  * Fixed itv plug-in, having "No player for this item found" 
(B#1833018,B#1833829)
+ * Fixed lastfm plug-in path using /tmp to use FREEVO_CACHEDIR (B#1834488)
  * Fixed helper plugins, not allowing tuples as arguments (B#1832837)
 
 == Release 1.7.4 (2007-11-15) ==

Modified: branches/rel-1/freevo/src/audio/plugins/lastfm.py
==============================================================================
--- branches/rel-1/freevo/src/audio/plugins/lastfm.py   (original)
+++ branches/rel-1/freevo/src/audio/plugins/lastfm.py   Mon Nov 19 14:07:27 2007
@@ -129,9 +129,9 @@
 
     def download_cover(self,pic_url):
         # Download album Cover to /tmp/freevo
-        os.system("rm /tmp/freevo/cover*.jpg")
+        os.system('rm -f %s/lfmcover_*.jpg' % config.FREEVO_CACHEDIR )
         self.covercount +=1
-        savefile = '/tmp/freevo/cover_' + str(time.time()) + '.jpg'
+        savefile = config.FREEVO_CACHEDIR + '/lfmcover_' + str(time.time()) + 
'.jpg'
         pic_file = urllib.urlopen(pic_url).read()
         save = open(savefile, 'w')
         print >> save, pic_file

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Freevo-cvslog mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/freevo-cvslog

Reply via email to