Control: tags -1 pending

Hey Manphiz,

Manphiz writes:

> Hi Amin,
>
> Amin Bandali <band...@gnu.org> writes:
>
>> Hiya,
>>
>> Manphiz writes:
>>
>>> Manphiz <manp...@gmail.com> writes:
>>>
>>>> Another finding is that in 28.x, if the term buffer have any further
>>>> questions to ask, debian-bug seems to consider the process stuck and
>>>> would just ignore everything and proceed.  In 29.x however, the term
>>>> buffer seems to be able to accept user input and can process the output
>>>> accordingly - even if the script requires sudo and prompt for password,
>>>> and debian-bug can properly include the output in the generated email
>>>> for bug report.  So with the merge request[4] it would instead skip all
>>>> potential additional information unfortunately.
>>>>
>>>
>>> Actually 28.x also works for user inputs if running term-exec without my
>>> problematic hooks so yeah!
>>>
>>>> As we do want to handle process termination better, while trying to keep
>>>> process from failing, I think temporarily disable term-exec-hook when
>>>> processing the output and restore after the report is generated should
>>>> probably work in most cases.  Just wondering whether this is acceptable
>>>> in the process of debian-bug?
>>>>
>>>
>>> Forgot to mention that this is implemented as the 2nd commit in the
>>> MR[4] and tested on bookworm and trixie to be working.
>>>
>>>> [4] https://salsa.debian.org/emacsen-team/debian-el/-/merge_requests/11
>>
>> Thank you for the patch. :-)  This seems like a reasonable fix to me,
>> and if David has no comments or objections I'd be happy to merge it
>> in the coming days.
>>
>> Quick request: would you please amend your patch/MR to add a
>> debian/changelog entry for the change(s)?  It would be good to
>> do a new upload soon, and it'd be nice to have debian/changelog
>> in tiptop shape for that.
>>
>> Thanks,
>> -a
>
> Thanks for the review!  I've pushed another commit adding the changelog
> entries, and changed some of the commit message to be consistent, so
> please force pull if you want to test again.  Would definitely look
> forward to hearing from David as well.

Cheers; and thanks, I've merged the MR now!

> P.S. I'm also testing some patches to fix the comp warnings and
> hopefully can be included in the next upload.  Will send another merge
> request when ready.

ACK, I'll rebase and review that one as well.

Thanks,
-a

Attachment: signature.asc
Description: PGP signature

Reply via email to