Re: [gentoo-portage-dev] [PATCH] sys-apps/portage: add native-extensions USE flag (bug 571444)
On Wed, Feb 1, 2017 at 4:15 PM, Zac Medico wrote: > On 02/01/2017 01:03 PM, Michał Górny wrote: >> W dniu 01.02.2017, śro o godzinie 09∶06 -0800, użytkownik Zac Medico >> napisał: >>> The native-extensions USE flag will enable building of the >>> libc bindings. This is not enabled by default because it does >>> not support cross compilation. >>> >>> X-Gentoo-bug: 571444 >>> X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=571444 >>> --- >>> sys-apps/portage/metadata.xml| 1 + >>> sys-apps/portage/portage-.ebuild | 7 ++- >>> 2 files changed, 7 insertions(+), 1 deletion(-) >>> >>> diff --git a/sys-apps/portage/metadata.xml b/sys- >>> apps/portage/metadata.xml >>> index e032ea5..882d3ba 100644 >>> --- a/sys-apps/portage/metadata.xml >>> +++ b/sys-apps/portage/metadata.xml >>> @@ -12,6 +12,7 @@ >>> >>> Build html API documentation with >>> epydoc. >>> Use inter-process communication between portage >>> and running ebuilds. >>> +Build native extensions. Cross- >>> compilation is not supported. >>> Preserve extended attributes (filesystem- >>> stored metadata) when installing files. Usually only required for >>> hardened systems. >>> >>> >>> diff --git a/sys-apps/portage/portage-.ebuild b/sys- >>> apps/portage/portage-.ebuild >>> index 6a6f515..981db26 100644 >>> --- a/sys-apps/portage/portage-.ebuild >>> +++ b/sys-apps/portage/portage-.ebuild >>> @@ -19,7 +19,7 @@ HOMEPAGE="https://wiki.gentoo.org/wiki/Project:Porta >>> ge" >>> LICENSE="GPL-2" >>> KEYWORDS="" >>> SLOT="0" >>> -IUSE="build doc epydoc +ipc linguas_ru selinux xattr" >>> +IUSE="build doc epydoc +ipc linguas_ru native-extensions selinux >>> xattr" >> >> Wouldn't it be better to enable it by default? > > Yeah, that should be fine. We may get a few duplicates of bug 594744 > reported, but that's no reason not to enable it by default. +1 on enabling native-extensions by default. The cross-compiling community is a small minority, and we should not base the default on their use case.
Re: [gentoo-portage-dev] [PATCH] sys-apps/portage: add native-extensions USE flag (bug 571444)
W dniu 01.02.2017, śro o godzinie 20∶46 -0500, użytkownik Michael Orlitzky napisał: > On 02/01/2017 04:03 PM, Michał Górny wrote: > > > SLOT="0" > > > -IUSE="build doc epydoc +ipc linguas_ru selinux xattr" > > > +IUSE="build doc epydoc +ipc linguas_ru native-extensions selinux > > > xattr" > > > > Wouldn't it be better to enable it by default? > > > > Please don't enshrine personal preferences into IUSE defaults unless > they're critical to the package. You're saying nonsense. There is no reason for most of Gentoo users to default to crippled Portage. -- Best regards, Michał Górny signature.asc Description: This is a digitally signed message part
Re: [gentoo-portage-dev] [PATCH] sys-apps/portage: add native-extensions USE flag (bug 571444)
On 02/01/2017 04:03 PM, Michał Górny wrote: >> SLOT="0" >> -IUSE="build doc epydoc +ipc linguas_ru selinux xattr" >> +IUSE="build doc epydoc +ipc linguas_ru native-extensions selinux >> xattr" > > Wouldn't it be better to enable it by default? > Please don't enshrine personal preferences into IUSE defaults unless they're critical to the package.
Re: [gentoo-portage-dev] [PATCH] sys-apps/portage: add native-extensions USE flag (bug 571444)
On 02/01/2017 01:03 PM, Michał Górny wrote: > W dniu 01.02.2017, śro o godzinie 09∶06 -0800, użytkownik Zac Medico > napisał: >> The native-extensions USE flag will enable building of the >> libc bindings. This is not enabled by default because it does >> not support cross compilation. >> >> X-Gentoo-bug: 571444 >> X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=571444 >> --- >> sys-apps/portage/metadata.xml| 1 + >> sys-apps/portage/portage-.ebuild | 7 ++- >> 2 files changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/sys-apps/portage/metadata.xml b/sys- >> apps/portage/metadata.xml >> index e032ea5..882d3ba 100644 >> --- a/sys-apps/portage/metadata.xml >> +++ b/sys-apps/portage/metadata.xml >> @@ -12,6 +12,7 @@ >> >> Build html API documentation with >> epydoc. >> Use inter-process communication between portage >> and running ebuilds. >> +Build native extensions. Cross- >> compilation is not supported. >> Preserve extended attributes (filesystem- >> stored metadata) when installing files. Usually only required for >> hardened systems. >> >> >> diff --git a/sys-apps/portage/portage-.ebuild b/sys- >> apps/portage/portage-.ebuild >> index 6a6f515..981db26 100644 >> --- a/sys-apps/portage/portage-.ebuild >> +++ b/sys-apps/portage/portage-.ebuild >> @@ -19,7 +19,7 @@ HOMEPAGE="https://wiki.gentoo.org/wiki/Project:Porta >> ge" >> LICENSE="GPL-2" >> KEYWORDS="" >> SLOT="0" >> -IUSE="build doc epydoc +ipc linguas_ru selinux xattr" >> +IUSE="build doc epydoc +ipc linguas_ru native-extensions selinux >> xattr" > > Wouldn't it be better to enable it by default? Yeah, that should be fine. We may get a few duplicates of bug 594744 reported, but that's no reason not to enable it by default. -- Thanks, Zac
Re: [gentoo-portage-dev] [PATCH] sys-apps/portage: add native-extensions USE flag (bug 571444)
W dniu 01.02.2017, śro o godzinie 09∶06 -0800, użytkownik Zac Medico
napisał:
> The native-extensions USE flag will enable building of the
> libc bindings. This is not enabled by default because it does
> not support cross compilation.
>
> X-Gentoo-bug: 571444
> X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=571444
> ---
> sys-apps/portage/metadata.xml| 1 +
> sys-apps/portage/portage-.ebuild | 7 ++-
> 2 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/sys-apps/portage/metadata.xml b/sys-
> apps/portage/metadata.xml
> index e032ea5..882d3ba 100644
> --- a/sys-apps/portage/metadata.xml
> +++ b/sys-apps/portage/metadata.xml
> @@ -12,6 +12,7 @@
>
> Build html API documentation with
> epydoc.
> Use inter-process communication between portage
> and running ebuilds.
> +Build native extensions. Cross-
> compilation is not supported.
> Preserve extended attributes (filesystem-
> stored metadata) when installing files. Usually only required for
> hardened systems.
>
>
> diff --git a/sys-apps/portage/portage-.ebuild b/sys-
> apps/portage/portage-.ebuild
> index 6a6f515..981db26 100644
> --- a/sys-apps/portage/portage-.ebuild
> +++ b/sys-apps/portage/portage-.ebuild
> @@ -19,7 +19,7 @@ HOMEPAGE="https://wiki.gentoo.org/wiki/Project:Porta
> ge"
> LICENSE="GPL-2"
> KEYWORDS=""
> SLOT="0"
> -IUSE="build doc epydoc +ipc linguas_ru selinux xattr"
> +IUSE="build doc epydoc +ipc linguas_ru native-extensions selinux
> xattr"
Wouldn't it be better to enable it by default?
>
> DEPEND="!build? ( $(python_gen_impl_dep 'ssl(+)') )
> >=app-arch/tar-1.27
> @@ -85,6 +85,11 @@ pkg_setup() {
> python_prepare_all() {
> distutils-r1_python_prepare_all
>
> + if use native-extensions; then
> + printf "[build_ext]\nportage-ext-modules=true" >> \
> + setup.cfg || die
> + fi
> +
> if ! use ipc ; then
> einfo "Disabling ipc..."
> sed -e "s:_enable_ipc_daemon =
> True:_enable_ipc_daemon = False:" \
--
Best regards,
Michał Górny
signature.asc
Description: This is a digitally signed message part
Re: [gentoo-portage-dev] [PATCH] sys-apps/portage: add native-extensions USE flag (bug 571444)
On 02/01/2017 09:57 AM, Brian Dolbec wrote:
> On Wed, 1 Feb 2017 09:06:37 -0800
> Zac Medico wrote:
>
>> The native-extensions USE flag will enable building of the
>> libc bindings. This is not enabled by default because it does
>> not support cross compilation.
>>
>> X-Gentoo-bug: 571444
>> X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=571444
>> ---
>> sys-apps/portage/metadata.xml| 1 +
>> sys-apps/portage/portage-.ebuild | 7 ++-
>> 2 files changed, 7 insertions(+), 1 deletion(-)
>>
>> diff --git a/sys-apps/portage/metadata.xml
>> b/sys-apps/portage/metadata.xml index e032ea5..882d3ba 100644
>> --- a/sys-apps/portage/metadata.xml
>> +++ b/sys-apps/portage/metadata.xml
>> @@ -12,6 +12,7 @@
>>
>> Build html API documentation with
>> epydoc. Use inter-process communication
>> between portage and running ebuilds.
>> +Build native extensions.
>> Cross-compilation is not supported. > name="xattr">Preserve extended attributes (filesystem-stored
>> metadata) when installing files. Usually only required for hardened
>> systems. diff --git
>> a/sys-apps/portage/portage-.ebuild
>> b/sys-apps/portage/portage-.ebuild index 6a6f515..981db26 100644
>> --- a/sys-apps/portage/portage-.ebuild +++
>> b/sys-apps/portage/portage-.ebuild @@ -19,7 +19,7 @@
>> HOMEPAGE="https://wiki.gentoo.org/wiki/Project:Portage";
>> LICENSE="GPL-2" KEYWORDS=""
>> SLOT="0"
>> -IUSE="build doc epydoc +ipc linguas_ru selinux xattr"
>> +IUSE="build doc epydoc +ipc linguas_ru native-extensions selinux
>> xattr"
>> DEPEND="!build? ( $(python_gen_impl_dep 'ssl(+)') )
>> >=app-arch/tar-1.27
>> @@ -85,6 +85,11 @@ pkg_setup() {
>> python_prepare_all() {
>> distutils-r1_python_prepare_all
>>
>> +if use native-extensions; then
>> +printf "[build_ext]\nportage-ext-modules=true" >> \
>> +setup.cfg || die
>> +fi
>> +
>> if ! use ipc ; then
>> einfo "Disabling ipc..."
>> sed -e "s:_enable_ipc_daemon =
>> True:_enable_ipc_daemon = False:" \
>
> looks good :)
>
Thanks, pushed:
https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=dbe0bd44461aff3097fde6c64147f1a61e30816d
--
Thanks,
Zac
Re: [gentoo-portage-dev] [PATCH] sys-apps/portage: add native-extensions USE flag (bug 571444)
On Wed, 1 Feb 2017 09:06:37 -0800
Zac Medico wrote:
> The native-extensions USE flag will enable building of the
> libc bindings. This is not enabled by default because it does
> not support cross compilation.
>
> X-Gentoo-bug: 571444
> X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=571444
> ---
> sys-apps/portage/metadata.xml| 1 +
> sys-apps/portage/portage-.ebuild | 7 ++-
> 2 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/sys-apps/portage/metadata.xml
> b/sys-apps/portage/metadata.xml index e032ea5..882d3ba 100644
> --- a/sys-apps/portage/metadata.xml
> +++ b/sys-apps/portage/metadata.xml
> @@ -12,6 +12,7 @@
>
> Build html API documentation with
> epydoc. Use inter-process communication
> between portage and running ebuilds.
> +Build native extensions.
> Cross-compilation is not supported. name="xattr">Preserve extended attributes (filesystem-stored
> metadata) when installing files. Usually only required for hardened
> systems. diff --git
> a/sys-apps/portage/portage-.ebuild
> b/sys-apps/portage/portage-.ebuild index 6a6f515..981db26 100644
> --- a/sys-apps/portage/portage-.ebuild +++
> b/sys-apps/portage/portage-.ebuild @@ -19,7 +19,7 @@
> HOMEPAGE="https://wiki.gentoo.org/wiki/Project:Portage";
> LICENSE="GPL-2" KEYWORDS=""
> SLOT="0"
> -IUSE="build doc epydoc +ipc linguas_ru selinux xattr"
> +IUSE="build doc epydoc +ipc linguas_ru native-extensions selinux
> xattr"
> DEPEND="!build? ( $(python_gen_impl_dep 'ssl(+)') )
> >=app-arch/tar-1.27
> @@ -85,6 +85,11 @@ pkg_setup() {
> python_prepare_all() {
> distutils-r1_python_prepare_all
>
> + if use native-extensions; then
> + printf "[build_ext]\nportage-ext-modules=true" >> \
> + setup.cfg || die
> + fi
> +
> if ! use ipc ; then
> einfo "Disabling ipc..."
> sed -e "s:_enable_ipc_daemon =
> True:_enable_ipc_daemon = False:" \
looks good :)
--
Brian Dolbec
[gentoo-portage-dev] [PATCH] sys-apps/portage: add native-extensions USE flag (bug 571444)
The native-extensions USE flag will enable building of the
libc bindings. This is not enabled by default because it does
not support cross compilation.
X-Gentoo-bug: 571444
X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=571444
---
sys-apps/portage/metadata.xml| 1 +
sys-apps/portage/portage-.ebuild | 7 ++-
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/sys-apps/portage/metadata.xml b/sys-apps/portage/metadata.xml
index e032ea5..882d3ba 100644
--- a/sys-apps/portage/metadata.xml
+++ b/sys-apps/portage/metadata.xml
@@ -12,6 +12,7 @@
Build html API documentation with epydoc.
Use inter-process communication between portage and
running ebuilds.
+Build native extensions. Cross-compilation
is not supported.
Preserve extended attributes (filesystem-stored
metadata) when installing files. Usually only required for hardened
systems.
diff --git a/sys-apps/portage/portage-.ebuild
b/sys-apps/portage/portage-.ebuild
index 6a6f515..981db26 100644
--- a/sys-apps/portage/portage-.ebuild
+++ b/sys-apps/portage/portage-.ebuild
@@ -19,7 +19,7 @@ HOMEPAGE="https://wiki.gentoo.org/wiki/Project:Portage";
LICENSE="GPL-2"
KEYWORDS=""
SLOT="0"
-IUSE="build doc epydoc +ipc linguas_ru selinux xattr"
+IUSE="build doc epydoc +ipc linguas_ru native-extensions selinux xattr"
DEPEND="!build? ( $(python_gen_impl_dep 'ssl(+)') )
>=app-arch/tar-1.27
@@ -85,6 +85,11 @@ pkg_setup() {
python_prepare_all() {
distutils-r1_python_prepare_all
+ if use native-extensions; then
+ printf "[build_ext]\nportage-ext-modules=true" >> \
+ setup.cfg || die
+ fi
+
if ! use ipc ; then
einfo "Disabling ipc..."
sed -e "s:_enable_ipc_daemon = True:_enable_ipc_daemon =
False:" \
--
2.10.2
