[devel] Repocop patches review (was: Re: Как получить чужие правки из gears)

[devel] Repocop patches review (was: Re: Как получить чужие правки из gears)

[Slava Semushin]

23 апреля 2011 г. 23:01 пользователь REAL <root@mmedia2.kemsu.ru> написал:
[...]
>> У меня не сработало. В git://git.altlinux.org/gears/ нет
>> соответствующих gear-репозиториев для пакетов pachi и netris.
>>
>> Кто-нибудь знает почему?
>
> потому что они собирались из srpm, т.е. искать из надо не в /gears/, а в
> /srpms/

Спасибо! Нашел.

Сижу, вот, печалюсь теперь: у меня в git-репо пакета pachi было два
мелких изменения в спеке, в новых сборках они не учтены (не знаю что
теперь мне с ними делать?). При попытке мержа изменений репокопа
получаю конфликт :(

Посмотрел на изменения, которые были внесены и они конечно правильные,
но если бы я их прикладывал сам, то точно бы немного подрихтовал.
Например:

+* Thu Feb 04 2010 Repocop Q. A. Robot <repocop@altlinux.org> 1.0-alt3.qa1
+- NMU (by repocop): the following fixes applied:
+  * pixmap-in-deprecated-location for pachi
+  * postclean-05-filetriggers for spec file

Про pixmap-in-deprecated-location понятно, а вот что за изменения
такое postclean-05-filetriggers я не понял. Судя по diff-у его нет.
Получается что в %changelog присутствует то, чего нет на самом деле?

Далее:

+mkdir -p %buildroot%_desktopdir
+cat > %buildroot%_desktopdir/%{name}.desktop <<EOF
+[Desktop Entry]
+Version=1.0
+Type=Application
+Name=Pachi
+GenericName=
+Comment=%{summary}
+Icon=%{name}
+Exec=%_gamesbindir/%name
+#Exec=%name
+Terminal=false
+Categories=Game;ArcadeGame;
+EOF

Смущают макросы в стиле ${name} вместо простого %name и
закомментированная строка с Exec, которая выглядит бесполезной.

Короче, я опечален, потому что к своим пакетам, репозиториям, коду и
всему остальному обычно отношусь весьма педантично, а тут такие
изменения, которые я мог бы поправить ещё до того как они попали ко
мне :(

Витают мысли убрать @everybody тогда со своих пакетов, чтобы меня, как
мэйнтейнера, просто не смогли проигнорировать.

-- 
Slava Semushin

Re: [devel] Repocop patches review (was: Re: Как получить чужие правки из gears)

[REAL]

23.04.2011 23:20, Slava Semushin пишет:
> Сижу, вот, печалюсь теперь: у меня в git-репо пакета pachi было два
> мелких изменения в спеке, в новых сборках они не учтены (не знаю что
> теперь мне с ними делать?). При попытке мержа изменений репокопа
> получаю конфликт :(

такие проблемы для сборки из srpm довольно обыденны. именно поэтому я 
обычно настаиваю, чтобы сборка переезжала на gear, если не планируется 
выдавать ACL никому.

> Витают мысли убрать @everybody тогда со своих пакетов, чтобы меня, как
> мэйнтейнера, просто не смогли проигнорировать.

тут зависит от мейнтейнера, конечно же. просто заранее сложно 
установить, мейнтейнер действующий или уже ушедший (например, george@ 
на личную почту вообще не отвечает, но в последние дни с его стороны 
была замечена активность в сизифе).

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] Repocop patches review

[REAL]

23.04.2011 23:33, REAL пишет:
> такие проблемы для сборки из srpm довольно обыденны. именно поэтому я
> обычно настаиваю, чтобы сборка переезжала на gear, если не планируется
> выдавать ACL никому.

s/выдавать/не выдавать/

-- 

REAL aka Евгений Ростовцев, программист ЦНИТ КемГУ

Re: [devel] Repocop patches review (was: Re: Как получить чужие правки из gears)

[Sergey Vlasov]

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

On Sat, Apr 23, 2011 at 11:20:35PM +0700, Slava Semushin wrote:
> Сижу, вот, печалюсь теперь: у меня в git-репо пакета pachi было два
> мелких изменения в спеке, в новых сборках они не учтены (не знаю что
> теперь мне с ними делать?). При попытке мержа изменений репокопа
> получаю конфликт :(

Это неудивительно, поскольку у репозиториев нет общей истории.
Однако, поскольку структура репозитория pachi.git не сильно отличается
от результата импорта src.rpm, можно попробовать объединить изменения
хотя бы с частичной автоматизацией:

  git fetch --no-tags git://git.altlinux.org/srpms/p/pachi.git refs/tags/1.0-alt3:refs/tags/1.0-alt3-qa sisyphus:qa
  git checkout qa
  git rebase --onto 1.0-alt3 1.0-alt3-qa
  git checkout master
  git merge qa

Потом можно удалить ненужный тег и бранч:

  git tag -d 1.0-alt3-qa
  git branch -d qa

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