[gentoo-dev] News Item: LastPass package migration - v2

2016-05-19 Thread Göktürk Yüksek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Here's the version 2 with the fixed author information. I forgot to send the initial email to the PR team, so doing that now. Will commit in 72 hours if there are no objections. - -- Title: LastPass package migration Author: Göktürk Yüksek

[gentoo-portage-dev] Re: [PATCH] repoman: declare '-x', '--xmlparse' command line options obsolete

2016-05-19 Thread Duncan
Göktürk Yüksek posted on Thu, 19 May 2016 18:45:29 -0400 as excerpted: > Repoman pulls in lxml unconditionally now and performs metadata checks > by default. This behavior makes these command line options obsolete > since forcing the default makes little sense. Declare them obsolete > instead of

Re: [gentoo-dev] [PATCH] flag-o-matic.eclass: bugfix for get-flag()

2016-05-19 Thread Mike Frysinger
On 16 May 2016 14:17, rindeal wrote: > So this is what it looks like now: still missing tests. see eclass/tests/flag-o-matic.sh. > - local f var findflag="$1" > - > - # this code looks a little flaky but seems to work for > - # everything we want ... > - # for example, if

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

2016-05-19 Thread Mike Frysinger
On 18 May 2016 22:25, aide...@gentoo.org wrote: > 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. what's your real use case ? i've never once thought

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

2016-05-19 Thread Mike Frysinger
On 20 May 2016 02:16, Mart Raudsepp wrote: > I would like if such a function would explicitly document that calling > this function means the caller should DEPEND on an awk provider. > Similarly, I would like that all ebuild that call 'sed' actually DEPEND > on sed, not just half of them. > I

Re: [gentoo-dev] new developers' keyword requests

2016-05-19 Thread Daniel Campbell
On 05/19/2016 07:51 AM, Jeroen Roovers wrote: > Perhaps it's a good idea to add a section to the devmanual about adding > new keywords to packages. > > Recruits in particular might benefit from some background on what > keywording means and when it should be done, especially before they > start

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

2016-05-19 Thread Göktürk Yüksek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Mart Raudsepp: > Ühel kenal päeval, K, 18.05.2016 kell 22:25, kirjutas > 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

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

2016-05-19 Thread Mart Raudsepp
Ühel kenal päeval, K, 18.05.2016 kell 22:25, kirjutas 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

[gentoo-portage-dev] [PATCH] repoman: declare '-x', '--xmlparse' command line options obsolete

2016-05-19 Thread Göktürk Yüksek
Repoman pulls in lxml unconditionally now and performs metadata checks by default. This behavior makes these command line options obsolete since forcing the default makes little sense. Declare them obsolete instead of removing them for backwards compatibility. Signed-off-by: Göktürk Yüksek

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

2016-05-19 Thread Robin H. Johnson
On Thu, May 19, 2016 at 10:50:48PM +0100, Amadeusz Żołnowski wrote: > "Robin H. Johnson" writes: > > 1. > > If your AWK command has an error, then this clobbers the file, a > > better variant, with die, would be: > > awk "$@" "$f" >"${tmpf}" || die "awk failed..." > > cp -f

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

2016-05-19 Thread Amadeusz Żołnowski
"Robin H. Johnson" writes: > 1. > If your AWK command has an error, then this clobbers the file, a > better variant, with die, would be: > awk "$@" "$f" >"${tmpf}" || die "awk failed..." > cp -f "${tmpf}" "$f" || die "copy back failed" Thanks! On the one hand it's a simple

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

2016-05-19 Thread Amadeusz Żołnowski
Ulrich Mueller writes: > Yes, of course. "|| die -n || return" if you want the function to > return at that point, if it was called under nonfatal. > >> https://blogs.gentoo.org/mgorny/2015/11/13/the-ultimate-guide-to-eapi-6 > > Note that the ${?} in the code example there is

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

2016-05-19 Thread Ulrich Mueller
> On Thu, 19 May 2016, Mart Raudsepp wrote: >> Ulrich Mueller writes: >> > The EAPI 6 method would be to use "|| die -n". Then the caller >> > either call "eawk" if the function should die automatically, or >> > "nonfatal eawk" if it should return with an error status. >

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

2016-05-19 Thread Mart Raudsepp
Ühel kenal päeval, N, 19.05.2016 kell 18:51, kirjutas Amadeusz Żołnowski: > Ulrich Mueller writes: > > The EAPI 6 method would be to use "|| die -n". Then the caller can > > either call "eawk" if the function should die automatically, or > > "nonfatal eawk" if it should return

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

2016-05-19 Thread Amadeusz Żołnowski
Ulrich Mueller writes: > The EAPI 6 method would be to use "|| die -n". Then the caller can > either call "eawk" if the function should die automatically, or > "nonfatal eawk" if it should return with an error status. Yes, that seems better. Hasn't it been introduced earlier

Re: [gentoo-dev] new developers' keyword requests

2016-05-19 Thread Mike Gilbert
On Thu, May 19, 2016 at 10:51 AM, Jeroen Roovers wrote: > Perhaps it's a good idea to add a section to the devmanual about adding > new keywords to packages. > > Recruits in particular might benefit from some background on what > keywording means and when it should be done,

[gentoo-dev] new developers' keyword requests

2016-05-19 Thread Jeroen Roovers
Perhaps it's a good idea to add a section to the devmanual about adding new keywords to packages. Recruits in particular might benefit from some background on what keywording means and when it should be done, especially before they start maintaining packages and then realise their packages are so

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

2016-05-19 Thread Zac Medico
On 05/19/2016 03:12 AM, Alexander Berntsen wrote: > 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

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

2016-05-19 Thread Anthony G. Basile
From: "Anthony G. Basile" The current method to check for the system locale is to use python's ctypes.util.find_library() to construct a full library path to the system libc.so which is then passed to ctypes.CDLL(). However, this gets bogged down in implementation dependant

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

2016-05-19 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-dev] [PATCH] eutils.eclass: Add awk wrapper - eawk - edit file in place

2016-05-19 Thread Ulrich Mueller
> On Wed, 18 May 2016, Amadeusz Żołnowski wrote: >> Also, wouldn't the absence of 'die' cause silent breakages? > I want to caller decide whether die or not and what error messge to > give. The EAPI 6 method would be to use "|| die -n". Then the caller can either call "eawk" if the function

Re: [gentoo-dev] Continuous Stabilization (Regarding Post Install Test Scripts)

2016-05-19 Thread Harald Alfred Weiner
Dear Pallav, how about using a dedicated eclass for arch-testing? E.g., you might provide a function in some eclass that only runs when called from pkg_postinst [1] phase AND when some configuration option is set. Then a maintainer / arch tester might provide the tests inside the ebuild

[gentoo-dev] Continuous Stabilization (Regarding Post Install Test Scripts)

2016-05-19 Thread Pallav Agarwal
This is a continuation of "[gentoo-dev] Proposal for changes for the next EAPI version" thread. Since the former subject isn't very relevant anymore. Target: A place to provide scripts that can be used by an automated arch-tester bot to stabilize Gentoo packages. These scripts can be as simple as