Control: severity -1 important
Control: tag -1 + confirmed - patch
Hi Ian,
thanks for forwarding this issue.
I meant to reply quite a while ago, but in the end seem to have
forgotton to actually send it. Most of it was written already in the
beginning of December.
Ian Jackson wrote:
> Steps to reproduce and observed behaviour:
>
> $ cat t.sh
> #!/bin/sh
> echo $#
> sleep 5
> $ ./t.sh {0..999}
> 1000
> $ screen ./t.sh {0..999}
> [ displays 1000 and exits ]
> $ screen screen ./t.sh {0..999}
> [ displays 62 and exits ]
> $
>
> Expected behaviour: 1000 in all cases.
So this issue basically solely exists at startup. Or possibly if
someone types in more than 62 parameters manually at "Ctrl-A :screen"
(which I can't imagine that it really happens).
> This is a potential data loss bug so might justify even "grave" but I
> thought that was overkill. I hope you will agree that this deserves
> an RC bug.
I thought about that a while (but didn't come to a conclusion quickly,
so I only started a few weeks after the bug report), but while I agree
that this is "data loss", the actual impact of this issue seems so
small that it IMHO does not deserve RC status.
I'm also not really happy with the attached patch (from your second
mail) and still undecided if I should really apply a patch that
deliberately causes a crash.
Then again, it might just be poorly named as it does not seem to cause
a crash (like a segmentation fault), just exiting in panic.
I'm removing the patch tag as the patch doesn't fix the issue but just
makes it more visible.
> This has been reported upstream here:
> https://savannah.gnu.org/bugs/?61504
Marked as forwarded, thanks. (Already did that back then after I saw
the bug report.)
Also note that upstream marked the bug report in Savannah as confirmed
as well. So there's hope that upstream finds a proper patch for it.
Regards, Axel
--
,''`. | Axel Beckert <a...@debian.org>, https://people.debian.org/~abe/
: :' : | Debian Developer, ftp.ch.debian.org Admin
`. `' | 4096R: 2517 B724 C5F6 CA99 5329 6E61 2FF9 CD59 6126 16B5
`- | 1024D: F067 EA27 26B9 C3FC 1486 202E C09E 1D89 9593 0EDE
