[devel] apt: принять фиксы

[devel] apt: принять фиксы

[Slava Semushin]

Здравствуйте!

Сделал исправления для нескольких простеньких багов в  apt-shell:
- #5953 (автодополнение для команды list)
- #18343 (выход из apt-shell поCtrl+D)
- #18256 (добавлено описание опций -g/-G в вывод help list)

Всё это доступно в гите:
http://git.altlinux.org/people/php_coder/packages/apt.git

Собственно, чего хочу-то?
- проверить всё это (я проверял, у меня работает)
- проверить русский/английский перевод для #18256
- возможно, что кто-то может/поможет добавить перевод на беларусский
(правильно написал?) язык (опять же см. #18256)

И главное: хочется услышать от ldv@/at@ ответ на вопрос: смержат ли
они все эти изменения или мне можно/нужно сделать NMU? (предварительно
смержив поправки от ldv@ к моим коммитам)

Всех с наступающим!


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] apt: принять фиксы

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 1017 bytes --]

On Wed, Dec 31, 2008 at 08:45:30PM +0600, Slava Semushin wrote:
> Здравствуйте!
> 
> Сделал исправления для нескольких простеньких багов в  apt-shell:
> - #5953 (автодополнение для команды list)
> - #18343 (выход из apt-shell поCtrl+D)
> - #18256 (добавлено описание опций -g/-G в вывод help list)
> 
> Всё это доступно в гите:
> http://git.altlinux.org/people/php_coder/packages/apt.git

Отправляй на сборку, чево людям голову морочить.

> Собственно, чего хочу-то?
> - проверить всё это (я проверял, у меня работает)
> - проверить русский/английский перевод для #18256
> - возможно, что кто-то может/поможет добавить перевод на беларусский
> (правильно написал?) язык (опять же см. #18256)

Вроде правильно хотя глаза на лоб лезут.
Великия и малыя и белыя и прочая и прочая.

> И главное: хочется услышать от ldv@/at@ ответ на вопрос: смержат ли
> они все эти изменения или мне можно/нужно сделать NMU? (предварительно
> смержив поправки от ldv@ к моим коммитам)
> 
> Всех с наступающим!

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] apt: принять фиксы

[Slava Semushin]

1 января 2009 г. 2:54 пользователь Alexey Tourbin <at@altlinux> написал:
[...]
>> Всё это доступно в гите:
>> http://git.altlinux.org/people/php_coder/packages/apt.git
>
> Отправляй на сборку, чево людям голову морочить.

Во-первых, потому что я не мэйнтейнер apt-а.
Во-вторых, хотелось бы чтобы мои изменения по-review-или.
В-третьих, тут часто говорят о том что не плохо было бы делать анонсы
и сообщать кто чем занимается.

Готов залить сборку в Сизиф. Проблема только в том, что я не знаю как
теперь стянуть один коммит от ldv@ =(

Нашел вот такой пример: http://www.altlinux.org/Git#workflow
Но, он что-то не работает:

$ git remote add ldv ssh://git/people/ldv/packages/apt.git
$ git fetch ldv master
$ git merge ldv/master
fatal: 'ldv/master' does not point to a commit

В выводе git branch -r тоже нет бранча ldv, а вроде бы должен.

ЧЯДНТ? (Что Я Делаю Не Так?)

[...]

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] apt: принять фиксы

[Ivan A. Melnikov]

On Thu, 1 Jan 2009 14:23:07 +0600
"Slava Semushin" <slava.semushin@gmail.com> wrote:

> $ git remote add ldv ssh://git/people/ldv/packages/apt.git
> $ git fetch ldv master

Здесь имхо логичнее (привычнее мне?) смотрится
$ git remote update ldv
хотя это скачает все ветки оттуда, так что если их много, существенно
разных, и Вы экономите траффик...

Эта форма git fetch всего лишь скачивает нужную информацию, не создавая
никаких веток...

