From: Andrey Savchenko <bircoph@altlinux.org> To: "devel@ where you _can_ ask" <devel-newbies@lists.altlinux.org> Subject: Re: [newbies] Ищу ментора Date: Fri, 5 Aug 2022 23:12:02 +0300 Message-ID: <20220805231202.2daac16e0ecf9d5dcc0a3a0b@altlinux.org> (raw) In-Reply-To: <1659658417.361514860@f490.i.mail.ru> [-- Attachment #1: Type: text/plain, Size: 3526 bytes --] Добрый вечер! On Fri, 05 Aug 2022 03:13:37 +0300 Усольцев Пётр wrote: > Изучаю C Это хорошо. > Хочу исправлять баги системы, хотя бы мелкие. Ищите в багзилле, ну или самостоятельно в ходе эксплуатации. Но баги мало исправить, их важно заапстримить — чтоб они хотя бы попали в пакет, мейнтенер которого не Вы, а ещё лучше — в апстрим проекта, если правка не очень уже специфичная для Альта. > Мейнтейнер не отправил баги в Сизиф значит(может он уже не > занимается этим). Со своей стороны, что мне было доступно - я все > сделал. Так Вы проявите настойчивость: мейнтенер мог и забыть про баг. Багзилла — не единственный способ связи, можно и e-mail написать. И Ваших патчей мало: нужно ещё spec обновить для их применения, а так Вы перекладываете часть рутинной работы на мейнтенера пакета. Можно подготовить серию патчей и выложить в формате git-format-patch, ну или как договоритесь с мейнтенером. Сами патчи мне не понравились: изменено API структуры, добавлено поле format, и во всех ненулевых значениях там одно и то же "%s%s". Тут сразу ворох проблем: 1) Изменения API нежелательны, т.к. поддерживать такой патч при обновлениях пакета будет тяжело. 2) Т.к. значение одно и то же, то можно реорганизовать код таким образом, что это поле не понадобится, а вместо него будет использована константа. 3) %s — потенциальный источник уязвимостей и переполнений буфера, нужно тщательно проверить, что недоверенных данных там не может оказаться. > Пятница, 5 августа 2022, 5:00 +08:00 от Andrey Savchenko <bircoph@altlinux.org>: > >Добрый вечер! > > > >On Tue, 02 Aug 2022 14:14:44 +0300 Усольцев Пётр wrote: > >> > >> Здравствуйте, ищу ментора. > >> Хочу попасть в Alt Team. > >Чем именно Вы бы хотели заниматься? Какие навыки или опыт имеются? > > > >> Зарегистрирован на портале BugZilla(под пользователем «usoltsev05@mail.ru») > >> и исправил 1 баг https://bugzilla.altlinux.org/42955 > > > >Патчи так и не попали ни в Сизиф, ни в апстрим. Так что часть > >работы сделана, но до конца не доведена. > > > >Best regards, > >Andrew Savchenko > > > Усольцев Пётр. Best regards, Andrew Savchenko [-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]
prev parent reply other threads:[~2022-08-05 20:12 UTC|newest] Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-08-04 21:00 ` Andrey Savchenko 2022-08-05 20:12 ` Andrey Savchenko [this message]
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20220805231202.2daac16e0ecf9d5dcc0a3a0b@altlinux.org \ --to=bircoph@altlinux.org \ --cc=devel-newbies@lists.altlinux.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
devel@ where you _can_ ask This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel-newbies/0 devel-newbies/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel-newbies devel-newbies/ http://lore.altlinux.org/devel-newbies \ devel-newbies@lists.altlinux.org devel-newbies@lists.altlinux.ru devel-newbies@lists.altlinux.com public-inbox-index devel-newbies Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel-newbies AGPL code for this site: git clone https://public-inbox.org/public-inbox.git