ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Slava Semushin <slava.semushin@gmail.com>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: [devel] Repocop patches review (was: Re:  Как получить чужие правки из gears)
Date: Sat, 23 Apr 2011 23:20:35 +0700
Message-ID: <BANLkTikSS573eEvsopyity5uMaFAMtxiyQ@mail.gmail.com> (raw)

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

             reply	other threads:[~2011-04-23 16:20 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-23 16:20 Slava Semushin [this message]
2011-04-23 16:33 ` REAL
2011-04-23 16:37   ` [devel] Repocop patches review REAL
2011-04-23 18:26 ` [devel] Repocop patches review (was: Re: Как получить чужие правки из gears) Sergey Vlasov

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=BANLkTikSS573eEvsopyity5uMaFAMtxiyQ@mail.gmail.com \
    --to=slava.semushin@gmail.com \
    --cc=devel@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

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/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 devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git