| |
| 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
| | |
| |
| 3.6, Аноним (-), 11:39, 03/11/2017 [^] [^^] [^^^] [ответить]
| +/– | |
Ты что, ведь там еще целый огромный хеш 56bef8511a576deef32d3e763b993b5001015c2d и время с точностью до секунды и часового пояса! Как страшно жить, не зная слово автоматизация.
> ЦЕЛЫЙ РЕВЬЮ
т.е., по-вашему, ревью патча, изменяющего пару символов, не нужен?
| | |
| |
| 4.9, нах (?), 11:55, 03/11/2017 [^] [^^] [^^^] [ответить]
| –3 +/– | |
> не зная слово автоматизация.
автоматизация не имеет ни малейшего отношения ни к ревью, ни к sign-off, обе операции делают люди, и они именно для этого и придуманы, чтобы нельзя было автоматически пропихнуть что-то не то. А вот когда то и другое требуется по малейшему чиху - очень вероятно, что "автоматизация" вида "отъ..сь, вот вам подпись" приведет к подписыванию неглядя уже и опасных изменений в коде.
> т.е., по-вашему, ревью патча, изменяющего пару символов, не нужен?
ревью и целых две подписи к патчу, исправляющему банальную опечатку в документации? Вы бредите. Разумеется, не нужен и вреден. Даже если туда слово х^й написать вместо исправления - мир вовсе не рухнет, в следующем патче исправишь...если вообще хоть кто-нибудь заметит.
| | |
| |
| 5.17, Мать драконов (?), 13:15, 03/11/2017 [^] [^^] [^^^] [ответить]
| +1 +/– | |
Порядок - он в головах должен быть. Умный разраб понимает, что могут возникнуть неудобства при работе с пробелами/табами. Особенно когда их меняют.
И если человек, присылающий патч не понимает этого, то он мyдак. И вообще лентяй. И козявка.
| | |
| |
| 6.20, Аноним (-), 13:53, 03/11/2017 [^] [^^] [^^^] [ответить]
| +2 +/– |
О чём вы? Речь про патч, который делает замену одного слова с "have" на "has" и при этом им потребовалось собрать целый коммитет...
| | |
| |
| 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 бы загнулся?
| | |
|
| 3.8, Аноним (-), 11:47, 03/11/2017 [^] [^^] [^^^] [ответить]
| +/– |
Они исправили "Many a flamewar have been fought" на "Many a flamewar has been fought"?
Странные. Я бы исправил на "Many flamewars have been fought"
| | |
| |
| 4.10, нах (?), 11:57, 03/11/2017 [^] [^^] [^^^] [ответить]
| –1 +/– | |
> Странные. Я бы исправил на "Many flamewars have been fought"
пришли им реквест. Они будут страшно рады заняться его изучением, аппрувом и подписыванием, вместо работы.
| | |
| |
| 5.24, Аноним (-), 14:36, 03/11/2017 [^] [^^] [^^^] [ответить]
| –2 +/– | |
Такое ощущение, что это ты один в этом проекте в три лица горбатишься, а они ерундой все занимаются. Это опенсорс, поэтому вместо критики пошёл бы черканул пару строк им в код самостоятельно.
"Как советовать, так все чатлане, как работать..." © Кин-дза-дза
| | |
|
|
|
| 2.19, Андрей (??), 13:21, 03/11/2017 [^] [^^] [^^^] [ответить]
| +/– | |
> Хорошая новость. Лично я не шлю патчи тем, кто требует что-то там подписать.
Ты оставил Столлмана вместе со всем FSF без поддержки. А мы думали, что они уж точно фонд добра.
| | |
| |
| |
| 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).
| | |
| |
| 5.40, Тот_Самый_Анонимус (?), 06:25, 10/11/2017 [^] [^^] [^^^] [ответить]
| –2 +/– | |
Т.е. я был прав, и ФСФ не принимает код под лицензией, которую люто советует другим... Лицемеры, о чём я и говорил.
| | |
|
|
|
|
| 1.5, Аноним (-), 11:34, 03/11/2017 [ответить] [﹢﹢﹢] [ · · · ]
| +2 +/– |
"в то время как платформа GitLab достаточно близка по возможностям" - если присмотреться, то даже опережает.
| | |
| 1.7, Аноним (-), 11:41, 03/11/2017 [ответить] [﹢﹢﹢] [ · · · ]
| +/– |
Открыл для себя gitlab и теперь не могу никак закрыть. Отличная штука. В том числе, используется отдельная инсталляция в компании. Конечно, комьюнити эдишн, ибо всем жалко денег.
| | |
| |
| |
| 3.13, alltiptop (ok), 12:32, 03/11/2017 [^] [^^] [^^^] [ответить]
| +1 +/– |
 Уязвимость 2013 года, недоступность на пару дней за несколько лет - так то надёжнее гитхаба и битбакета выглядит. Это без учёта того что проект открыт и можно развернуть хостинг у себя.
| | |
| 3.14, Аноним (-), 12:32, 03/11/2017 [^] [^^] [^^^] [ответить]
| +2 +/– |
После второго случая у них очень правильные бэкапы с неоднократным дублированием. Они делали разбор полётов, почему все бэкапы на момент факапа были сломаны, и позже описали то, как у них сделано теперь.
| | |
| 3.21, blackst0ne (ok), 13:58, 03/11/2017 [^] [^^] [^^^] [ответить]
| +1 +/– |
 1. Эти ребята сделали правильные выводы.
2. Для суперперестраховки можно развернуть GitLab на контролируемом сервере, а не использовать gitlab.com
| | |
| |
| 4.28, Аноним (-), 16:18, 03/11/2017 [^] [^^] [^^^] [ответить]
| +1 +/– | |
> можно развернуть GitLab на контролируемом сервере, а не использовать gitlab.com
так и сделано, конечно. Не понимаю ссзб хранящих бизнес-данные у дяди
| | |
|
|
|
| |
| 2.29, Аноним (-), 16:20, 03/11/2017 [^] [^^] [^^^] [ответить]
| +/– | |
> Для ниосилевших коньсоль?
Консоль никуда не делась. А удобная работа с merge requests появилась
| | |
| |
| |
| 4.39, Володимир (?), 15:48, 07/11/2017 [^] [^^] [^^^] [ответить]
| +/– |
 > )) одиночкам и гит не нужен, new folder, new folder1, new folder
> old))))
Одиночкам кодревью не нужен, а контроль версий любому нормальному человеку нужен.
| | |
|
|
|
| |
| 2.36, Аноним (-), 19:08, 04/11/2017 [^] [^^] [^^^] [ответить]
| +/– |
Да, Alioth безнадёжно устарел. Как, впрочем, и вообще нехилая часть дебиановской инфраструктуры.
| | |
|
|
