Re: [gentoo-dev] [PATCH] mate.eclass: drop static-libs whenever possible

2020-08-02 Thread Adam Feldman
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

2020-08-02 Thread Mart Raudsepp
Ü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

2020-08-01 Thread Adam Feldman
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

2020-08-01 Thread Michał Górny
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

2020-08-01 Thread Adam Feldman
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