Re: [gentoo-dev] [PATCH] toolchain.eclass: fix crossdev powerpc-*-musl builds

2021-12-26 Thread Sam James


> On 27 Dec 2021, at 05:24, Georgy Yakovlev  wrote:
> 
> otherwise initial build may fail with:
> unknown long double size, cannot define BFP_FMT

If possible, would you mind filing a bug with the build log
of it failing (and brief steps on how to get it) just so
we can easily test if we can drop it in future?

(We have a lot of hacks like this where I worry
we'll never be able to drop them with confidence)

If it takes a huge amount of work to get there, then don't
bother, as it's negligible gain.

> 
> Signed-off-by: Georgy Yakovlev 
> ---
> eclass/toolchain.eclass | 5 +
> 1 file changed, 5 insertions(+)
> 
> diff --git a/eclass/toolchain.eclass b/eclass/toolchain.eclass
> index fd03ba176276..1102c4fc5d56 100644
> --- a/eclass/toolchain.eclass
> +++ b/eclass/toolchain.eclass
> @@ -1099,6 +1099,11 @@ toolchain_src_configure() {
>   # Set up defaults based on current CFLAGS
>   is-flagq -mfloat-gprs=double && confgcc+=( --enable-e500-double 
> )
>   [[ ${CTARGET//_/-} == *-e500v2-* ]] && confgcc+=( 
> --enable-e500-double )
> + if [[ ${CTARGET} == powerpc-*-musl ]]; then
> + # fix: unknown long double size, cannot define BFP_FMT
> + confgcc+=( --disable-decimal-float )
> + export gcc_cv_target_ldbl128=no
> + fi
>   ;;
>   ppc64)
>   # On ppc64 big endian target gcc assumes elfv1 by default,
> --
> 2.34.1
> 
> 

Looks fine otherwise though.

best,
sam


signature.asc
Description: Message signed with OpenPGP


[gentoo-dev] [PATCH] toolchain.eclass: fix crossdev powerpc-*-musl builds

2021-12-26 Thread Georgy Yakovlev
otherwise initial build may fail with:
 unknown long double size, cannot define BFP_FMT

Signed-off-by: Georgy Yakovlev 
---
 eclass/toolchain.eclass | 5 +
 1 file changed, 5 insertions(+)

diff --git a/eclass/toolchain.eclass b/eclass/toolchain.eclass
index fd03ba176276..1102c4fc5d56 100644
--- a/eclass/toolchain.eclass
+++ b/eclass/toolchain.eclass
@@ -1099,6 +1099,11 @@ toolchain_src_configure() {
# Set up defaults based on current CFLAGS
is-flagq -mfloat-gprs=double && confgcc+=( --enable-e500-double 
)
[[ ${CTARGET//_/-} == *-e500v2-* ]] && confgcc+=( 
--enable-e500-double )
+   if [[ ${CTARGET} == powerpc-*-musl ]]; then
+   # fix: unknown long double size, cannot define BFP_FMT
+   confgcc+=( --disable-decimal-float )
+   export gcc_cv_target_ldbl128=no
+   fi
;;
ppc64)
# On ppc64 big endian target gcc assumes elfv1 by default,
-- 
2.34.1




[gentoo-dev] [PATCH] profiles/base/make.defaults: treat dtbs like modules

2021-12-26 Thread Georgy Yakovlev
when dist-kernel gets uninstalled, /lib/modules/
is left behind on purpose. However, on some systems kernels also install
device-tree files to /boot/dtbs/

leaving /lib/modules, but removing device-tree file may leave system
unbootable, so let's treat dtbs directory similar to modules.

Signed-off-by: Georgy Yakovlev 
---
 profiles/base/make.defaults | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/profiles/base/make.defaults b/profiles/base/make.defaults
index f25074f9d81a..be5651f0ac43 100644
--- a/profiles/base/make.defaults
+++ b/profiles/base/make.defaults
@@ -134,11 +134,11 @@ PYTHON_SINGLE_TARGET="python3_9"
 
 # Michał Górny  (2013-08-10)
 # Moved from portage's make.globals.
-# 1) do not uninstall kernel modules and therefore allow replacing them,
+# 1) do not uninstall kernel modules/dtbs and therefore allow replacing them,
 # 2,3) removed wrt bug #663170,
 # 4) protect /var/{run,lock} symlinks for bug 519620
-COLLISION_IGNORE="/lib/modules/*"
-UNINSTALL_IGNORE="/lib/modules/* /var/run /var/lock"
+COLLISION_IGNORE="/boot/dtbs/* /lib/modules/*"
+UNINSTALL_IGNORE="/boot/dtbs/* /lib/modules/* /var/run /var/lock"
 
 # Andreas K. Hüttel  (2013-08-23)
 # Make emerge messages default to English as per Council decision
-- 
2.34.1




Re: [gentoo-dev] [PATCH 1/5] gnome2-utils.eclass: phase out emktemp

2021-12-26 Thread Mart Raudsepp
Ühel kenal päeval, P, 26.12.2021 kell 10:50, kirjutas Michał Górny:
> On Sun, 2021-12-26 at 09:44 +, Marek Szuba wrote:
> > 
> > On 13 December 2021 17:24:18 UTC, Mart Raudsepp 
> > wrote:
> > 
> > > Actually I kind of preferred a static name over straight mktemp,
> > > because emktemp supported other cases than a pure mktemp usage
> > > does.
> > > But I don't know if it could ever clash things in some weird
> > > situations.
> > 
> > That last part is the message I tried to convey in my e-mail, sorry
> > if I wasn't clear enough.
> > 
> > Anyway, could anyone with more Postage/PMS experience weigh in on
> > this? If it is indeed safe then the eclass could be modified
> > further, e.g. to use static names with EAPI-8+ but stick with
> > mktemp for older EAPIs just to be safe.
> 
> I suppose it's not specified strictly but T should be safe for all
> sane
> uses.  If it weren't, we'd already be in deep trouble and gnome2-
> utils
> would be the least of our concerns.
> 
> That said, making this EAPI-conditional is just an unnecessary
> complexity.

It's already hardcoded to $T via using mktemp instead of emktemp (which
supported lack of $T or mktemp utility, unlike the replacement that was
merged) - so if $T and mktemp is guaranteed, we are good there. It was
introduced with mktemp in the first place and then vapier changed it to
emktemp without any reason given beyond probably just doing it for
everything to support lack of mktemp on the system or some such.


Now the question was, can we hardcode the name instead - is it possible
for it to be running things in parallel and clash on the name, e.g.:

* postinst and postrm ran at the same time on replacement: is it doing
._unmerge dir in portage an implementation detail that we shouldn't
rely on it?

* postinst and/or postrm ran with multilib support: could they be
running in parallel in the future for the different ABIs -- should we
include the ABI in the static filename at least to avoid clashes there?


Mart


signature.asc
Description: This is a digitally signed message part


Re: [gentoo-dev] USE-flag gnome-keyring isn't accurate anymore

2021-12-26 Thread John Johnson
"keyring" conveys the appropriate meaning, imho.

On Sat, Dec 25, 2021, 5:19 AM Marek Szuba  wrote:

>
>
> On 24 December 2021 08:48:08 UTC, Pacho Ramos  wrote:
>
> >> > I think “secret” may be too generic and “libsecret” is not ideal in
> case
> >> > an implemention comes along that is named differently. How about
> >> > “secret-service”?
> >>
> >> I think this is a good idea.
> >>
> >
> >And "keyring"? I am not sure if users not familiar with "libsecret" will
> >understand what "secret*" means in this context
>
> Definitely a good idea. And I second "keyring", seeing as this term is
> also in use on other OSes.
>
> --
> Marecki
>
>


Re: [gentoo-dev] [PATCH 1/5] gnome2-utils.eclass: phase out emktemp

2021-12-26 Thread Michał Górny
On Sun, 2021-12-26 at 09:44 +, Marek Szuba wrote:
> 
> On 13 December 2021 17:24:18 UTC, Mart Raudsepp  wrote:
> 
> > Actually I kind of preferred a static name over straight mktemp,
> > because emktemp supported other cases than a pure mktemp usage does.
> > But I don't know if it could ever clash things in some weird
> > situations.
> 
> That last part is the message I tried to convey in my e-mail, sorry if I 
> wasn't clear enough.
> 
> Anyway, could anyone with more Postage/PMS experience weigh in on this? If it 
> is indeed safe then the eclass could be modified further, e.g. to use static 
> names with EAPI-8+ but stick with mktemp for older EAPIs just to be safe.

I suppose it's not specified strictly but T should be safe for all sane
uses.  If it weren't, we'd already be in deep trouble and gnome2-utils
would be the least of our concerns.

That said, making this EAPI-conditional is just an unnecessary
complexity.

-- 
Best regards,
Michał Górny




Re: [gentoo-dev] [PATCH 1/5] gnome2-utils.eclass: phase out emktemp

2021-12-26 Thread Marek Szuba



On 13 December 2021 17:24:18 UTC, Mart Raudsepp  wrote:

>Actually I kind of preferred a static name over straight mktemp,
>because emktemp supported other cases than a pure mktemp usage does.
>But I don't know if it could ever clash things in some weird
>situations.

That last part is the message I tried to convey in my e-mail, sorry if I wasn't 
clear enough.

Anyway, could anyone with more Postage/PMS experience weigh in on this? If it 
is indeed safe then the eclass could be modified further, e.g. to use static 
names with EAPI-8+ but stick with mktemp for older EAPIs just to be safe.

-- 
Marecki