From: Adolf Belka <adolf.belka@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: Question about use of xxxMACHINExxx
Date: Mon, 08 Jul 2024 16:51:26 +0200 [thread overview]
Message-ID: <773ae7c0-d130-401f-b15a-d89f3b9b4cc4@ipfire.org> (raw)
In-Reply-To: <8A595DF1-45F7-449A-94A0-8A1864D411C6@ipfire.org>
[-- Attachment #1: Type: text/plain, Size: 2530 bytes --]
Hi Michael,
On 08/07/2024 15:59, Michael Tremer wrote:
> Hello Adolf,
>
> Sorry for not being faster at this… Still catching up :)
No problems.
>
> This is basically a variable which used to be called MACHINE but was replaced with xxxMACHINExxx because the string “MACHINE” was actually used in some filename. Not sure I am happy with that name…
>
> When the root files are generated, we don’t replace anything, but if we read a rootfile from config/rootfiles, we will replace this variable with the architecture we are building for. Sometimes this method allows us to have just a generic root file for all architectures, sometimes this does not work because there are extra files or something like that.
>
> So when you have such a case, you need to manually replace the architecture with the placeholder and check if all other architectures generate the same file.
I am not sure that I fully understand. In the repo there are rootfiles in config/rootfiles/common/aarch64/ such as boost, dtc, gcc which have aarch64 in the rootfile. None of these have been replaced by xxxMACGHINExxx.
In config/rootfiles/packages/aarch64/samba all the aarch64 references are still present. In /config/rootfiles/packages/x86_64/samba, all the references to x86_64 are replaced by xxxMACHINExxx as that is what I have been doing.
If the samba rootfile in aarch64 should have all its aarch64 references replaced by xxxMACHINExxx then that has not been happening and is also not the case for rootfiles in config/rootfiles/common/x86_64/ such as gcc, grub, rust etc
If the rootfiles that are in the x86_64 and aarch64 directories should have the architecture name replaced by xxxMACHINExxx then I need to do a v2 version of the samba update I just sent out earlier today as the x86_64 and aarch64 rootfiles do not have xxxMACHINExxx replacing the architecture reference.
Regards,
Adolf.
>
> -Michael
>
>> On 5 Jul 2024, at 11:39, Adolf Belka <adolf.belka(a)ipfire.org> wrote:
>>
>> Hi All,
>>
>> When a rootfile is in one of the specific architecture directories, e.g. aarch64 do all the aarch64 references just get left as that or should they get changed to xxxMACHINExxx.
>>
>> Looking at some files in the directories it looks like they get left alone but when I have done the samba rootfile update for x86_64 I have , most times, changed any x86_64 reference to xxxMACHINExxx.
>>
>> What is the correct approach so I don't go messing things up.
>>
>> Regards,
>>
>> Adolf
>>
>
next prev parent reply other threads:[~2024-07-08 14:51 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-05 10:39 Adolf Belka
2024-07-08 13:59 ` Michael Tremer
2024-07-08 14:51 ` Adolf Belka [this message]
2024-07-08 15:42 ` Michael Tremer
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=773ae7c0-d130-401f-b15a-d89f3b9b4cc4@ipfire.org \
--to=adolf.belka@ipfire.org \
--cc=development@lists.ipfire.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