| Message ID | 20240818090346.947914-1-thomas.petazzoni@bootlin.com |
|---|---|
| Headers | show |
| Series | Misc utils/genrandconfig improvements | expand |
Thomas, All, On 2024-08-18 11:03 +0200, Thomas Petazzoni via buildroot spake thusly: > Hello, > > My starting point for this series was that I wanted to create some > configurations that have BR2_BACKUP_SITE="", so that we do not use the > Buildroot backup mirror, to ensure that upstream locations are still > around. > > While implementing this, I realized that all the semi-random > configuration tweaks that we do in gen_config() are in fact completely > overridden by "make randconfig", when genrandconfig is used without > any toolchain CSV. Series applied to master, thanks. I did some changes on patch 2, so I'll reply there as well.. Regards, Yann E. MORIN. > So I thought it was time for a bit of clean-up: > > - Drop some fixups related to toolchains that are no longer tested > (patch 1) > > - Completely remove the support for using a toolchain CSV file. "make > randconfig" support has been around for quite a while, it gives good > results, so let's use that as the default now and only use fully > random configurations (patch 2) > > - Adapt the fine-tuning logic to the fact that "make randconfig" > overwrites everything: move the fine-tuning logic *after* "make > randconfig", but keep only what makes sense, since "make randconfig" > already randomizes everything (patch 3) > > - Implement what I wanted: set BR2_BACKUP_SITE="" for some builds > (patch 4) > > - Rework the logging of genrandconfig, as it is quite noisy right now, > but only with irrelevant messages (patch 5) > > Changes since v1: > > - Fix PATCH 3/5, which was breaking the randomness. Indeed the tuning > done *after* randconfig was done by overwriting the .config > entirely, instead of appending to it (open mode was "w+", while it > should have been "a") > > Let me know what you think. > > Thomas > > Thomas Petazzoni (5): > utils/genrandconfig: remove fixups related to untested CT-NG > toolchains > utils/genrandconfig: remove support for toolchain CSV > utils/genrandconfig: rework fine-tuning logic > utils/genrandconfig: do not use BR2_BACKUP_SITE for some builds > utils/genrandconfig: improve logging > > utils/genrandconfig | 268 ++++++++------------------------------------ > 1 file changed, 44 insertions(+), 224 deletions(-) > > -- > 2.46.0 > > _______________________________________________ > buildroot mailing list > buildroot@buildroot.org > https://lists.buildroot.org/mailman/listinfo/buildroot
Hello, My starting point for this series was that I wanted to create some configurations that have BR2_BACKUP_SITE="", so that we do not use the Buildroot backup mirror, to ensure that upstream locations are still around. While implementing this, I realized that all the semi-random configuration tweaks that we do in gen_config() are in fact completely overridden by "make randconfig", when genrandconfig is used without any toolchain CSV. So I thought it was time for a bit of clean-up: - Drop some fixups related to toolchains that are no longer tested (patch 1) - Completely remove the support for using a toolchain CSV file. "make randconfig" support has been around for quite a while, it gives good results, so let's use that as the default now and only use fully random configurations (patch 2) - Adapt the fine-tuning logic to the fact that "make randconfig" overwrites everything: move the fine-tuning logic *after* "make randconfig", but keep only what makes sense, since "make randconfig" already randomizes everything (patch 3) - Implement what I wanted: set BR2_BACKUP_SITE="" for some builds (patch 4) - Rework the logging of genrandconfig, as it is quite noisy right now, but only with irrelevant messages (patch 5) Changes since v1: - Fix PATCH 3/5, which was breaking the randomness. Indeed the tuning done *after* randconfig was done by overwriting the .config entirely, instead of appending to it (open mode was "w+", while it should have been "a") Let me know what you think. Thomas Thomas Petazzoni (5): utils/genrandconfig: remove fixups related to untested CT-NG toolchains utils/genrandconfig: remove support for toolchain CSV utils/genrandconfig: rework fine-tuning logic utils/genrandconfig: do not use BR2_BACKUP_SITE for some builds utils/genrandconfig: improve logging utils/genrandconfig | 268 ++++++++------------------------------------ 1 file changed, 44 insertions(+), 224 deletions(-)