public inbox for gentoo-portage-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Brian Dolbec <dolsen@gentoo.org>
To: gentoo-portage-dev@lists.gentoo.org
Subject: Re: [gentoo-portage-dev] [PATCHES] @changed-deps + missing slot check reposted for bernalex
Date: Wed, 13 Aug 2014 23:58:27 -0700	[thread overview]
Message-ID: <20140813235827.3fa8cbb1.dolsen@gentoo.org> (raw)
In-Reply-To: <53EBD41E.7010603@gentoo.org>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

On Wed, 13 Aug 2014 23:09:50 +0200
Alexander Berntsen <bernalex@gentoo.org> wrote:

>  
> For 3, I think it looks nice but you should check with Brian & Tom
> about whether to put it in now or in the refactor branch. Also, please
> ask QA about it so we don't get another bikeshed marathon to deal
> with.
> 
> Thanks for doing this!
> - -- 
> Alexander
> bernalex@gentoo.org

I've already reviewed this patch in the original thread.  I also had
him re-do it into a separate function so it would be easy to insert
into the current master without disrupting the rewrite.  The rewrite
should be easily rebased including this patch plus the few others
recently applied.

But yes, having QA's approval is needed...

I know it is already scheduled with them. :)

- -- 
Brian Dolbec <dolsen>

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (GNU/Linux)

iQF8BAEBCgBmBQJT7F4TXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w
ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXQ4Njg4RkQxQ0M3MUMxQzA0RUFFQTQyMzcy
MjE0RDkwQTAxNEYxN0NCAAoJECIU2QoBTxfL82oIANWK+BzWLXVg6A7+sXWAlVWH
J+WHtubENpq/Cl25H/MSF7altoZ4ZpZVJ15E1ZvGIV47g3A8mZJQEmwSvQ6ZYdll
YotyIGQTsETgyAUe4THngsOx8+mP1MP8mHYl8nHAz6Tu5+OJQxgrdHai3JAsKH/X
KhGOHhhIBsThP9ooj2WoLjijwZ0epgmyN3T6SDy3GxOs18ugdXQuR3EGi5Wa2JCo
57xHdfZIvPFL2tBiLQuDwiN62Ygb5RfyNfiqYTy9zypK1tnNihUWIF8QYZ3NgFVc
pYWYXLVTbvd8/SDAUTcOy+Z5a9hVBZzudXJbi4YVRpSkRHV4YZx1AkojwbKyye0=
=H614
-----END PGP SIGNATURE-----

  reply	other threads:[~2014-08-14  7:14 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-13 17:20 [gentoo-portage-dev] [PATCHES] @changed-deps + missing slot check reposted for bernalex Michał Górny
2014-08-13 17:20 ` [gentoo-portage-dev] [PATCH 1/3] vardb: Do not remove := from vdb on unmatched dependencies Michał Górny
2014-08-13 17:20 ` [gentoo-portage-dev] [PATCH 2/3] sets: Introduce @changed-deps to update packages which need dep changes Michał Górny
2014-08-13 17:20 ` [gentoo-portage-dev] [PATCH 3/3] repoman: Add check for missing slot values/slot operators Michał Górny
2014-08-13 21:09 ` [gentoo-portage-dev] [PATCHES] @changed-deps + missing slot check reposted for bernalex Alexander Berntsen
2014-08-14  6:58   ` Brian Dolbec [this message]
2014-08-27 20:53 ` Brian Dolbec
2014-08-28  9:00 ` Michał Górny

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=20140813235827.3fa8cbb1.dolsen@gentoo.org \
    --to=dolsen@gentoo.org \
    --cc=gentoo-portage-dev@lists.gentoo.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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox