diff mbox series

nativesdk-gzip: fix reproducibility issues

Message ID 20240411133255.3217812-1-omatiush@cisco.com
State Accepted, archived
Commit fc18ff491111c22c3372c6de21d37945906d2287
Headers show
Series nativesdk-gzip: fix reproducibility issues | expand

Commit Message

There is absolute path in zgrep binary. It is related
to the changes that fix an issue with zgrep on Solaris
and other platforms with 'grep -e'. I'd like to ask here
if Yocto supports Solaris as host architecture, and
if there's a reliable way to detect the host arch.
If there is, I'll place additional check to be
compatible with it.

Original change:
git.savannah.gnu.org/cgit/gzip.git/commit?id=60b3f4eb672a85de28ecaad47ed426f8c8fe6c32

Signed-off-by: Oleh Matiusha <omatiush@cisco.com>
---
 meta/recipes-extended/gzip/gzip.inc | 1 +
 1 file changed, 1 insertion(+)

Comments

Khem Raj April 11, 2024, 6:41 p.m. UTC | #1
On Thu, Apr 11, 2024 at 6:32 AM Oleh Matiusha via
lists.openembedded.org <omatiush=cisco.com@lists.openembedded.org>
wrote:
>
> There is absolute path in zgrep binary. It is related
> to the changes that fix an issue with zgrep on Solaris
> and other platforms with 'grep -e'. I'd like to ask here
> if Yocto supports Solaris as host architecture, and
> if there's a reliable way to detect the host arch.
> If there is, I'll place additional check to be
> compatible with it.

We do not have support for solaris build hosts.

>
> Original change:
> git.savannah.gnu.org/cgit/gzip.git/commit?id=60b3f4eb672a85de28ecaad47ed426f8c8fe6c32
>
> Signed-off-by: Oleh Matiusha <omatiush@cisco.com>
> ---
>  meta/recipes-extended/gzip/gzip.inc | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/meta/recipes-extended/gzip/gzip.inc b/meta/recipes-extended/gzip/gzip.inc
> index b32584033b..62b3e2f4f9 100644
> --- a/meta/recipes-extended/gzip/gzip.inc
> +++ b/meta/recipes-extended/gzip/gzip.inc
> @@ -8,6 +8,7 @@ inherit autotools texinfo
>  export DEFS="NO_ASM"
>
>  EXTRA_OEMAKE:class-target = "GREP=${base_bindir}/grep"
> +EXTRA_OEMAKE:append:class-nativesdk = " GREP=grep"
>  EXTRA_OECONF:append:libc-musl = " gl_cv_func_fflush_stdin=yes "
>
>  do_install:append () {
> --
> 2.33.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#198130): https://lists.openembedded.org/g/openembedded-core/message/198130
> Mute This Topic: https://lists.openembedded.org/mt/105462485/1997914
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Can it be accepted in upstream then?
Richard Purdie April 12, 2024, 9:35 a.m. UTC | #3
On Thu, 2024-04-11 at 13:32 +0000, Oleh Matiusha via
lists.openembedded.org wrote:
> There is absolute path in zgrep binary. It is related
> to the changes that fix an issue with zgrep on Solaris
> and other platforms with 'grep -e'. I'd like to ask here
> if Yocto supports Solaris as host architecture, and
> if there's a reliable way to detect the host arch.
> If there is, I'll place additional check to be
> compatible with it.

How far did you get builds to work on Solaris out of interest?

We don't support it since none of us have access to it and it is
something we've never really been asked about before.

We did used to add BUILD_OS to OVERRIDES so that you could do build
system overrides to variables. It was so rarely used we ended up
dropping it though.

I'm not against the idea of Solaris support but I'd want to understand
how invasive it is and it may be better starting as a separate layer.
That layer could append to OVERRIDES to make the metadata easier for
example.

Also, is this running builds on Solaris, or is this building SDKs to
run on Solaris. I suddenly realised this could be taken both ways.

Cheers,

Richard
​How far did you get builds to work on Solaris out of interest?


Perhaps you have misunderstood me. I have neither the desire nor the hardware to work on providing Solaris support, but I asked if Yocto supports or plans to support Solaris to see if my patch is compatible with the current version.
Richard Purdie April 12, 2024, 3:40 p.m. UTC | #5
On Fri, 2024-04-12 at 14:55 +0000, Oleh Matiusha -X (omatiush -
GLOBALLOGIC INC at Cisco) wrote:
> > ​How far did you get builds to work on Solaris out of interest?
> 
> 
> Perhaps you have misunderstood me. I have neither the desire nor the
> hardware to work on providing Solaris support, but I asked if Yocto
> supports or plans to support Solaris to see if my patch is compatible
> with the current version.

Definitely misunderstanding! We have no support or plans for support
for Solaris.

Cheers,

Richard
diff mbox series

Patch

diff --git a/meta/recipes-extended/gzip/gzip.inc b/meta/recipes-extended/gzip/gzip.inc
index b32584033b..62b3e2f4f9 100644
--- a/meta/recipes-extended/gzip/gzip.inc
+++ b/meta/recipes-extended/gzip/gzip.inc
@@ -8,6 +8,7 @@  inherit autotools texinfo
 export DEFS="NO_ASM"
 
 EXTRA_OEMAKE:class-target = "GREP=${base_bindir}/grep"
+EXTRA_OEMAKE:append:class-nativesdk = " GREP=grep"
 EXTRA_OECONF:append:libc-musl = " gl_cv_func_fflush_stdin=yes "
 
 do_install:append () {