Bug#709325: Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-30 Thread Sven Joachim
Control: severity -1 normal

On 2013-05-22 21:26 +0200, Sven Joachim wrote:

> On 2013-05-22 16:40 +0200, Sven Joachim wrote:
>
>> On 2013-05-22 12:02 +0200, Thomas Dickey wrote:
>>
>>> Per my previous comment, I would have expected this to mark in some
>>> way a blocking-bug to prevent ncurses to propagate until this issue
>>> is resolved.
>>
>> Indeed, doing that now.  Hopefully it's not too late.
>
> Unfortunately it _was_ too late, ncurses migrated to testing today so
> this problem will affect testing users until dialog is fixed. :-(

Meanwhile a fixed dialog has migrated to testing, so I think the
severity should be downgraded.  Probably I'll ad a versioned Breaks
against dialog to libtinfo5 so that users don't experience problems
after partial upgrades.

Sven


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



Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-24 Thread Thomas Dickey
On Fri, May 24, 2013 at 10:14:02AM +0200, Santiago Vila wrote:
> On Wed, 22 May 2013, Sven Joachim wrote:
> > Which does not help, because the dialog version in testing has the same
> > problem with the new ncurses.
> 
> I was obviously confused, sorry.
> 
> Anyway, there is now a new dialog by Thomas which I'll package asap.

sounds good.  If I'd noticed that ncurses was near testing, I'd have
done this fix a week ago...

-- 
Thomas E. Dickey 
http://invisible-island.net
ftp://invisible-island.net


signature.asc
Description: Digital signature


Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-24 Thread Santiago Vila
On Wed, 22 May 2013, Sven Joachim wrote:

> Control: clone -1 -2
> Control: reassign -2 libncursesw5 5.9+20130504-1 
> Control: block -2 by -1
> 
> On 2013-05-22 12:02 +0200, Thomas Dickey wrote:
> 
> > On Wed, May 22, 2013 at 11:16:16AM +0200, Santiago Vila wrote:
> >> severity 708829 serious
> >> thanks
> >> 
> >> I don't know how many users will be affected by this,
> 
> A quick search[1] showed that at least libucimf, ssft and
> grml-debootstrap are affected in Debian.
> 
> >> so just to be
> >> safe, I'm going to mark this as "serious", only to prevent this to
> >> propagate to testing.
> 
> Which does not help, because the dialog version in testing has the same
> problem with the new ncurses.

I was obviously confused, sorry.

Anyway, there is now a new dialog by Thomas which I'll package asap.


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



Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-23 Thread Thomas Dickey
On Wed, May 22, 2013 at 09:26:53PM +0200, Sven Joachim wrote:
> On 2013-05-22 16:40 +0200, Sven Joachim wrote:
> 
> > On 2013-05-22 12:02 +0200, Thomas Dickey wrote:
> >
> >> Per my previous comment, I would have expected this to mark in some
> >> way a blocking-bug to prevent ncurses to propagate until this issue
> >> is resolved.
> >
> > Indeed, doing that now.  Hopefully it's not too late.
> 
> Unfortunately it _was_ too late, ncurses migrated to testing today so
> this problem will affect testing users until dialog is fixed. :-(

The problem is the calls to putp in util.c, which always write to stdout.

Before my changes, putp had (incorrectly for some time) written using ncurses's
own file descriptor passed in from newterm.  I noticed/fixed that while ironing
out the changes related to 20120825 

I'll fix this in dialog by ensuring that it uses the same file descriptor
as that used for newterm.

-- 
Thomas E. Dickey 
http://invisible-island.net
ftp://invisible-island.net


signature.asc
Description: Digital signature


Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-22 Thread Thomas Dickey
On Wed, May 22, 2013 at 09:26:53PM +0200, Sven Joachim wrote:
> On 2013-05-22 16:40 +0200, Sven Joachim wrote:
> 
> > On 2013-05-22 12:02 +0200, Thomas Dickey wrote:
> >
> >> Per my previous comment, I would have expected this to mark in some
> >> way a blocking-bug to prevent ncurses to propagate until this issue
> >> is resolved.
> >
> > Indeed, doing that now.  Hopefully it's not too late.
> 
> Unfortunately it _was_ too late, ncurses migrated to testing today so
> this problem will affect testing users until dialog is fixed. :-(

yes... it hadn't occurred to me that ncurses was ready to go (I've been
juggling a lot of things recently).  I'll put together a fix for this
one tonight/tomorrow.

-- 
Thomas E. Dickey 
http://invisible-island.net
ftp://invisible-island.net


signature.asc
Description: Digital signature


Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-22 Thread Sven Joachim
On 2013-05-22 16:40 +0200, Sven Joachim wrote:

> On 2013-05-22 12:02 +0200, Thomas Dickey wrote:
>
>> Per my previous comment, I would have expected this to mark in some
>> way a blocking-bug to prevent ncurses to propagate until this issue
>> is resolved.
>
> Indeed, doing that now.  Hopefully it's not too late.

Unfortunately it _was_ too late, ncurses migrated to testing today so
this problem will affect testing users until dialog is fixed. :-(

Cheers,
   Sven


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



Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-22 Thread Sven Joachim
Control: clone -1 -2
Control: reassign -2 libncursesw5 5.9+20130504-1 
Control: block -2 by -1

On 2013-05-22 12:02 +0200, Thomas Dickey wrote:

> On Wed, May 22, 2013 at 11:16:16AM +0200, Santiago Vila wrote:
>> severity 708829 serious
>> thanks
>> 
>> I don't know how many users will be affected by this,

A quick search[1] showed that at least libucimf, ssft and
grml-debootstrap are affected in Debian.

>> so just to be
>> safe, I'm going to mark this as "serious", only to prevent this to
>> propagate to testing.

Which does not help, because the dialog version in testing has the same
problem with the new ncurses.

> Per my previous comment, I would have expected this to mark in some
> way a blocking-bug to prevent ncurses to propagate until this issue
> is resolved.

Indeed, doing that now.  Hopefully it's not too late.

Cheers,
   Sven


1. http://codesearch.debian.net/search?q=dialog+--stdout


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



Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-22 Thread Thomas Dickey
On Wed, May 22, 2013 at 11:16:16AM +0200, Santiago Vila wrote:
> severity 708829 serious
> thanks
> 
> I don't know how many users will be affected by this, so just to be
> safe, I'm going to mark this as "serious", only to prevent this to
> propagate to testing.

Per my previous comment, I would have expected this to mark in some
way a blocking-bug to prevent ncurses to propagate until this issue
is resolved.

-- 
Thomas E. Dickey 
http://invisible-island.net
ftp://invisible-island.net


signature.asc
Description: Digital signature


Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-22 Thread Santiago Vila
severity 708829 serious
thanks

I don't know how many users will be affected by this, so just to be
safe, I'm going to mark this as "serious", only to prevent this to
propagate to testing.

Thanks.


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



Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-18 Thread Thomas Dickey
On Sat, May 18, 2013 at 11:03:52PM +0200, Vaclav Ovsik wrote:
> Package: dialog
> Version: 1.2-20121230-1
> Severity: normal
> 
> Dear Maintainer,
> after ncurses packages upgraded to version 5.9+20130504-1 my shell
> script with dialogs stopped working. After some investigation I found,
> that dialog sends terminal control sequence for maybe clearing terminal
> to stdout. Consider:
> 
> zito@bobek:~$ dialog --stdout --menu 'my menu' 15 60 0 first "1st menu item" 
> second "2nd menu item" >/tmp/x
> zito@bobek:~$ xxd /tmp/x
> 000: 1b5b 324a 1b5b 3f34 376c 1b38 1b5b 481b  .[2J.[?47l.8.[H.
> 010: 5b32 4a66 6972 7374  [2Jfirst
> 
> Normal behaviour is back when ncurses packages downgraded to 5.9-10
> version. Despite this fact I reported this problem against dialog,
> I suspect dialog maybe uses something from ncurses incorrectly and this
> problem occurred now with some change in ncurses.

yes - I changed something in ncurses, starting here:

20120825
+ change output buffering scheme, using buffer maintained by ncurses
  rather than stdio, to avoid problems with SIGTSTP handling (report
  by Brian Bloniarz).

(the fix for this problem will be to add an fflush in dialog to work as
intended)

-- 
Thomas E. Dickey 
http://invisible-island.net
ftp://invisible-island.net


signature.asc
Description: Digital signature


Bug#708829: dialog broken by ncurses 5.9+20130504-1, with --stdout sends term controls to stdout

2013-05-18 Thread Vaclav Ovsik
Package: dialog
Version: 1.2-20121230-1
Severity: normal

Dear Maintainer,
after ncurses packages upgraded to version 5.9+20130504-1 my shell
script with dialogs stopped working. After some investigation I found,
that dialog sends terminal control sequence for maybe clearing terminal
to stdout. Consider:

zito@bobek:~$ dialog --stdout --menu 'my menu' 15 60 0 first "1st menu item" 
second "2nd menu item" >/tmp/x
zito@bobek:~$ xxd /tmp/x
000: 1b5b 324a 1b5b 3f34 376c 1b38 1b5b 481b  .[2J.[?47l.8.[H.
010: 5b32 4a66 6972 7374  [2Jfirst

Normal behaviour is back when ncurses packages downgraded to 5.9-10
version. Despite this fact I reported this problem against dialog,
I suspect dialog maybe uses something from ncurses incorrectly and this
problem occurred now with some change in ncurses.
And maybe I'm wrong :).
Thanks for your time
-- 
Zito

-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.8-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=cs_CZ.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages dialog depends on:
ii  debianutils   4.3.4
ii  libc6 2.17-3
ii  libncursesw5  5.9+20130504-1
ii  libtinfo5 5.9+20130504-1

dialog recommends no packages.

dialog suggests no packages.

-- 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