Bug#337605: gstreamer0.8-misc: Upgrade removes /root/.gstreamer-0.8/registry.xml ?!

2005-11-05 Thread Justin Pryzby
Package: gstreamer0.8-misc
Severity: important
Version: 0.8.8-3
File: /root/.gstreamer-0.8/registry.xml

Preparing to replace gstreamer0.8-misc 0.8.8-3 (using 
.../gstreamer0.8-misc_0.8.11-2_i386.deb) ...
removed `/root/.gstreamer-0.8/registry.xml'
rmdir: removing directory, /root/.gstreamer-0.8
Unpacking replacement gstreamer0.8-misc ...

I can't fathom a legitimate reason to remove anything under /root/,
ever.  I hope that I am I misinterprettign this message, but I just
read the preinst script, and I am not:

if [ -r /root/.gstreamer-0.8/registry.xml ]; then
current=`cat /root/.gstreamer-0.8/registry.xml`
if [ $orig = $current ]; then
rm -f -v /root/.gstreamer-0.8/registry.xml || true
rmdir -v /root/.gstreamer-0.8 || true
fi
fi


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#337605: gstreamer0.8-misc: Upgrade removes /root/.gstreamer-0.8/registry.xml ?!

2005-11-05 Thread Loic Minier
Hi,

On Sat, Nov 05, 2005, Justin Pryzby wrote:
 removed `/root/.gstreamer-0.8/registry.xml'
 rmdir: removing directory, /root/.gstreamer-0.8
 Unpacking replacement gstreamer0.8-misc ...
 I can't fathom a legitimate reason to remove anything under /root/,
 ever.  I hope that I am I misinterprettign this message, but I just
 read the preinst script, and I am not:
 if [ -r /root/.gstreamer-0.8/registry.xml ]; then
 current=`cat /root/.gstreamer-0.8/registry.xml`
 if [ $orig = $current ]; then
 rm -f -v /root/.gstreamer-0.8/registry.xml || true
 rmdir -v /root/.gstreamer-0.8 || true
 fi
 fi

 Previously, gst-register-0.8 was called without the GST_REGISTRY
 environment, and that caused some spurious files to be written to
 /root/.gstreamer-0.8.

 I noticed this bug by cheer luck, and all system ever having a
 gstreamer package installed (be it 0.6 or 0.8) had some spurious
 directories.  Hence, I decided to fix the problem and try removing as
 much cruft as possible, so all packages where the fix appeared saw the
 addition of code to detect whether the file was automatically
 generated.

 As you can see, the script is very cautious about removing a file with
 the exact same contents as one generated by the package, and only
 removes the directory if empty.

 If you think something else should have been done, please suggest what.

   Bye,

-- 
Loïc Minier [EMAIL PROTECTED]
What do we want? BRAINS!When do we want it? BRAINS!



Bug#337605: gstreamer0.8-misc: Upgrade removes /root/.gstreamer-0.8/registry.xml ?!

2005-11-05 Thread Justin Pryzby
severity 337605 minor
thanks

