Re: [gentoo-dev] [PATCH] rebar.eclass: Build Erlang/OTP projects using dev-util/rebar

2016-05-18 Thread Michał Górny
On Wed, 18 May 2016 23:35:18 +0100 aide...@gentoo.org wrote: > From: Amadeusz Żołnowski > > It is an eclass providing functions to build Erlang/OTP projects using > dev-util/rebar. All packages in upcoming category dev-erlang are going > to use this eclass. > --- >

Re: [gentoo-dev] [PATCH] eutils.eclass: Add awk wrapper - eawk - edit file in place

2016-05-18 Thread Göktürk Yüksek
Jeroen Roovers: > On Wed, 18 May 2016 19:31:38 -0400 > Göktürk Yüksek wrote: > >> There could be some performance implications. cat will usually do >> slow, buffered I/O. cp tries to be smarter with allocation, i.e. it >> may take advantage of the btrfs specific clone to do a

Re: [gentoo-dev] [PATCH] eutils.eclass: Add awk wrapper - eawk - edit file in place

2016-05-18 Thread Jeroen Roovers
On Wed, 18 May 2016 19:31:38 -0400 Göktürk Yüksek wrote: > There could be some performance implications. cat will usually do > slow, buffered I/O. cp tries to be smarter with allocation, i.e. it > may take advantage of the btrfs specific clone to do a O(1) copy. Really?

Re: [gentoo-dev] [PATCH] rebar.eclass: Build Erlang/OTP projects using dev-util/rebar

