Bug#117436: testing on squeeze

2011-08-14 Thread Gergely Nagy
I just tried to reproduce this on my squeeze system, with a freshly
built gramofile, and in all terminal emulators I tried (xterm,
gnome-terminal  konsole), it behaved correctly:

I started gramofile, it's menu appeared, I pressed Q, it exited, and
cleared the screen, giving me a clean prompt back.

However, I could reproduce the problem on the console, where it left a
few things on the screen afterwards.

-- 
|8]



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



Bug#117436: testing on squeeze

2011-08-14 Thread Gergely Nagy
tags 117436 + patch
thanks

Actually, I could reproduce this on the console, to some extent: the
gramofile screen remained there, but above my prompt.

Nevertheless, the attached patch (to be applied on top of the one in
#636885) modifies gramofile to clear the screen anyway. Tested the
result on the text console, in various terminal emulators, and it
behaved as it should have: cleared the screen when exiting.

-- 
|8]

From 4a5fbe7e77e30437865f608fdc0df8bb231488ad Mon Sep 17 00:00:00 2001
From: Gergely Nagy alger...@madhouse-project.org
Date: Sun, 14 Aug 2011 14:34:12 +0200
Subject: [PATCH] debian: Add a new patch to clear the screen.

---
 debian/changelog|5 -
 debian/patches/92-clearscreen.patch |   17 +
 debian/patches/series   |1 +
 3 files changed, 22 insertions(+), 1 deletions(-)
 create mode 100644 debian/patches/92-clearscreen.patch

diff --git a/debian/changelog b/debian/changelog
index 1181fe7..b06c561 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -5,8 +5,11 @@ gramofile (1.6-8.1) unstable; urgency=low
   + This resulted in converting to Format: 3.0 (quilt).
   + Standards-Version bumped to 3.9.2.
   + All patches quilt refreshed to apply cleanly.
+  * New patches:
+  + 92-clearscreen.patch: Explicitly clear the screen after exiting
+gramofile (Closes: #117436).
 
- -- Gergely Nagy alger...@madhouse-project.org  Sun, 14 Aug 2011 13:18:37 +0200
+ -- Gergely Nagy alger...@madhouse-project.org  Sun, 14 Aug 2011 14:31:21 +0200
 
 gramofile (1.6-8) unstable; urgency=medium
 
diff --git a/debian/patches/92-clearscreen.patch b/debian/patches/92-clearscreen.patch
new file mode 100644
index 000..02427ca
--- /dev/null
+++ b/debian/patches/92-clearscreen.patch
@@ -0,0 +1,17 @@
+Description: Explicitly clear the screen after exiting gramofile.
+ Explicitly call clear() and refresh() in finishmenu(), so that the
+ screen gets properly cleared.
+Author: Gergely Nagy alger...@madhouse-project.org
+Bug-Debian: http://bugs.debian.org/117436
+
+--- gramofile-1.6.orig/gramofile.c
 gramofile-1.6/gramofile.c
+@@ -38,6 +38,8 @@ init_curses (void)
+ static void
+ finishmenu (int sig)
+ {
++  clear ();
++  refresh ();
+   endwin ();
+   exit (0);
+ }
diff --git a/debian/patches/series b/debian/patches/series
index 6ddf9c3..8312ad4 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -12,3 +12,4 @@
 80_fix_wav_length.patch
 90_report_recording_errors.patch
 91_rename_cdrecord_wodim.patch
+92-clearscreen.patch
-- 
1.7.2.5