ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Ilya Mashkin <oddity@altlinux.ru>
To: "Dmitry V. Levin" <ldv@altlinux.org>
Cc: ALT Devel discussion list <devel@lists.altlinux.org>
Subject: Re: [devel] [#269792] DONE (try 2) srpm=gprolog-1.4.5-alt2.src.rpm
Date: Thu, 15 Apr 2021 17:09:50 +0300
Message-ID: <CAG=RUSm4V3do+gdyLkM8H8-hZB=OxrSRjYXjaOEW7sq8nVCPCQ@mail.gmail.com> (raw)
In-Reply-To: <20210413013132.GA19620@altlinux.org>

On Tue, Apr 13, 2021 at 4:31 AM Dmitry V. Levin <ldv@altlinux.org> wrote:
>
> On Tue, Apr 13, 2021 at 01:03:32AM +0000, Girar pender (oddity) wrote:
> > http://git.altlinux.org/tasks/archive/done/_263/269792/logs/events.2.1.log
> >
> > 2021-Apr-13 01:00:07 :: task #269792 for sisyphus resumed by oddity:
> > #100 build gprolog-1.4.5-alt2.src.rpm
> [...]
> > #100 gprolog 1.4.5-alt1 -> 1.4.5-alt2
> >  Tue Apr 13 2021 Ilya Mashkin <oddity@altlinux.ru> 1.4.5-alt2
> >  - fix build
> [...]
> > gprolog: git inheritance check relaxed by oddity
> > warning: update of `gprolog' by srpm upload, /gears/g/gprolog.git branch `sisyphus' will be removed
> [...]
> > 2021-Apr-13 01:03:05 :: created /srpms/g/gprolog.git branch `sisyphus'
> > 2021-Apr-13 01:03:05 :: saved /gears/g/gprolog.git branch `sisyphus' as `old/sisyphus-task269792'
>
> Такой "fix build" простой заменой формата хранения исходного кода не спрячешь.
> Нет, спасибо большое, конечно, но таких "fix build" в Сизифе не требуется.

Плюс:
> 2021-Apr-13 01:34:17 :: task #269796 for sisyphus started by qa_ldv:
> 2021-Apr-13 01:34:17 :: message: Remove gprolog-1.4.5-alt2 package raped by "fix build".


Дима, хотелось бы более доброжелательных и понятных комментариев.
Например: Илья, не стоит так делать [потому что...]
Еще и учитывая, что это маленький, не особо кому нужный, кроме меня,
вяло меняющийся десятилетиями пакет, который и так удалился бы через
неделю.

Если сократить до тех строк, которые на что-то влияют, то:


> diff --git a/gprolog.spec b/gprolog.spec
> index dc76c29..f7d2886 100644
> --- a/gprolog.spec
> +++ b/gprolog.spec

> +Patch3:   gprolog.make-print-submake-output.patch

Забытый мелкий патчик из прошлой версии, не особо нужный.

> -ExclusiveArch: x86_64 %ix86
> +%set_gcc_version 8
> +BuildRequires:  gcc8 libgdb-devel perl-devel
> +ExclusiveArch: x86_64 %ix86 %e2k

Перл здесь лишний, уберу, а остальное нужно чтобы и под 586 собралось
и под x86_64.
 %e2k добавлен на будущее, он ни на что не влияет.

> +%patch3 -p1

> +%define _lto_cflags %{nil}
> +%define _legacy_common_support 1
> +%add_optflags -fcommon

Аналогично стандартные вещи из FC и ALT  чтобы и под 586 собралось и
под x86_64 в текущем окружении.

Можно было подробнее changelog написать? Можно, но необязательно.
Напишу подробнее.

Можно было не переводить в srpm?  Можно, но необязательно. Могу и в git вернуть.

В итоге я не вижу каких-то проблем и препятствий для нахождения пакета в Cизифе.

  reply	other threads:[~2021-04-15 14:09 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-13  1:31   ` Dmitry V. Levin
2021-04-15 14:09     ` Ilya Mashkin [this message]
2021-04-15 15:34       ` Dmitry V. Levin

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='CAG=RUSm4V3do+gdyLkM8H8-hZB=OxrSRjYXjaOEW7sq8nVCPCQ@mail.gmail.com' \
    --to=oddity@altlinux.ru \
    --cc=devel@lists.altlinux.org \
    --cc=ldv@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