public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: "rajmohan r" <semc.2042@gmail.com>
To: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH] glibc-y2038-tests:add tests-special in run-built-tests yes
Date: Thu, 24 Apr 2025 07:50:50 -0700	[thread overview]
Message-ID: <28087.1745506250655631551@lists.openembedded.org> (raw)
In-Reply-To: <CANNYZj8ineDyXv8pNgb3GkTJ2RFFm=md5-JiPRo2TxbouP9PPg@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1564 bytes --]

On Tue, Apr 22, 2025 at 03:52 PM, Alexander Kanavin wrote:

> 
> On Tue, 22 Apr 2025 at 13:03, rajmohan r via lists.openembedded.org
> <semc.2042=gmail.com@lists.openembedded.org> wrote:
> 
>> On Thu, Apr 10, 2025 at 07:34 PM, Alexander Kanavin wrote:
>> 
>> Thanks. This patch is invasive and adds to glibc maintenance burden.
>> Which begs the question: do we need the recipe at all? Is there a
>> particular reason you started looking into this issue?
>> 
>> When we moved from yocto kirkstone branch to scarthgap branch, we found
>> that
>> compilation for this module taking longer time. hence started looking
>> which stage
>> in this module takes time. Found that it is in do_check() stage.
>> 
>> 
>> It is by now well established that glibc itself works as it should,
>> that all affected 32 bit targets are configured to use 64 bit time_t,
>> and that any lingering y2038 issues are in components other than the c
>> library, and usually come from C programming mistakes (e.g. storing
>> timestamps in long). Maybe we can simply remove the recipe?
>> 
>> This recipe gives only two test cases in the packages to test. below are
>> the
>> test cases seen for this package
>> io/ftwtest
>> io/ftwtest-time64
> 
> Right. I propose that this recipe be altogether removed (with the
> rationale I provided above which you can add to the commit message).
> Can you send a patch for that?

Patch is send here https://lists.openembedded.org/g/openembedded-core/message/215381 please review. Thanks for reviewing.

> 
> Alex

[-- Attachment #2: Type: text/html, Size: 1777 bytes --]

      reply	other threads:[~2025-04-24 14:50 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-10 10:07 [PATCH] glibc-y2038-tests:add tests-special in run-built-tests yes rajmohan r
2025-04-10 19:33 ` [OE-core] " Alexander Kanavin
2025-04-22 11:03   ` rajmohan r
2025-04-22 15:52     ` [OE-core] " Alexander Kanavin
2025-04-24 14:50       ` rajmohan r [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=28087.1745506250655631551@lists.openembedded.org \
    --to=semc.2042@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox