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