Re: [gentoo-dev] [PATCH] mate.eclass: drop static-libs whenever possible
On 8/2/20 3:56 AM, Mart Raudsepp wrote:
> Ühel kenal päeval, L, 01.08.2020 kell 14:32, kirjutas Adam Feldman:
>> + gnome2_src_configure ${mateconf[@]} "$@"
>
> This might be something we may want to consider adding to gnome2.eclass
> instead?
> But I guess that's something to consider for any EAPI-7 port instead,
> should one eventually happen (instead of everything moving to meson
> instead).
> So no objections adding it to mate.eclass for the time being.
> gnome2.eclass would need too much checking that it's fine to bother at
> this point without the EAPI bump. But remind about it, should a EAPI-7
> gnome2 ever go up for review here :)
>
>
> Mart
>
Would you like a patch for gnome2.eclass now with a:
if has ${EAPI:-0} 7; then
fi
block so you'll be set to go upon updating the eclass to EAPI 7?
--
Thanks,
Adam Feldman
Gentoo Developer
np-hard...@gentoo.org
0x671C52F118F89C67
signature.asc
Description: OpenPGP digital signature
Re: [gentoo-dev] [PATCH] mate.eclass: drop static-libs whenever possible
Ühel kenal päeval, L, 01.08.2020 kell 14:32, kirjutas Adam Feldman:
> + gnome2_src_configure ${mateconf[@]} "$@"
This might be something we may want to consider adding to gnome2.eclass
instead?
But I guess that's something to consider for any EAPI-7 port instead,
should one eventually happen (instead of everything moving to meson
instead).
So no objections adding it to mate.eclass for the time being.
gnome2.eclass would need too much checking that it's fine to bother at
this point without the EAPI bump. But remind about it, should a EAPI-7
gnome2 ever go up for review here :)
Mart
signature.asc
Description: This is a digitally signed message part
Re: [gentoo-dev] [PATCH] mate.eclass: drop static-libs whenever possible
On 8/1/20 4:25 PM, Michał Górny wrote:
> On Sat, 2020-08-01 at 14:32 -0400, Adam Feldman wrote:
>> Whenever package doesn't have static-libs available and selected,
>> if configure script has static-libs selectable, explicitly disable
>>
>> Bug: https://bugs.gentoo.org/695878
>> ---
>> eclass/mate.eclass | 12 +++-
>> 1 file changed, 11 insertions(+), 1 deletion(-)
>>
>> diff --git a/eclass/mate.eclass b/eclass/mate.eclass
>> index db511aedb1f1..d6718872c05d 100644
>> --- a/eclass/mate.eclass
>> +++ b/eclass/mate.eclass
>> @@ -115,7 +115,17 @@ mate_src_prepare() {
>> # MATE specific configure handling
>> # Stub to gnome2_src_configure()
>> mate_src_configure() {
>> -gnome2_src_configure "$@"
>> +
>> +local mateconf=()
>> +
>> +# Pass --disable-static whenever possible
>> +if ! use_if_iuse static-libs; then
>> +if grep -q "enable-static" "${ECONF_SOURCE:-.}"/configure; then
>> +mateconf+=( --disable-static )
>> +fi
>> +fi
>> +
>> +gnome2_src_configure ${mateconf[@]} "$@"
>
> "${mateconf[@]}". Using [@] without quoting makes no sense, and I'm
> sure linters will complain.
>
>> }
>>
>> # @FUNCTION: mate_src_install
>
Fixed. Thanks for the review.
--
Thanks,
Adam Feldman
Gentoo Developer
np-hard...@gentoo.org
0x671C52F118F89C67
signature.asc
Description: OpenPGP digital signature
Re: [gentoo-dev] [PATCH] mate.eclass: drop static-libs whenever possible
On Sat, 2020-08-01 at 14:32 -0400, Adam Feldman wrote:
> Whenever package doesn't have static-libs available and selected,
> if configure script has static-libs selectable, explicitly disable
>
> Bug: https://bugs.gentoo.org/695878
> ---
> eclass/mate.eclass | 12 +++-
> 1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/eclass/mate.eclass b/eclass/mate.eclass
> index db511aedb1f1..d6718872c05d 100644
> --- a/eclass/mate.eclass
> +++ b/eclass/mate.eclass
> @@ -115,7 +115,17 @@ mate_src_prepare() {
> # MATE specific configure handling
> # Stub to gnome2_src_configure()
> mate_src_configure() {
> - gnome2_src_configure "$@"
> +
> + local mateconf=()
> +
> + # Pass --disable-static whenever possible
> + if ! use_if_iuse static-libs; then
> + if grep -q "enable-static" "${ECONF_SOURCE:-.}"/configure; then
> + mateconf+=( --disable-static )
> + fi
> + fi
> +
> + gnome2_src_configure ${mateconf[@]} "$@"
"${mateconf[@]}". Using [@] without quoting makes no sense, and I'm
sure linters will complain.
> }
>
> # @FUNCTION: mate_src_install
--
Best regards,
Michał Górny
signature.asc
Description: This is a digitally signed message part
[gentoo-dev] [PATCH] mate.eclass: drop static-libs whenever possible
Whenever package doesn't have static-libs available and selected,
if configure script has static-libs selectable, explicitly disable
Bug: https://bugs.gentoo.org/695878
---
eclass/mate.eclass | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/eclass/mate.eclass b/eclass/mate.eclass
index db511aedb1f1..d6718872c05d 100644
--- a/eclass/mate.eclass
+++ b/eclass/mate.eclass
@@ -115,7 +115,17 @@ mate_src_prepare() {
# MATE specific configure handling
# Stub to gnome2_src_configure()
mate_src_configure() {
- gnome2_src_configure "$@"
+
+ local mateconf=()
+
+ # Pass --disable-static whenever possible
+ if ! use_if_iuse static-libs; then
+ if grep -q "enable-static" "${ECONF_SOURCE:-.}"/configure; then
+ mateconf+=( --disable-static )
+ fi
+ fi
+
+ gnome2_src_configure ${mateconf[@]} "$@"
}
# @FUNCTION: mate_src_install
--
2.26.2