> $ git merge ldv/master

Можно ещё порекомендовать вместо пары git fetch + git merge просто

$ git pull ldv master

Делает то же самое, но без лишних плясок. У меня сработало.

[...]

P.S. Отдельное спасибо за Ctrl+D в apt-shell -- прямо подарок к новому
году!
 
-- 
Best Regards,
Ivan A. Melnikov

Re: [devel] apt: принять фиксы

[Michael Shigorin]

On Thu, Jan 01, 2009 at 02:23:07PM +0600, Slava Semushin wrote:
> Готов залить сборку в Сизиф. Проблема только в том, что я не
> знаю как теперь стянуть один коммит от ldv@ =(

git cherry-pick?

> Нашел вот такой пример: http://www.altlinux.org/Git#workflow
> Но, он что-то не работает:
> $ git remote add ldv ssh://git/people/ldv/packages/apt.git

BTW можно просто git:/people/...


On Thu, Jan 01, 2009 at 12:06:03PM +0300, Ivan A. Melnikov wrote:
> P.S. Отдельное спасибо за Ctrl+D в apt-shell -- прямо подарок
> к новому году!

Ага :-)

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] apt: принять фиксы

[Денис Смирнов]

[-- Attachment #1: Type: text/plain, Size: 462 bytes --]

On Thu, Jan 01, 2009 at 06:57:58PM +0200, Michael Shigorin wrote:
>> Готов залить сборку в Сизиф. Проблема только в том, что я не
>> знаю как теперь стянуть один коммит от ldv@ =(
MS> git cherry-pick?

Кстати я в таких случаях (кто-то другой сделал работу, а моя еще
полноценно не опубликована) предпочитаю все-таки rebase.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] apt: принять фиксы

[Slava Semushin]

1 января 2009 г. 15:06 пользователь Ivan A. Melnikov <iv@altlinux> написал:
[...]
> Можно ещё порекомендовать вместо пары git fetch + git merge просто
>
> $ git pull ldv master
>
> Делает то же самое, но без лишних плясок. У меня сработало.

У меня тоже. Спасибо большое!

P.S. Залил apt в incoming, надеюсь, что пройдёт, хотя в ACL меня вроде бы нет.

[...]

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] apt: принять фиксы

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 520 bytes --]

On Fri, Jan 02, 2009 at 12:54:21AM +0600, Slava Semushin wrote:
> 1 января 2009 г. 15:06 пользователь Ivan A. Melnikov <iv@altlinux> написал:
> [...]
> > Можно ещё порекомендовать вместо пары git fetch + git merge просто
> >
> > $ git pull ldv master
> >
> > Делает то же самое, но без лишних плясок. У меня сработало.
> 
> У меня тоже. Спасибо большое!
> 
> P.S. Залил apt в incoming, надеюсь, что пройдёт, хотя в ACL меня вроде бы нет.

acl не пройдёт, но я пакет увижу, проверю и пропущу.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] apt: принять фиксы

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 802 bytes --]

On Thu, Jan 01, 2009 at 02:23:07PM +0600, Slava Semushin wrote:
> 1 января 2009 г. 2:54 пользователь Alexey Tourbin <at@altlinux> написал:
> [...]
> >> Всё это доступно в гите:
> >> http://git.altlinux.org/people/php_coder/packages/apt.git
> >
> > Отправляй на сборку, чево людям голову морочить.
> 
> Во-первых, потому что я не мэйнтейнер apt-а.
> Во-вторых, хотелось бы чтобы мои изменения по-review-или.

NMU подразумевает peer review, и никакого NMU в идеале не должно быть
без peer review.  То есть достаточно квалифицированный чел должен
посмотреть чево туда хотят залить прежде чем это туда зальётся.

Но реализация может быть разной.  Например, мастер раз в день сидит
и смотрит чево кто куда заливает.  Или же возможна более гибкая схема
просмотра и подтверждения изменений.

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]