[newbies] Дайджест списка рассылки devel-newbies; том 51, выпуск 6

[Усольцев Пётр]

> > Пачти я формировал по рекомендации на Telegram канале так:
> > git clone git://git.altlinux.org/gears/m/mate-calc.git
> > Вношу правки
> > git commit
> > git format-patch -1
> > далее создадутся патч-файлы
> > Создаю задачу в bagzilla и прилагаю эти патчи.
> > Ну вот обновите ещё и spec по этому же алгориму.
> > Когда отвечаете на дайджест, пожалуйста, обращайте внимание но
> комментарий в дайджесте, процитированный ниже.
>
 
Здравствуйте, спасибо за ответы.
У меня появился ментор Алексей Шабалин.
Прохожу сейчас обучения, для возможности вступить в Team.
Я пока не будут ничего делать по своим старым наработкам, т.к. есть вероятность, что после прохождения, я или все переделаю, или отменю свои патчи, т.к. совсем другой взгляд уже начинает формироваться.
  > > Ищите в багзилле, ну или самостоятельно в ходе эксплуатации.
> > Но баги мало исправить, их важно заапстримить ? чтоб они хотя
> > бы попали в пакет, мейнтенер которого не Вы, а ещё лучше ?
> > в апстрим проекта, если правка не очень уже специфичная для Альта.
> >  
> > - Я так понял, вы говорите, что я должен сам внести правки в пакет.
> > Но для этого нужно иметь права доступа, которые даются, только участникам AltTeam(кем пока я не являюсь)
> > Поэтому я и написал что сделал максимум, который был возможен.
> > Вы непрвильно поняли. Сделать необходимые изменения и закоммитить
> пакет ? это две разные операции. Закоммитить Вы на самом деле пока
> что не можете, но делать это Вас никто и не просил; а вот
> подготовить все нужные изменения в Ваши силах.
> > Просто поставьте себя на место мейнтенера пакета: чем меньше для
> него работы, тем больше вероятность, что он пример изменения.
>
 
Это я понял, но чтобы сделать все правильно — нужно сначала пройти в TEAM, чем сейчас и занят.
  > > Спасибо за конструктивную критику, но вывод из ваших замечаний такой:
> > Нужно было отменить заявку в Багзилле, т.к. корректное ее выполнение привело бы к сложностям дальнейшего обновления.
> > Написать в канал Telegram, чтобы отменили заявку и признали ее ?неоправданной??
> > Зачем закрывать баг в багзилле? Просто доработайте патчи. Несколько
> итераций изначально предложенного патчсета перед тем, как его
> примут ? совершенно нормальная, рутинная для апстримов процедура.
>
 
Понял вас, спасибо.
  > > 1) Изменения API нежелательны, т.к. поддерживать такой патч при
> > Задача стояла таким образом чтобы формат вывода ?руб.? выводился в другом месте.
> > Поэтому у меня было 2 варинта:
> > * Сделать костыль на проверку именно значения ?руб.?
> > * Сделать сохранение формата прямо в валюте, чтобы для каждой валюты можно было указать свой формат
> > Я выбрал вариант ?2, хоть он и требует перестройки API, но таким образом я расширил функционал.
> > Формат вывода валюты у них вообще такой ?%%s%s?, я испоьзовал ?типовой формат?
>
> Вариант Вы выбрали правильно. Но зачем добавлять поле в структуру
> и везде присваивать ему одну и ту же константу, если можно просто
> использовать константу там, где сейчас используется поле структуры?
>  
 
Присваивается не везде одно и тоже, для России и Беларусии значение отличается :)

> > Я уже 10 лет имею опыт программирования, но в языке 1С и поэтому не считаю решение некорректным.
> > Забудьте об 1С. Программирование на C/C++ существенно от него
> отличается не только синтаксисом, но и логикой построения кода.
>
 
Это я знаю, я просто хотел сказать, что я не совсем новичок, что есть опыт, но другой правда.
  > > Единственное, что я не могу отрицать ? что API поменялся.
> > Но корректное решение задачи невозможно без изменения API.(на мой взгляд, но тут я могу ошибаться, т.к. в С новичок)
> > Я выше написал один из возможных способов сохранения API
> и упрощения кода.
>
 
Ясно, спасибо.
 
----------- следующая часть -----------
Вложение в формате HTML было удалено...
URL: <http://lists.altlinux.org/pipermail/devel-newbies/attachments/20220829/2325b202/attachment.html>