| |
| 2.4, anonymous (??), 11:29, 03/11/2017 [^] [^^] [^^^] [ответить] [↓] [↑] [п©б╘п▒Б┬≥Б∙≈ п©б╘п▒Б┬≥Б∙≥п©б╘п▒Б┬≥Б∙⌡п©б╘п▒Б┬≥Б∙▓п©б╘п▒Б┬≥я▒я▐Б√░п▒Б√═Б■─п©б╘п▒Б┬≥ц╥я▐Б√░п▒Б√═Б√└п©б╘п▒Б┬≥Б∙⌡я▐Б√░п▒Б√═Б■─я▐Б√░п▒Б√═Б√▒]
| –1 +/– | |
С этой бюрократией люди порой начинают производить идиократию. Вот коммит из QEMU:
# git log CODING_STYLE
commit 56bef8511a576deef32d3e763b993b5001015c2d
Author: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
Date: Fri Sep 30 02:04:28 2016 +0200
CODING_STYLE: Fix a typo ("have" vs. "has")
Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
И вы не поверите, что это за серьёзный патч, котоый имеет аж ДВЕ подписи и ЦЕЛЫЙ РЕВЬЮ:
diff --git a/CODING_STYLE b/CODING_STYLE
index e7fde15..f53180b 100644
--- a/CODING_STYLE
+++ b/CODING_STYLE
@@ -9,7 +9,7 @@ patches before submitting.
Of course, the most important aspect in any coding style is whitespace.
Crusty old coders who have trouble spotting the glasses on their noses
can tell the difference between a tab and eight spaces from a distance
-of approximately fifteen parsecs. Many a flamewar have been fought and
+of approximately fifteen parsecs. Many a flamewar has been fought and
lost on this issue.
QEMU indents are four spaces. Tabs are never used, except in Makefiles
| | |
| |
| |
| 4.9, нах (?), 11:55, 03/11/2017 [^] [^^] [^^^] [ответить] [↓] [п©б╘п▒Б┬≥Б∙≈ п©б╘п▒Б┬≥Б∙≥п©б╘п▒Б┬≥Б∙⌡п©б╘п▒Б┬≥Б∙▓п©б╘п▒Б┬≥я▒я▐Б√░п▒Б√═Б■─п©б╘п▒Б┬≥ц╥я▐Б√░п▒Б√═Б√└п©б╘п▒Б┬≥Б∙⌡я▐Б√░п▒Б√═Б■─я▐Б√░п▒Б√═Б√▒]
| –3 +/– | |
> не зная слово автоматизация.
автоматизация не имеет ни малейшего отношения ни к ревью, ни к sign-off, обе операции делают люди, и они именно для этого и придуманы, чтобы нельзя было автоматически пропихнуть что-то не то. А вот когда то и другое требуется по малейшему чиху - очень вероятно, что "автоматизация" вида "отъ..сь, вот вам подпись" приведет к подписыванию неглядя уже и опасных изменений в коде.
> т.е., по-вашему, ревью патча, изменяющего пару символов, не нужен?
ревью и целых две подписи к патчу, исправляющему банальную опечатку в документации? Вы бредите. Разумеется, не нужен и вреден. Даже если туда слово х^й написать вместо исправления - мир вовсе не рухнет, в следующем патче исправишь...если вообще хоть кто-нибудь заметит.
| | |
| |
| |
| |
| 7.26, нах (?), 14:57, 03/11/2017 [^] [^^] [^^^] [ответить] [↑] [п©б╘п▒Б┬≥Б∙≈ п©б╘п▒Б┬≥Б∙≥п©б╘п▒Б┬≥Б∙⌡п©б╘п▒Б┬≥Б∙▓п©б╘п▒Б┬≥я▒я▐Б√░п▒Б√═Б■─п©б╘п▒Б┬≥ц╥я▐Б√░п▒Б√═Б√└п©б╘п▒Б┬≥Б∙⌡я▐Б√░п▒Б√═Б■─я▐Б√░п▒Б√═Б√▒]
| –1 +/– | |
> О чём вы?
это вам была прекрасная иллюстрация на тему "тысячи глаз", и, кстати, в применении к code review и подписыванию коммитов оно работает точно так же - то есть, никак. Люди не в состоянии читать буквы и соображать головой. Чем меньше ваша разработка зависит от их умения и желания это делать, тем, на самом деле, только лучше.
| | |
|
|
| 5.31, Аноним (-), 18:27, 03/11/2017 [^] [^^] [^^^] [ответить] [↑] [п©б╘п▒Б┬≥Б∙≈ п©б╘п▒Б┬≥Б∙≥п©б╘п▒Б┬≥Б∙⌡п©б╘п▒Б┬≥Б∙▓п©б╘п▒Б┬≥я▒я▐Б√░п▒Б√═Б■─п©б╘п▒Б┬≥ц╥я▐Б√░п▒Б√═Б√└п©б╘п▒Б┬≥Б∙⌡я▐Б√░п▒Б√═Б■─я▐Б√░п▒Б√═Б√▒]
| +2 +/– | |
> автоматизация не имеет ни малейшего отношения ни к ревью, ни к sign-off, обе операции делают люди, и они именно для этого и придуманы, чтобы нельзя было автоматически пропихнуть что-то не то. А вот когда то и другое требуется по малейшему чиху - очень вероятно, что "автоматизация" вида "отъ..сь, вот вам подпись" приведет к подписыванию неглядя уже и опасных изменений в коде.
Видимо разработчики Qemu предпочитают следовать одному заведённому порядку, а не писать длинные списки исключений, когда порядок можно изменить.
| | |
|
| 4.22, Аноним (-), 14:04, 03/11/2017 [^] [^^] [^^^] [ответить] [↑] [п©б╘п▒Б┬≥Б∙≈ п©б╘п▒Б┬≥Б∙≥п©б╘п▒Б┬≥Б∙⌡п©б╘п▒Б┬≥Б∙▓п©б╘п▒Б┬≥я▒я▐Б√░п▒Б√═Б■─п©б╘п▒Б┬≥ц╥я▐Б√░п▒Б√═Б√└п©б╘п▒Б┬≥Б∙⌡я▐Б√░п▒Б√═Б■─я▐Б√░п▒Б√═Б√▒]
| +/– | |
> Ты что, ведь там еще целый огромный хеш 56bef8511a576deef32d3e763b993b5001015c2d и время с точностью до секунды и часового пояса! Как страшно жить, не зная слово автоматизация.
О чём вы? Люди целенаправлено лично вручную ставили свои подписи на изменение в одном предложении с "have" на "has".
> т.е., по-вашему, ревью патча, изменяющего пару символов, не нужен?
Ревью патча + две подписи на файл CODING_STYLE, где банально изменяется опечатка? Нет не нужен.
Или вы считаете, что если бы приняли такой патч, где банально исправляется "have" на "has" в текстовом документе (то есть это даже не код), без подписи, то человек бы подом отсудил себе миллионы и бедный QEMU бы загнулся?
| | |
|
|
| |
| |
| 4.37, Андрей (??), 04:02, 05/11/2017 [^] [^^] [^^^] [ответить] [п©б╘п▒Б┬≥Б∙≈ п©б╘п▒Б┬≥Б∙≥п©б╘п▒Б┬≥Б∙⌡п©б╘п▒Б┬≥Б∙▓п©б╘п▒Б┬≥я▒я▐Б√░п▒Б√═Б■─п©б╘п▒Б┬≥ц╥я▐Б√░п▒Б√═Б√└п©б╘п▒Б┬≥Б∙⌡я▐Б√░п▒Б√═Б■─я▐Б√░п▒Б√═Б√▒]
| –1 +/– | |
Если это FSF-проект (gcc, gnulib, emacs,..), то это обязательно:
Before incorporating significant changes, make sure that the person who wrote the changes has signed copyright papers and that the Free Software Foundation has received and signed them. We may also need an employer’s disclaimer from the person’s employer, which confirms that the work was not part of the person’s job and the employer makes no claim on it. However, a copy of the person’s employment contract, showing that the employer can’t claim any rights to this work, is often sufficient.
Т.е. иногда нужно даже ещё и у работодателя получать разрешение. Что дополнительно тормозит развитие FSF-проектов. Но зато они смогут отстаивать свою лицензию в суде.
Единственная альтернатива для личного кода - это передать свой патч в общественное достояние (public domain).
| | |
|
|
|
|
