[buildd-tools-devel] Bug#590211: Bug#590211: Bug#590211: sbuild: Please use one script to perform apt-get update, upgrade, and/or dist-upgrade

[Roger Leigh]

tags 590211 + fixed-upstream pending
thanks

On Mon, Aug 16, 2010 at 03:25:39AM -0400, Andres Mejia wrote:
> On Saturday 24 July 2010 19:02:34 Andres Mejia wrote:
> > Package: sbuild
> > Version: 0.60.1-2
> > Severity: wishlist
> > Tags: patch
> > 
> > It would be helpful if one script (perhaps sbuild-update) were to be used
> > to perform apt-get update, upgrade, and/or dist-upgrade on an sbuild
> > chroot. This is especially helpful for those (like me) who use an sbuild
> > chroot tarball.
> > 
> > This patch is from bug #545215. I've submitted a new bug with just this new
> > change since I thought this change alone would be easier to review.
> 
> Just as I mentioned in my other emails, I've setup a git repository for my 
> changes at http://github.com/ceros/sbuild. The branch for this bug is
> 'sbuild-update'.

Thanks, it looks good.  Some minor comments:

"may change at a later revision" should probably be
"may change in the future" or "may change in a future release"
(slightly more easy to understand)

Also, the command "sbuild-clean" includes --clean, --autoclean
and --autoremove commands which are also related.  Might these
also be worth moving into sbuild-update?  The cleanup could
be done following the existing update/upgrade part.

I've merged the existing patch in, but it would be great to do
the above as well if you agree.


Many thanks,
Roger

-- 
  .''`.  Roger Leigh
 : :' :  Debian GNU/Linux             http://people.debian.org/~rleigh/
 `. `'   Printing on GNU/Linux?       http://gutenprint.sourceforge.net/
   `-    GPG Public Key: 0x25BFB848   Please GPG sign your mail.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/buildd-tools-devel/attachments/20100819/748b9a24/attachment.pgp>