Subject: Re: CVS commit: pkgsrc/pkgtools/pkg_rolling-replace
To: None <tnn@netbsd.org>
From: Greg Troxel <gdt@ir.bbn.com>
List: tech-pkg
Date: 08/06/2007 13:38:29
--=-=-=
Tobias Nygren <tnn@netbsd.org> writes:
> Module Name: pkgsrc
> Committed By: tnn
> Date: Mon Aug 6 15:07:56 UTC 2007
>
> Modified Files:
> pkgsrc/pkgtools/pkg_rolling-replace: Makefile
> pkgsrc/pkgtools/pkg_rolling-replace/files: pkg_rolling-replace.sh
>
> Log Message:
> Update to pkg_rolling-replace-0.6:
> - Explicitly unset 'unsafe_depends' and 'rebuild' tags after replacing.
> Needed after changes in pkgsrc/mk/flavor/pkg/replace.mk, fixes PR 36378.
I think this is the wrong fix - doing "make replace" should clear those
tags. While pkg_rolling-replace is the primary (and now only) consumer
of the tags, I wanted the system to be such that all the basic
operations respected the invariants on tags. This enables one to 'make
replace' and later run pkg_rolling-replace, and not lose the information
that dependencies need updating. So, make replace, run independently of
pkg_rr should do the right thing, which is to clear both those tags.
So instead, perhaps pkg_rr should assert that the tags are gone after
replacing, and make replace should be fixed to unset these tags. I
gather it used to lose all tags, and now preserves them.
Later, I'd like to have unsafe_depends and unsafe_depends_strict, with
both being set for ABI-changing updates and only the latter if the
package claims not to have an ABI change. This way pkg_rolling-replace
can avoid much rebuilding, but take a flag to clean the
unsafe_depends_strict flags if something is broken.
> - Introduce ${PKG_ADMIN}, ${PKG_INFO}, ${PKG_CHK} variables instead of
> calling the tools directly by name.
That's of course fine.
Thanks for working on pkg_rolling-replace - it's great to see both
someone else using it and it getting improved.
--=-=-=
Content-Type: application/pgp-signature
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (NetBSD)
iD8DBQFGt1yV+vesoDJhHiURAoxuAJ9XaijTVAOO2wRPpMkil/p4qlLLWACdG85R
533ozQ1sZTnlEdUdX7iGGt4=
=MvAl
-----END PGP SIGNATURE-----
--=-=-=--