2016-05-18 Thread Peter Stuge
Cool! aide...@gentoo.org wrote: > +_find_dep_version() { > + local pn="$1" > + local p > + > + pushd "${EPREFIX}$(get_erl_libs)" >/dev/null > + for p in ${pn} ${pn}-*; do > + if [[ -d ${p} ]]; then > + echo "${p#${pn}-}" > +

Re: [gentoo-dev] [PATCH] eutils.eclass: Add awk wrapper - eawk - edit file in place

2016-05-18 Thread Göktürk Yüksek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Amadeusz Żołnowski: > Göktürk Yüksek writes: >>> + cat "${f}" >"${tmpf}" || return 1 >> Why shell redirection with cat instead of cp? both are in >> coreutils. > > I thought cp could overwrite file mode of already existing

Re: [gentoo-dev] [PATCH] eutils.eclass: Add awk wrapper - eawk - edit file in place

2016-05-18 Thread Amadeusz Żołnowski
Göktürk Yüksek writes: >> +cat "${f}" >"${tmpf}" || return 1 > Why shell redirection with cat instead of cp? both are in coreutils. I thought cp could overwrite file mode of already existing tmp file, but actually it doesn't, so cp can be here as well. Is there actual

Re: [gentoo-dev] [PATCH] eutils.eclass: Add awk wrapper - eawk - edit file in place

2016-05-18 Thread Robin H. Johnson
On Wed, May 18, 2016 at 10:25:02PM +0100, aide...@gentoo.org wrote: > From: Amadeusz Żołnowski > > awk doesn't have the -i option like sed and if editing file in place is > desired, additional steps are required. eawk uses tmp file to make it > look to the caller editing

[gentoo-dev] [PATCH] rebar.eclass: Build Erlang/OTP projects using dev-util/rebar

2016-05-18 Thread aidecoe
From: Amadeusz Żołnowski It is an eclass providing functions to build Erlang/OTP projects using dev-util/rebar. All packages in upcoming category dev-erlang are going to use this eclass. --- eclass/rebar.eclass | 220 1

Re: [gentoo-dev] [PATCH] eutils.eclass: Add awk wrapper - eawk - edit file in place

2016-05-18 Thread Göktürk Yüksek
aide...@gentoo.org: > From: Amadeusz Żołnowski > > awk doesn't have the -i option like sed and if editing file in place is > desired, additional steps are required. eawk uses tmp file to make it > look to the caller editing happens in place. > --- > eclass/eutils.eclass | 13

[gentoo-dev] [PATCH] eutils.eclass: Add awk wrapper - eawk - edit file in place

2016-05-18 Thread aidecoe
From: Amadeusz Żołnowski awk doesn't have the -i option like sed and if editing file in place is desired, additional steps are required. eawk uses tmp file to make it look to the caller editing happens in place. --- eclass/eutils.eclass | 13 + 1 file changed, 13

Re: [gentoo-dev] [RFC] Enable CMAKE_WARN_UNUSED_CLI by default in cmake-utils for EAPI>=6

2016-05-18 Thread Maciej Mrozowski
On Wednesday 18 of May 2016 09:22:53 Andrew Savchenko wrote: > On Mon, 02 May 2016 18:06:44 +0200 Maciej Mrozowski wrote: > > Hello, > > > > General advise: do not convert ebuilds inheriting cmake-utils to EAPI 6 > > unless you know what you are doing (you are fully aware of eclass > > behaviour

[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

[gentoo-dev] Package up for grabs: sys-boot/gummiboot

2016-05-18 Thread Michał Górny
I'm no longer willing to maintain sys-boot/gummiboot and this makes it maintainer-needed. The package is no longer maintained upstream, and has been merged into systemd. It seems that there are still people using it without systemd though, so I'm not going to lastrite it myself, and prefer getting

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.

[gentoo-dev] Re: Proposal for changes for the next EAPI version

2016-05-18 Thread Duncan
Ciaran McCreesh posted on Tue, 17 May 2016 12:29:36 +0100 as excerpted: > On Tue, 17 May 2016 07:26:03 -0400 Michael Orlitzky > wrote: >> We already have "emerge --config" which is expected to be run after the >> install process has completed, so I don't think that this is too

[gentoo-dev] Re: Transitioning from #!/sbin/runscript to,#!/sbin/openrc-run

2016-05-18 Thread Austin English
On 05/03/2016 11:27 PM, Austin English wrote: > Hi there, > > I've been working on the transition from #!/sbin/runscript to > #!/sbin/openrc-run [1], by starting on the maintainer-needed packages. > That's done (aside from some stabilizations needed, but I'll deal with that > latter). The trouble

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-dev] Proposal for enhancement to PMS/EAPI7+

2016-05-18 Thread M. J. Everitt
On 18/05/16 07:43, Michał Górny wrote: > On Wed, 18 May 2016 04:07:07 +0100 > "M. J. Everitt" wrote: > >> I've just been party to a discussion over in the Proxy Maintainers >> channel .. and the subject of correct ways to install documentation >> popped up. It seems to me

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

Re: [gentoo-dev] Proposal for enhancement to PMS/EAPI7+

2016-05-18 Thread Michał Górny
On Wed, 18 May 2016 04:07:07 +0100 "M. J. Everitt" wrote: > I've just been party to a discussion over in the Proxy Maintainers > channel .. and the subject of correct ways to install documentation > popped up. It seems to me rather quirky, that there is no middle ground > in

Re: [gentoo-dev] Proposal for enhancement to PMS/EAPI7+

2016-05-18 Thread Ulrich Mueller
> On Wed, 18 May 2016, Kent Fredric wrote: > On 18 May 2016 at 17:40, Ulrich Mueller wrote: >> Only two lines. Do you think this is untidy? > It only becomes untidy where you don't already have a src_install. > Then it becomes 4 lines. > 4 lines of which 3 are redundant

Re: [gentoo-dev] [RFC] Enable CMAKE_WARN_UNUSED_CLI by default in cmake-utils for EAPI>=6

2016-05-18 Thread Andrew Savchenko
On Mon, 02 May 2016 18:06:44 +0200 Maciej Mrozowski wrote: > Hello, > > General advise: do not convert ebuilds inheriting cmake-utils to EAPI 6 > unless > you know what you are doing (you are fully aware of eclass behaviour removed > with https://bugs.gentoo.org/show_bug.cgi?id=514384). > >