devel@ where you _can_ ask
 help / color / mirror / Atom feed
From: "Vladimir D. Seleznev" <vseleznv@altlinux.org>
To: "devel@ where you _can_ ask" <devel-newbies@lists.altlinux.org>
Subject: Re: [newbies] [#197105] EPERM curseofwar.git=1.2.0.0.11.git47f7989-alt1
Date: Wed, 20 Dec 2017 10:29:51 +0300
Message-ID: <20171220072951.GA21919@portlab> (raw)
In-Reply-To: <alpine.LFD.2.20.1712200921330.7714@imap.altlinux.org>

Такую сборку нельзя пропускать в Сизиф: вы получили gear-репозиторий, в
котором в релизном теге лежат исходники из апстримного git-репозитория,
и рядом же в каталоге curseofwar-master исходники прошлой версии из
предыдущей сборки, когда пакет собирался из sourcerpm. Их надо было
вычистить, или оформить репозиторий по-другому. Далее, мне категорически
не понятно наличие несколько секций %package с описанием каких-то
пакетов %name-ncurses и %name-ncurses, при этом отсутствие каких-либо
упакованных в них файлов или зависимостей на что-либо.

Также я полностью поддерживаю imz@: схема с упаковкой архива из .
создаёт проблемы. В случае, когда за основу берётся астримный
git-репозиторий, не самой плохой схемой будет хранение патчей не в виде
файлов-патчей, а в коммитах с последующей генерацией кумулятивного
патча:

    $ cat .gear/rules
    tar: v@version@:.
    diff v@version@:. .

и упаковкой исходников из тега, предоставляемого апстримом (получаем
практически чистые исходники).

Но и такая схема имеет подводные камни, особенно когда у апстрима релизы
в отдельных ветках без наследования релизом истории предыдущего релиза:
в этом случае просто смёржить новую версию не получится, надо делать
rebase патчей.

On Wed, Dec 20, 2017 at 09:30:17AM +0300, Ivan Zakharyaschev wrote:
> Hi!
> 
> On Tue, 19 Dec 2017, Grigory Ustinov wrote:
> 
> > Обновил CurseOfWar. Можно пропустить?
> 
> Сейчас там в .gear/rules что-то вроде:
> 
> tar: .
> copy: *.patch
> 
> Я бы при такой схеме, когда патчи лежат в отдельных файлах, а исходники 
> нетронуты, хотел бы увидеть что-то вроде:
> 
> tar: neven-sajko-master
> copy: *.patch
> 
> иначе сидишь и думаешь (или робот сидит и не знает при автоматическкой 
> заливке новых исходников), были ли исходники изменены тобой,нужно ли 
> разбираться с этими изменениями. (Ср. сделанное в этом году автоматическое 
> обновление исходников питоновских модулей, когда человеку было просто не 
> под силу решить, на какие пакеты нужно обратить внимание, а на какие -- 
> нет, потому что там нет изменений исходников в не-upstream-ных коммитах.)
> 
> Чтобы это сработало, нужно подготовить ветку как-то так:
> 
> git branch neven-sajko-master upstream/neven-sajko-master
> # edit .gear/rules
> gear-store-tags -avc
> 
> > On 19.12.2017 20:43, Girar Builder awaiter robot wrote:
> >>  http://git.altlinux.org/tasks/197105/logs/events.1.1.log
> >>
> >>  2017-Dec-19 17:39:48 :: task #197105 for sisyphus started by grenka:
> >>  #100 build 1.2.0.0.11.git47f7989-alt1 from
> >>  /people/grenka/packages/curseofwar.git
> >>  2017-Dec-19 17:39:48 :: waiting for a shared lock on sisyphus
> >>  2017-Dec-19 17:40:08 :: acquired a shared lock on sisyphus
> >>  2017-Dec-19 17:40:08 :: [x86_64] #100 curseofwar.git
> >>  1.2.0.0.11.git47f7989-alt1: build start
> >>  2017-Dec-19 17:40:08 :: [i586] #100 curseofwar.git
> >>  1.2.0.0.11.git47f7989-alt1: build start
> >>  2017-Dec-19 17:40:44 :: [x86_64] #100 curseofwar.git
> >>  1.2.0.0.11.git47f7989-alt1: build OK
> >>  2017-Dec-19 17:40:47 :: [i586] #100 curseofwar.git
> >>  1.2.0.0.11.git47f7989-alt1: build OK
> >>  2017-Dec-19 17:40:51 :: build check OK
> >>  2017-Dec-19 17:40:51 :: noarch check OK
> >>  2017-Dec-19 17:40:53 :: plan: src +1 -1 =18229, i586 +2 -2 =33727, x86_64
> >>  +2 -2 =33724
> >>  2017-Dec-19 17:40:53 :: version check OK
> >>  2017-Dec-19 17:41:32 :: generated apt indices
> >>  2017-Dec-19 17:41:32 :: created next repo
> >>  2017-Dec-19 17:41:44 :: dependencies check OK
> >>  2017-Dec-19 17:42:16 :: ELF symbols check OK
> >>  2017-Dec-19 17:42:37 :: [x86_64] #100 curseofwar: install check OK
> >>  2017-Dec-19 17:42:37 :: [i586] #100 curseofwar: install check OK
> >>  2017-Dec-19 17:42:56 :: [x86_64] #100 curseofwar-debuginfo: install check
> >>  OK
> >>  2017-Dec-19 17:42:57 :: [i586] #100 curseofwar-debuginfo: install check OK
> >>  2017-Dec-19 17:43:14 :: [x86_64-i586] generated apt indices
> >>  2017-Dec-19 17:43:14 :: [x86_64-i586] created next repo
> >>  2017-Dec-19 17:43:22 :: [x86_64-i586] dependencies check OK
> >>  2017-Dec-19 17:43:22 :: gears inheritance check OK
> >>  2017-Dec-20 17:43:22 :: srpm inheritance check OK
> >>  girar-check-perms: access to curseofwar DENIED for grenka: does not belong
> >>  to maintainers list yet
> >>  curseofwar: Operation not permitted
> >>  2017-Dec-19 17:43:22 :: acl check FAILED
> >>  2017-Dec-19 17:43:22 :: task #197105 for sisyphus EPERM

-- 
   С уважением,
   Владимир Селезнев


      reply	other threads:[~2017-12-20  7:29 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-19 17:46 ` Grigory Ustinov
2017-12-20  6:30   ` Ivan Zakharyaschev
2017-12-20  7:29     ` Vladimir D. Seleznev [this message]

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=20171220072951.GA21919@portlab \
    --to=vseleznv@altlinux.org \
    --cc=devel-newbies@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

devel@ where you _can_ ask

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel-newbies/0 devel-newbies/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-newbies devel-newbies/ http://lore.altlinux.org/devel-newbies \
		devel-newbies@lists.altlinux.org devel-newbies@lists.altlinux.ru devel-newbies@lists.altlinux.com
	public-inbox-index devel-newbies

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


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