Size: a a a

2021 April 02

c

codingteam@cjr in codingteam
Minoru
> <@phosor> я еще и фикс сделал, и у меня заработало, но он слишком грязный и для опенсорса ящитаю не пойдет, поэтому я просто взял другую, где автора ждать не надо и патчи писать тоже

> он слишком грязный и для опенсорса ящитаю не пойдет

все равно отправь, пусть майнтейнер решает. Если твой патч фиксит известный баг, то наверняка к нему проявят интерес. Если не готов допиливать или поддерживать — так и скажи, но все равно отправь. Потому что ты *уже* сделал что-то ценное, разобравшись в проблеме; очень жаль будет потерять это знание
источник

c

codingteam@cjr in codingteam
Minoru
> <@phosor> про мое отношение с опенсорсом https://youtu.be/77B2-Pk1fls?t=1937

ты тоже написал 14 строк на Си, в которых затем нашли 4 ошибки? Я не понял, как это видео описывает твоё отношение с опенсорцом
источник

AS

Andrey Sorokin in codingteam
codingteam@cjr
Minoru
> <@phosor> я еще и фикс сделал, и у меня заработало, но он слишком грязный и для опенсорса ящитаю не пойдет, поэтому я просто взял другую, где автора ждать не надо и патчи писать тоже

> он слишком грязный и для опенсорса ящитаю не пойдет

все равно отправь, пусть майнтейнер решает. Если твой патч фиксит известный баг, то наверняка к нему проявят интерес. Если не готов допиливать или поддерживать — так и скажи, но все равно отправь. Потому что ты *уже* сделал что-то ценное, разобравшись в проблеме; очень жаль будет потерять это знание
этот код уже есть в стандартной библиотеке и он куда там интересней чем мой, лучше его оттуда копипастить
источник

AS

Andrey Sorokin in codingteam
codingteam@cjr
Minoru
> <@phosor> про мое отношение с опенсорсом https://youtu.be/77B2-Pk1fls?t=1937

ты тоже написал 14 строк на Си, в которых затем нашли 4 ошибки? Я не понял, как это видео описывает твоё отношение с опенсорцом
я тоже делал пул-реквест и меня потом с исправлениями хотели заебать, но я не поддался, но делать пулл реквесты перестал
источник

AS

Andrey Sorokin in codingteam
хорошо что это было не ядро линупса
источник

c

codingteam@cjr in codingteam
Minoru
@phosor: ценность твоего патча не в коде, а в том, что он показывает, в чём именно состоит проблема и как её можно было бы решить. Так что даже если где-то есть код получше, все равно имеет смысл отправить патч, чтобы передать знание о проблеме
источник

O

Omap in codingteam
давайте не будем филосовствовать
источник

AS

Andrey Sorokin in codingteam
codingteam@cjr
Minoru
@phosor: ценность твоего патча не в коде, а в том, что он показывает, в чём именно состоит проблема и как её можно было бы решить. Так что даже если где-то есть код получше, все равно имеет смысл отправить патч, чтобы передать знание о проблеме
чувак сказал, что ему 5 лет никто об этом не писал, и он уже думать забыл про это, отсюда я сделал вывод что это найух никому не нужно и сам не стал слать
источник

t

ttldtor in codingteam
Omap
давайте не будем филосовствовать
всё равно никто не умеет
источник

c

codingteam@cjr in codingteam
Minoru
> <@phosor> чувак сказал, что ему 5 лет никто об этом не писал, и он уже думать забыл про это, отсюда я сделал вывод что это найух никому не нужно и сам не стал слать

то ли никому не нужно, то ли никто из тех, кого задела проблема, не смог её починить. Я думаю, ты зря порешал за автора, что делать с патчем (но я не видел всей дискуссии, может быть, он намекнул, что хочет вылизанное решение, а ты был не готов вносить исправления)
источник

AS

Andrey Sorokin in codingteam
codingteam@cjr
Minoru
@phosor: ценность твоего патча не в коде, а в том, что он показывает, в чём именно состоит проблема и как её можно было бы решить. Так что даже если где-то есть код получше, все равно имеет смысл отправить патч, чтобы передать знание о проблеме
причем там патч это два присвоения по одному условию на каждое, и третья строчка это выбор из двух переменных по условию у меня, когда можно вообще без третьего этого условия сделать, то есть если ты хоть как то стандарт читал ты это сам напишешь
источник

AS

Andrey Sorokin in codingteam
только это желательно еще все в цикл обернуть для правильности, а это уже есть в стандартной либе, причем гораздо лучше и правильней, в общем от моего говнокода только хуже бы стало
источник

AS

Andrey Sorokin in codingteam
такое даже показывать стыдно как патч какой то
источник

c

codingteam@cjr in codingteam
Minoru
я же говорю, в этой ситуации нужно акцентироваться не на коде, а на твоих находках, на новых фактах о проблеме
источник

c

codingteam@cjr in codingteam
Minoru
а свой патч можешь показывать как POC fix
источник

💮

💮 in codingteam
ttldtor
всё равно никто не умеет
Если «никто не умеет», то значит определение «умеет» не откалибровано. Надо нормировать планку так чтобы умело 50% людей.
источник

AO

Alibek a1batross Oma... in codingteam
Я в таких случаях иногда доделываю патч за контрибутера
источник

c

codingteam@cjr in codingteam
Minoru
и прямо рядом написать, что такой же код, но нормальный, лучше взять из стандартной либы
источник

AO

Alibek a1batross Oma... in codingteam
Это даже недолго, как бы просто причесать и всё
источник

AO

Alibek a1batross Oma... in codingteam
Сложно конечно когда речь идёт о тысячах строк...
источник