Bug#641803: podget: playlists are always empty

2012-12-29 Thread Dave Vehrs
Updated this fix to include a check for mktemp in addition to temfile.  
So it should work for people who have either.


This patch will be including in version 0.6.9, for which I submitted a 
request for sponsorship.  Here's hoping we get it soon!


Dave Vehrs


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#641803: podget: playlists are always empty

2011-09-16 Thread Renat Sabitov
Package: podget
Version: 0.5.8-1
Severity: normal

Playlists created by podget are always empty. The problem is in the buggy pipe 
which takes data from the same file that it uses to put data back. Patch is 
available:


--- /usr/bin/podget.old 2011-09-16 14:29:17.0 +0400
+++ /usr/bin/podget 2011-09-16 14:32:46.0 +0400
@@ -623,7 +623,10 @@

 # Sort new playlist
 if [ -e $dir_library/$playlist_name ] ; then
-cat $dir_library/$playlist_name | sort  
$dir_library/$playlist_name
+   PL_TMP=`tempfile`
+cat $dir_library/$playlist_name | sort  $PL_TMP
+   cat $PL_TMP  $dir_library/$playlist_name
+   rm $PL_TMP

 # Create ASX Playlist
 if [ ${asx_playlist} -gt 0 ] ; then



-- System Information:
Debian Release: 6.0.2
  APT prefers stable
  APT policy: (500, 'stable')
Architecture: i386 (i686)

Kernel: Linux 2.6.32-5-486
Locale: LANG=ru_RU.UTF-8, LC_CTYPE=ru_RU.UTF-8 (charmap=UTF-8) (ignored: LC_ALL 
set to ru_RU.UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages podget depends on:
ii  bash4.1-3The GNU Bourne Again SHell
ii  coreutils   8.5-1GNU core utilities
ii  grep2.6.3-3  GNU grep, egrep and fgrep
ii  sed 4.2.1-7  The GNU sed stream editor
ii  tofrodos1.7.8.debian.1-2 Converts DOS - Unix text files, 
ii  wget1.12-2.1 retrieves files from the web

podget recommends no packages.

Versions of packages podget suggests:
ii  libc6 2.11.2-10  Embedded GNU C Library: Shared lib

-- no debconf information



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#641803: podget: playlists are always empty

2011-09-16 Thread Dave Vehrs
Actually had this fixed in the Sourceforge version of Podget but I
liked your use of the tempfile command.  So I modified my version to
use it as well and modified the Debian requirements to include
debianutils.

I will be submitting a new version to mentors.debian.net to update the
version in Debian soon.

Thank you and I'll let you know when its acceoted,

Dave

On Fri, Sep 16, 2011 at 4:52 AM, Renat Sabitov s...@srr.pp.ru wrote:
 Package: podget
 Version: 0.5.8-1
 Severity: normal

 Playlists created by podget are always empty. The problem is in the buggy 
 pipe which takes data from the same file that it uses to put data back. Patch 
 is available:


 --- /usr/bin/podget.old 2011-09-16 14:29:17.0 +0400
 +++ /usr/bin/podget     2011-09-16 14:32:46.0 +0400
 @@ -623,7 +623,10 @@

     # Sort new playlist
     if [ -e $dir_library/$playlist_name ] ; then
 -        cat $dir_library/$playlist_name | sort  
 $dir_library/$playlist_name
 +       PL_TMP=`tempfile`
 +        cat $dir_library/$playlist_name | sort  $PL_TMP
 +       cat $PL_TMP  $dir_library/$playlist_name
 +       rm $PL_TMP

         # Create ASX Playlist
         if [ ${asx_playlist} -gt 0 ] ; then



 -- System Information:
 Debian Release: 6.0.2
  APT prefers stable
  APT policy: (500, 'stable')
 Architecture: i386 (i686)

 Kernel: Linux 2.6.32-5-486
 Locale: LANG=ru_RU.UTF-8, LC_CTYPE=ru_RU.UTF-8 (charmap=UTF-8) (ignored: 
 LC_ALL set to ru_RU.UTF-8)
 Shell: /bin/sh linked to /bin/dash

 Versions of packages podget depends on:
 ii  bash                    4.1-3            The GNU Bourne Again SHell
 ii  coreutils               8.5-1            GNU core utilities
 ii  grep                    2.6.3-3          GNU grep, egrep and fgrep
 ii  sed                     4.2.1-7          The GNU sed stream editor
 ii  tofrodos                1.7.8.debian.1-2 Converts DOS - Unix text files,
 ii  wget                    1.12-2.1         retrieves files from the web

 podget recommends no packages.

 Versions of packages podget suggests:
 ii  libc6                         2.11.2-10  Embedded GNU C Library: Shared 
 lib

 -- no debconf information






--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org