[gentoo-portage-dev] Re: [PATCH v2] Change how the tmp file for the commit msg is made (bug 571546)

2016-05-18 Thread Zac Medico
On 05/18/2016 11:57 AM, Doug Goldstein wrote: > On 5/18/16 11:38 AM, Zac Medico wrote: >> On Tue, May 17, 2016 at 11:43 PM, Alexander Berntsen >> wrote: >>> -BEGIN PGP SIGNED MESSAGE- >>> Hash: SHA512 >>> >>> On 17/05/16 15:36, Doug Goldstein wrote: ping? I

Re: [gentoo-portage-dev] [PATCH] news.py: Check only for major version when parsing

2016-05-18 Thread Alexander Berntsen
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 18/05/16 17:46, Zac Medico wrote: > not fnmatch.fnmatch(format_match.group(1), '1.*')): Wait, why "not"? - -- Alexander berna...@gentoo.org https://secure.plaimi.net/~alexander -BEGIN PGP SIGNATURE- Version: GnuPG v2

Re: [gentoo-portage-dev] [PATCH] news.py: Check only for major version when parsing

2016-05-18 Thread Alexander Berntsen
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 18/05/16 17:46, Zac Medico wrote: > if (format_match is not None and not > fnmatch.fnmatch(format_match.group(1), '1.*')): Yes! Sorry! I had made that change, but then forgotten to stage it. So sorry. It's been hell of a day, so it just slipped

[gentoo-portage-dev] Re: [PATCH v2] Change how the tmp file for the commit msg is made (bug 571546)

2016-05-18 Thread Zac Medico
On Tue, May 17, 2016 at 11:43 PM, Alexander Berntsen wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA512 > > On 17/05/16 15:36, Doug Goldstein wrote: >> ping? I believe I covered all the review items but I see this is >> not in master yet. Is there anything further you

Re: [gentoo-portage-dev] [PATCH] pym/portage/util/locale.py: add a C module to check locale

2016-05-18 Thread Anthony G. Basile
On 5/18/16 10:51 AM, Anthony G. Basile wrote: > On 5/18/16 10:26 AM, Brian Dolbec wrote: >> On Wed, 18 May 2016 06:08:20 -0400 >> "Anthony G. Basile" wrote: >> >>> On 5/18/16 2:50 AM, Alexander Berntsen wrote: On 17/05/16 14:47, Anthony G. Basile wrote: >

Re: [gentoo-portage-dev] [PATCH] pym/portage/util/locale.py: add a C module to check locale

2016-05-18 Thread Anthony G. Basile
On 5/18/16 10:26 AM, Brian Dolbec wrote: > On Wed, 18 May 2016 06:08:20 -0400 > "Anthony G. Basile" wrote: > >> On 5/18/16 2:50 AM, Alexander Berntsen wrote: >>> On 17/05/16 14:47, Anthony G. Basile wrote: Since this is the first python module written in C

Re: [gentoo-portage-dev] [PATCH] pym/portage/util/locale.py: add a C module to check locale

2016-05-18 Thread Anthony G. Basile
On 5/17/16 9:38 AM, Brian Dolbec wrote: > > Overall I like it, but I'll let others review the actual code > implementation since I'm not an experienced "C" coder. In 30 > years, I never did more than hello world a few times in C. And the > pascal coding I did in College is more like python than

Re: [gentoo-portage-dev] [PATCH] pym/portage/util/locale.py: add a C module to check locale

2016-05-18 Thread Brian Dolbec
On Wed, 18 May 2016 06:08:20 -0400 "Anthony G. Basile" wrote: > On 5/18/16 2:50 AM, Alexander Berntsen wrote: > > On 17/05/16 14:47, Anthony G. Basile wrote: > >> Since this is the first python module written in C included in > >> portage, as a side effect, we

Re: [gentoo-portage-dev] [PATCH] pym/portage/util/locale.py: add a C module to check locale

2016-05-18 Thread Anthony G. Basile
On 5/18/16 2:50 AM, Alexander Berntsen wrote: > On 17/05/16 14:47, Anthony G. Basile wrote: >> Since this is the first python module written in C included in >> portage, as a side effect, we introduce the machinary for future >> modules in setup.py. > Split it into two commits. > Read the code,

[gentoo-portage-dev] [PATCH] news.py: Check only for major version when parsing

2016-05-18 Thread Alexander Berntsen
Only check the major version of news items, as GLEP 42 specifies an upgrade path for them. Future revisions to news item formats may yield minor number increments. GLEP 42 further ensures that only forwards-compatible changes may incur, as incompatible changes demand a major version increment.

Re: [gentoo-portage-dev] [PATCH] pym/portage/util/locale.py: add a C module to check locale

2016-05-18 Thread Alexander Berntsen
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 17/05/16 14:47, Anthony G. Basile wrote: > Since this is the first python module written in C included in > portage, as a side effect, we introduce the machinary for future > modules in setup.py. Split it into two commits. - -- Alexander

Re: [gentoo-portage-dev] [PATCH] dblink: add locks for parallel-install with blockers (bug 576888)

2016-05-18 Thread Alexander Berntsen
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 17/05/16 15:48, Brian Dolbec wrote: > Alexander, with 2.3.0_rc1-r1 released, I think we can re-open > portage code for more patches before the official 2.3.0 release. > We now know the split install is working, it only had the one > portage bug

Re: [gentoo-portage-dev] [PATCH] dblink: add locks for parallel-install with blockers (bug 576888)

2016-05-18 Thread Alexander Berntsen
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 On 16/05/16 21:20, Zac Medico wrote: > Can we merge this now? Yes! - -- Alexander berna...@gentoo.org https://secure.plaimi.net/~alexander -BEGIN PGP SIGNATURE- Version: GnuPG v2