Control: tag -1 - patch
Hi,
while the patch indeed fixes all my current cases, it still doesn't
seem the correct fix:
Axel Beckert wrote:
> So the main issue is that aptitude is connected to some terminal, wants
> to get rid of some output, but there is nothing which takes the
> output.
>
> I'm not yet 100% sure about the correct fix, mostly because I still have
> no idea which change made this issue appear.
>
> Besides the manual workaround mentioned above, I've found two other
> changes which cause the issue to vanish in the example above:
>
> 1) If I remove the "yes '' |" the issue is gone.
>
> But the "yes" is necessary for the cases which are not covered by
> --force-conf{def,old} e.g. first installs with config files already
> present. See commit 169ee18d77a6a80248bdbd1d95cf626638219cb5 and the
> changelog entry for 1.2.15-1.
>
> 2) If I add a "< /dev/null" behind the aptitude call:
>
> yes '' | aptitude install -y < /dev/null
>
> The "< /dev/null" was actually present in aptitude-robot-session
> until the "yes '' |" was added, but it seems that nowadays both are
> necessary to avoid the drawbacks of (1) mentioned above.
>
> So I assume that the following patch will fix the issue:
Despite it may not look so, that patch is effectively just a revert of
169ee18d77a6a80248bdbd1d95cf626638219cb5 and hence only replaces one
issue with another, because
→ echo bar > /tmp/bar.txt
→ bash -c 'echo foo | cat -v < /tmp/bar.txt'
bar
→ dash -c 'echo foo | cat -v < /tmp/bar.txt'
bar
i.e. the initial "yes '' |" in the patch would be ignored.
Need to dig deeper...
Regards, Axel
--
,''`. | Axel Beckert <a...@debian.org>, http://people.debian.org/~abe/
: :' : | Debian Developer, ftp.ch.debian.org Admin
`. `' | 1024D: F067 EA27 26B9 C3FC 1486 202E C09E 1D89 9593 0EDE
`- | 4096R: 2517 B724 C5F6 CA99 5329 6E61 2FF9 CD59 6126 16B5
--
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
