[devel] Q: task approval

[devel] Q: task approval

[Konstantin Lepikhov]

Привет!

А что поменялось в approval что теперь оно не работает ожидаемым образом?

$ ssh build.alt task approve --help
task approve - approve a subtask of a task

Usage: task approve [--revoke] <task id> <subtask number>

$ ssh build.alt task approve <...> <...>
task approve: Go ahead and type the text of approval

.
.

И все, могу только прервать этот процесс по Ctrl-C.

-- 
WBR et al.

Re: [devel] Q: task approval

[Dmitry V. Levin]

On Wed, Feb 12, 2020 at 10:50:38PM +0100, Konstantin Lepikhov wrote:
> Привет!
> 
> А что поменялось в approval что теперь оно не работает ожидаемым образом?
> 
> $ ssh build.alt task approve --help
> task approve - approve a subtask of a task
> 
> Usage: task approve [--revoke] <task id> <subtask number>
> 
> $ ssh build.alt task approve <...> <...>
> task approve: Go ahead and type the text of approval
> 
> .
> .
> 
> И все, могу только прервать этот процесс по Ctrl-C.

Оно просто читает stdin и ждёт конца файла.


-- 
ldv

Re: [devel] Q: task approval

[Grigory Ustinov]

13.02.2020 1:16, Dmitry V. Levin пишет:
> On Wed, Feb 12, 2020 at 10:50:38PM +0100, Konstantin Lepikhov wrote:
>> Привет!
>>
>> А что поменялось в approval что теперь оно не работает ожидаемым образом?
>>
>> $ ssh build.alt task approve --help
>> task approve - approve a subtask of a task
>>
>> Usage: task approve [--revoke] <task id> <subtask number>
>>
>> $ ssh build.alt task approve <...> <...>
>> task approve: Go ahead and type the text of approval
>>
>> .
>> .
>>
>> И все, могу только прервать этот процесс по Ctrl-C.
> Оно просто читает stdin и ждёт конца файла.
Ага и ещё она не схавает пробел и две точки. Из надёжного источника 
доподлинно известно, что лучшая фраза - это glfm. Она достаточно 
универсальная, поэтому её можно даже не оборачивать в алиас и в то же 
время не вызывает недоумения у людей, которым важно читать эти аппрувы.

Замечу также: там написано по-английски "напечатайте текст одобрения". 
Не причину, а текст. А текст может быть любым. В принципе, я склонен 
перейти к "ok" - это тоже текст, в отличие от " ..". Ну просто потому 
что я всегда очень детально рассматриваю те таски, которые аппрувлю. А 
ещё я думаю, что маленький ацл ставится людьми исключительно в 
ситуациях, когда пакет важен для них и в таких случаях эти людям было бы 
нелогично выдавать аппрув не глядя.

Re: [devel] Q: task approval

[Konstantin Lepikhov]

Hi Dmitry!

On 02/13/2020, at 01:16:10 AM you wrote:

> On Wed, Feb 12, 2020 at 10:50:38PM +0100, Konstantin Lepikhov wrote:
> > Привет!
> > 
> > А что поменялось в approval что теперь оно не работает ожидаемым образом?
> > 
> > $ ssh build.alt task approve --help
> > task approve - approve a subtask of a task
> > 
> > Usage: task approve [--revoke] <task id> <subtask number>
> > 
> > $ ssh build.alt task approve <...> <...>
> > task approve: Go ahead and type the text of approval
> > 
> > .
> > .
> > 
> > И все, могу только прервать этот процесс по Ctrl-C.
> 
> Оно просто читает stdin и ждёт конца файла.
Наверное, стоит это отразить в документации, т.к. такое поведение не
очевидно.

-- 
WBR et al.