From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ldv@altlinux.org>
Date: Thu, 31 Oct 2019 15:59:57 +0300
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Message-ID: <20191031125957.GA19388@altlinux.org>
References: <20191031115808.35705-1-darktemplar@altlinux.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
 protocol="application/pgp-signature"; boundary="k1lZvvs/B4yU6o8G"
Content-Disposition: inline
In-Reply-To: <20191031115808.35705-1-darktemplar@altlinux.org>
Subject: Re: [devel] [PATCH for rpm] Add triggers circumvension for
 packagekit offline update
X-BeenThere: devel@lists.altlinux.org
X-Mailman-Version: 2.1.12
Precedence: list
Reply-To: ALT Linux Team development discussions <devel@lists.altlinux.org>
List-Id: ALT Linux Team development discussions <devel.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/options/devel>,
 <mailto:devel-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/devel>
List-Post: <mailto:devel@lists.altlinux.org>
List-Help: <mailto:devel-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
 <mailto:devel-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Thu, 31 Oct 2019 12:59:57 -0000
Archived-At: <http://lore.altlinux.org/devel/20191031125957.GA19388@altlinux.org/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>


--k1lZvvs/B4yU6o8G
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Thu, Oct 31, 2019 at 02:58:08PM +0300, Aleksei Nikiforov wrote:
> Joint change with Oleg Solovyov (mcpain@)

Thanks.

It's not clear from the commit message why one might need to disable
rpmdb --rebuilddb after rpm update.

Could you elaborate, please?

> ---
>  alt/rpm.spec | 2 ++
>  1 file changed, 2 insertions(+)
>=20
> diff --git a/alt/rpm.spec b/alt/rpm.spec
> index 6217493b7..d93e701b1 100644
> --- a/alt/rpm.spec
> +++ b/alt/rpm.spec
> @@ -385,6 +385,7 @@ ls -A tests/rpmtests.dir 2>/dev/null ||:
>  #[ ! -L %%_rpmlibdir/noarch-alt-%%_target_os ] || rm -f %%_rpmlibdir/noa=
rch-alt-%%_target_os ||:
> =20
>  %post
> +if [ ! -f %_localstatedir/PackageKit/disable-rpm-triggers ]; then
>  #chgrp %%name %%_localstatedir/%%name/[A-Z]*
>  [ -n "$DURING_INSTALL" -o -n "$BTE_INSTALL" ] ||
>          %_rpmlibdir/pdeath_execute $PPID %_rpmlibdir/postupdate
> @@ -393,6 +394,7 @@ ls -A tests/rpmtests.dir 2>/dev/null ||:
>  if set /var/cache/apt/*.bin && [ -f "$1" ]; then
>          %_rpmlibdir/pdeath_execute $PPID rm -f "$@"
>  fi
> +fi
>  :

If this change is needed, it should rather look this way:

if [ -f %_localstatedir/PackageKit/disable-rpm-triggers ]; then
	exit 0
fi


--=20
ldv

--k1lZvvs/B4yU6o8G
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIcBAEBCAAGBQJdutrNAAoJEAVFT+BVnCUI788QAKZYiEIKliaKMcZcwt49Q3yO
fqLxDiB1WTBgesiMB1DE0m+EqNiGIVDH9rn2IvXDel0n9tEUhksDO6DjTKNgNwd0
hPdMO3KOTVsR675tH1QiqSWCthr2nw48kcZT5pkeUtchEIt/NZk5/1lZh6nIGUqx
qxgXrnj2q9l4AlsPJa/BTBoNrdqE/VDIQFFoisv70U3/3reiMtfrKcc0uB3PTzlk
xhEkREzpC2Bg5lAEYzxHNgoGwxOuHW2Yk4irLmIC6P0fcmHIremCB3xsnBnyJQDi
1fEQVgwasvOPUWfZSK/DUyfQUxWtfLDZFpNLg9hf/y7AmnCGk05BoP9DqKpx6MHN
kiwx4bWexMLvINbsoDPPv+hJUWVjB9el57qSJUZU536XVSjIW0CjgMD6j9rZos0s
PktTb93m9ErIbQjWARVoVe9tyPMZ4JTQImIAw+VzqKSA8g//042SISklXd4woLZZ
8dNjaohYK97keT15SR5x9lTV5wXtJFg97jU0Dyja9Xly8yfBg6J+qDcdSEhm5Bn+
QX6XKNU3Cy9Pv76XgM3Jh/RKLLjkeHw/CZNSvMvnys42JHRUksX4D0Ji6iKZgvox
L+JNE0o5sghHplZYYxQEieAv9yNz4FMYZH0hEtRuT3hfNNWo4aN0iB6tjMhP4wLf
0QhTUZ+vfzVaSKaktmtH
=qMuY
-----END PGP SIGNATURE-----

--k1lZvvs/B4yU6o8G--