On Sat, Nov 05, 2005 at 11:21:21AM +0100, Loic Minier wrote:
 On Sat, Nov 05, 2005, Justin Pryzby wrote:
  removed `/root/.gstreamer-0.8/registry.xml'
  rmdir: removing directory, /root/.gstreamer-0.8
  Unpacking replacement gstreamer0.8-misc ...
  I can't fathom a legitimate reason to remove anything under /root/,
  ever.  I hope that I am I misinterprettign this message, but I just
  read the preinst script, and I am not:
  if [ -r /root/.gstreamer-0.8/registry.xml ]; then
  current=`cat /root/.gstreamer-0.8/registry.xml`
  if [ $orig = $current ]; then
  rm -f -v /root/.gstreamer-0.8/registry.xml || true
  rmdir -v /root/.gstreamer-0.8 || true
  fi
  fi
 
  Previously, gst-register-0.8 was called without the GST_REGISTRY
  environment, and that caused some spurious files to be written to
  /root/.gstreamer-0.8.
Okay;

  I noticed this bug by cheer luck, and all system ever having a
  gstreamer package installed (be it 0.6 or 0.8) had some spurious
  directories.  Hence, I decided to fix the problem and try removing as
  much cruft as possible, so all packages where the fix appeared saw the
  addition of code to detect whether the file was automatically
  generated.
Okay; why do you use a sh = test rather than diff followed by a check
of $? ?.  (diff Priority: required)

  As you can see, the script is very cautious about removing a file with
  the exact same contents as one generated by the package, and only
  removes the directory if empty.
Actually I didn't even read the conditional part last night.

  If you think something else should have been done, please suggest what.
I agree that this is pretty safe.  Maybe leave the bug open until this
snipped is removed from preinst?  Which I guess is the first upload
after etch release, correct?

But is the problem actually fixed??

  (Reading database ... 174462 files and directories currently installed.)
  Preparing to replace gstreamer0.8-misc 0.8.8-2 (using 
.../gstreamer0.8-misc_0.8.11-2_i386.deb) ...
  removed `/root/.gstreamer-0.8/registry.xml'
  rmdir: removing directory, /root/.gstreamer-0.8
  Unpacking replacement gstreamer0.8-misc ...
  Setting up gstreamer0.8-misc (0.8.11-2) ...
  
  [EMAIL PROTECTED]:~$ ls /root/.g
  .gconf/  .gnome/  .gnome2_private/ 
  .gconfd/ .gnome2/ .gstreamer/  
  [EMAIL PROTECTED]:~$ ls /root/.gstreamer/registry.xml 
  [EMAIL PROTECTED]:~$ ls /root/.gstreamer/registry.xml 
  /root/.gstreamer/registry.xml

-- 
Clear skies,
Justin


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#337605: gstreamer0.8-misc: Upgrade removes /root/.gstreamer-0.8/registry.xml ?!

2005-11-05 Thread Loic Minier
retitle 337605 [post-etch] please remove postinst snipset to cleanup 
/root/.gstreamer-0.8
thanks

Hi,

On Sat, Nov 05, 2005, Justin Pryzby wrote:
 Okay; why do you use a sh = test rather than diff followed by a check
 of $? ?.  (diff Priority: required)

 I didn't think, I don't think this is terribly ugly.

 I agree that this is pretty safe.  Maybe leave the bug open until this
 snipped is removed from preinst?  Which I guess is the first upload
 after etch release, correct?

 Fine for me.

 But is the problem actually fixed??
   [EMAIL PROTECTED]:~$ ls /root/.g
   .gconf/  .gnome/  .gnome2_private/ 
   .gconfd/ .gnome2/ .gstreamer/  

 As I explained, I noticed the problem when I took up the packages, and
 I thought it was too late for gstreamer 0.6 (called gstreamer) which
 was already gone from the archive.  Hence, I only fixed the 0.8 series,
 I didn't want to fix 0.6 stuff in the 0.8 series...

-- 
Loïc Minier [EMAIL PROTECTED]
What do we want? BRAINS!When do we want it? BRAINS!



Bug#337605: gstreamer0.8-misc: Upgrade removes /root/.gstreamer-0.8/registry.xml ?!

2005-11-05 Thread Justin Pryzby
On Sat, Nov 05, 2005 at 06:21:35PM +0100, Loic Minier wrote:
 retitle 337605 [post-etch] please remove postinst snipset to cleanup 
 /root/.gstreamer-0.8
 thanks
 
 Hi,
 
 On Sat, Nov 05, 2005, Justin Pryzby wrote:
  Okay; why do you use a sh = test rather than diff followed by a check
  of $? ?.  (diff Priority: required)
 
  I didn't think, I don't think this is terribly ugly.
Okay.

  I agree that this is pretty safe.  Maybe leave the bug open until this
  snipped is removed from preinst?  Which I guess is the first upload
  after etch release, correct?
 
  Fine for me.
Great.

  But is the problem actually fixed??
[EMAIL PROTECTED]:~$ ls /root/.g
.gconf/  .gnome/  .gnome2_private/ 
.gconfd/ .gnome2/ .gstreamer/  
 
  As I explained, I noticed the problem when I took up the packages, and
  I thought it was too late for gstreamer 0.6 (called gstreamer) which
  was already gone from the archive.  Hence, I only fixed the 0.8 series,
  I didn't want to fix 0.6 stuff in the 0.8 series...
OH okay.

-- 
Clear skies,
Justin


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]