You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@nuttx.apache.org by Sebastien Lorquet <se...@lorquet.fr> on 2025/01/31 10:29:23 UTC
Identified critical bug that breaks configure.sh
After git bisect over more than 2000 revisions, I found that this commit
broke configure.sh for my board
slo@slolin:~/Sources/ccv5-env/nuttx$ git bisect good
27685aa179ad7608394fb8a97e11ef532888a75d is the first bad commit
commit 27685aa179ad7608394fb8a97e11ef532888a75d
Author: yezhonghui <ye...@xiaomi.com>
Date: Tue Sep 24 20:05:27 2024 +0800
Fix config repeat define in include deconfig mode
Signed-off-by: yezhonghui <ye...@xiaomi.com>
tools/process_config.sh | 2 ++
1 file changed, 2 insertions(+)
Congratulations the-one-we-should-not-name
Now please fix it
../product/tools/configure-nuttx.sh
Copy files
sed: -e expression n°1, caractère 11: caractères inutiles après la commande
sed: -e expression n°1, caractère 16: commande inconnue: « O »
sed: -e expression n°1, caractère 17: caractères inutiles après la commande
sed: -e expression n°1, caractère 14: commande inconnue: « S »
sed: -e expression n°1, caractère 11: caractères inutiles après la commande
sed: -e expression n°1, caractère 13: commande inconnue: « O »
sed: -e expression n°1, caractère 17: commande inconnue: « O »
sed: -e expression n°1, caractère 11: commande inconnue: « m »
sed: -e expression n°1, caractère 14: commande inconnue: « Z »
sed: -e expression n°1, caractère 15: commande inconnue: « O »
Select CONFIG_HOST_LINUX=y
Sebastien
Re: Identified critical bug that breaks configure.sh
Posted by Sebastien Lorquet <se...@lorquet.fr>.
Reverting this does not seem to be a problem at all and fixes my problem.
Why was this added in the first place?
Maybe it increased some developer metric to show how productive they are.
We'll never know.
Sebastien
On 31/01/2025 11:29, Sebastien Lorquet wrote:
> After git bisect over more than 2000 revisions, I found that this
> commit broke configure.sh for my board
>
>
> slo@slolin:~/Sources/ccv5-env/nuttx$ git bisect good
> 27685aa179ad7608394fb8a97e11ef532888a75d is the first bad commit
> commit 27685aa179ad7608394fb8a97e11ef532888a75d
> Author: yezhonghui <ye...@xiaomi.com>
> Date: Tue Sep 24 20:05:27 2024 +0800
>
> Fix config repeat define in include deconfig mode
>
> Signed-off-by: yezhonghui <ye...@xiaomi.com>
>
> tools/process_config.sh | 2 ++
> 1 file changed, 2 insertions(+)
>
> Congratulations the-one-we-should-not-name
>
> Now please fix it
>
>
> ../product/tools/configure-nuttx.sh
> Copy files
> sed: -e expression n°1, caractère 11: caractères inutiles après la
> commande
> sed: -e expression n°1, caractère 16: commande inconnue: « O »
> sed: -e expression n°1, caractère 17: caractères inutiles après la
> commande
> sed: -e expression n°1, caractère 14: commande inconnue: « S »
> sed: -e expression n°1, caractère 11: caractères inutiles après la
> commande
> sed: -e expression n°1, caractère 13: commande inconnue: « O »
> sed: -e expression n°1, caractère 17: commande inconnue: « O »
> sed: -e expression n°1, caractère 11: commande inconnue: « m »
> sed: -e expression n°1, caractère 14: commande inconnue: « Z »
> sed: -e expression n°1, caractère 15: commande inconnue: « O »
> Select CONFIG_HOST_LINUX=y
>
>
> Sebastien
>
>
>
Re: Identified critical bug that breaks configure.sh
Posted by Tomek CEDRO <to...@cedro.info>.
On Fri, Jan 31, 2025 at 5:26 PM Sebastien Lorquet <se...@lorquet.fr> wrote:
> Dont loose too many time, reversing the commit already fixed the problem.
> Or I'll charge you my own time :)
> I think the two cancel anyway :)
> It's a stm32f429 cpu, so you have all the stm32 "common" board stuff
> inbetween.
> host is debian 12 or wsl.
> the problem happen just when running configure.sh, we've observed this
> on several developer machines.
> I think you should have a few custom boards on your build farm.
Unable to reproduce on WSL :-( I may try myself in a free moment
(really hard to find recently)..
--
CeDeROM, SQ7MHZ, http://www.tomek.cedro.info
Re: Identified critical bug that breaks configure.sh
Posted by Sebastien Lorquet <se...@lorquet.fr>.
Dont loose too many time, reversing the commit already fixed the problem.
Or I'll charge you my own time :)
I think the two cancel anyway :)
It's a stm32f429 cpu, so you have all the stm32 "common" board stuff
inbetween.
host is debian 12 or wsl.
the problem happen just when running configure.sh, we've observed this
on several developer machines.
I think you should have a few custom boards on your build farm.
Sebastien
On 31/01/2025 17:21, Tomek CEDRO wrote:
> Sebastien, please provide details on build configuration / how to
> reproduce, we cannot reproduce the problem.
>
> I think I will charge you hourly rate for playing the "deaf phone" as
> your secretary :D :D
>
> @cederom which defconfig is used? I try the master branch sim:nsh with
> sed 4.9 without problem:
>
> xiaoxiang@LAPTOP-MCMVB0JB:~/backup/os/nuttx/nuttx$ ./tools/configure.sh sim:nsh
> Copy files
> Select CONFIG_HOST_LINUX=y
> Refreshing...
> CP: arch/dummy/Kconfig to
> /home/xiaoxiang/backup/os/nuttx/nuttx/arch/dummy/dummy_kconfig
> CP: boards/dummy/Kconfig to
> /home/xiaoxiang/backup/os/nuttx/nuttx/boards/dummy/dummy_kconfig
> LN: platform/board to /home/xiaoxiang/backup/os/nuttx/apps/platform/dummy
> LN: include/arch to arch/sim/include
> LN: include/arch/board to
> /home/xiaoxiang/backup/os/nuttx/nuttx/boards/sim/sim/sim/include
> LN: drivers/platform to /home/xiaoxiang/backup/os/nuttx/nuttx/drivers/dummy
> LN: include/arch/chip to
> /home/xiaoxiang/backup/os/nuttx/nuttx/arch/sim/include/sim
> LN: arch/sim/src/chip to /home/xiaoxiang/backup/os/nuttx/nuttx/arch/sim/src/sim
> LN: arch/sim/src/board to
> /home/xiaoxiang/backup/os/nuttx/nuttx/boards/sim/sim/sim/src
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/audioutils
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/benchmarks
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/boot
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/canutils
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/crypto
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/database
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/examples/mcuboot
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/examples/module
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/examples/rust
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/examples/sotest
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/examples
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/fsutils
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/games
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/graphics
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/industry
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/inertial
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/interpreters/luamodules
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/interpreters
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/logging
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/lte
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/math
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/mlearning
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/netutils
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/sdr
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/system
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/arch
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/cxx
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/drivers
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/fs
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/libc
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/mm
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/sched
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/videoutils
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/wireless/bluetooth
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/wireless/ieee802154
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/wireless
> mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps
> Loaded configuration '.config'
> Configuration saved to '.config'
> xiaoxiang@LAPTOP-MCMVB0JB:~/backup/os/nuttx/nuttx$
> xiaoxiang@LAPTOP-MCMVB0JB:~/backup/os/nuttx/nuttx$ sed --version
> sed (GNU sed) 4.9-dirty
> Copyright (C) 2022 Free Software Foundation, Inc.
> License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
> This is free software: you are free to change and redistribute it.
> There is NO WARRANTY, to the extent permitted by law.
>
> Written by Jay Fenlason, Tom Lord, Ken Pizzini,
> Paolo Bonzini, Jim Meyering, and Assaf Gordon.
>
> This sed program was built with SELinux support.
> SELinux is disabled on this system.
>
> GNU sed home page: <https://www.gnu.org/software/sed/>.
> General help using GNU software: <https://www.gnu.org/gethelp/>.
> E-mail bug reports to: <bu...@gnu.org>.
>
>
>
Re: Identified critical bug that breaks configure.sh
Posted by Tomek CEDRO <to...@cedro.info>.
Sebastien, please provide details on build configuration / how to
reproduce, we cannot reproduce the problem.
I think I will charge you hourly rate for playing the "deaf phone" as
your secretary :D :D
@cederom which defconfig is used? I try the master branch sim:nsh with
sed 4.9 without problem:
xiaoxiang@LAPTOP-MCMVB0JB:~/backup/os/nuttx/nuttx$ ./tools/configure.sh sim:nsh
Copy files
Select CONFIG_HOST_LINUX=y
Refreshing...
CP: arch/dummy/Kconfig to
/home/xiaoxiang/backup/os/nuttx/nuttx/arch/dummy/dummy_kconfig
CP: boards/dummy/Kconfig to
/home/xiaoxiang/backup/os/nuttx/nuttx/boards/dummy/dummy_kconfig
LN: platform/board to /home/xiaoxiang/backup/os/nuttx/apps/platform/dummy
LN: include/arch to arch/sim/include
LN: include/arch/board to
/home/xiaoxiang/backup/os/nuttx/nuttx/boards/sim/sim/sim/include
LN: drivers/platform to /home/xiaoxiang/backup/os/nuttx/nuttx/drivers/dummy
LN: include/arch/chip to
/home/xiaoxiang/backup/os/nuttx/nuttx/arch/sim/include/sim
LN: arch/sim/src/chip to /home/xiaoxiang/backup/os/nuttx/nuttx/arch/sim/src/sim
LN: arch/sim/src/board to
/home/xiaoxiang/backup/os/nuttx/nuttx/boards/sim/sim/sim/src
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/audioutils
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/benchmarks
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/boot
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/canutils
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/crypto
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/database
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/examples/mcuboot
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/examples/module
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/examples/rust
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/examples/sotest
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/examples
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/fsutils
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/games
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/graphics
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/industry
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/inertial
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/interpreters/luamodules
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/interpreters
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/logging
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/lte
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/math
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/mlearning
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/netutils
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/sdr
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/system
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/arch
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/cxx
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/drivers
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/fs
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/libc
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/mm
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing/sched
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/testing
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/videoutils
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/wireless/bluetooth
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/wireless/ieee802154
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps/wireless
mkkconfig in /home/xiaoxiang/backup/os/nuttx/apps
Loaded configuration '.config'
Configuration saved to '.config'
xiaoxiang@LAPTOP-MCMVB0JB:~/backup/os/nuttx/nuttx$
xiaoxiang@LAPTOP-MCMVB0JB:~/backup/os/nuttx/nuttx$ sed --version
sed (GNU sed) 4.9-dirty
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Written by Jay Fenlason, Tom Lord, Ken Pizzini,
Paolo Bonzini, Jim Meyering, and Assaf Gordon.
This sed program was built with SELinux support.
SELinux is disabled on this system.
GNU sed home page: <https://www.gnu.org/software/sed/>.
General help using GNU software: <https://www.gnu.org/gethelp/>.
E-mail bug reports to: <bu...@gnu.org>.
--
CeDeROM, SQ7MHZ, http://www.tomek.cedro.info
Re: Identified critical bug that breaks configure.sh
Posted by Tomek CEDRO <to...@cedro.info>.
On Fri, Jan 31, 2025 at 3:10 PM Sebastien Lorquet <se...@lorquet.fr> wrote:
> Hi,
> On 31/01/2025 15:07, Tomek CEDRO wrote:
> > On Fri, Jan 31, 2025 at 2:54 PM Sebastien Lorquet<se...@lorquet.fr> wrote:
> >> On 31/01/2025 13:56, Tomek CEDRO wrote:
> >>> Can you please report that Issue on GitHub? 🙂
> >> I dont have github accounts anymore.
> > The mentioned commit was part of PR
> > https://github.com/apache/nuttx/pull/14552/files
> >
> > It changes tools/process_config.sh like this:
> >
> > else
> > local key_config="$(echo "$line" | cut -d= -f1)="
> > sed -i "/$key_config/d" $output_file
> > echo "$line" >> $output_file
> > fi
> > done < "$input_file"
> >
> > There is no `sed -e` invocation here. There is one on line 30 but that
> > was not changed. Also there is no tools/configure-nuttx.sh script in
> > our repo.
> slo@slolin:~/Sources/product-env/nuttx$ cat
> ../product/tools/configure-nuttx.sh
> tools/configure.sh -a ../product/apps/ ../ccv5/board/configs/net
>
> Simple as that.
>
> I reverted this commit in our local fork, so fix it or not, either works.
I have reported that problem for you on the mentioned PR. I will
forward information that reverting the commit fixes the issue.
> > Also I am replyting to your emails while I should be working too..
> > please create working gh account just for problems reporting that
> > would save time for everyone :-)
> >
> No.
If you are not part of the fix process then do not expect results :-(
--
CeDeROM, SQ7MHZ, http://www.tomek.cedro.info
Re: Identified critical bug that breaks configure.sh
Posted by Sebastien Lorquet <se...@lorquet.fr>.
Hi,
On 31/01/2025 15:07, Tomek CEDRO wrote:
> On Fri, Jan 31, 2025 at 2:54 PM Sebastien Lorquet<se...@lorquet.fr> wrote:
>> On 31/01/2025 13:56, Tomek CEDRO wrote:
>>> Can you please report that Issue on GitHub? 🙂
>> I dont have github accounts anymore.
> The mentioned commit was part of PR
> https://github.com/apache/nuttx/pull/14552/files
>
> It changes tools/process_config.sh like this:
>
> else
> local key_config="$(echo "$line" | cut -d= -f1)="
> sed -i "/$key_config/d" $output_file
> echo "$line" >> $output_file
> fi
> done < "$input_file"
>
> There is no `sed -e` invocation here. There is one on line 30 but that
> was not changed. Also there is no tools/configure-nuttx.sh script in
> our repo.
slo@slolin:~/Sources/product-env/nuttx$ cat
../product/tools/configure-nuttx.sh
tools/configure.sh -a ../product/apps/ ../ccv5/board/configs/net
Simple as that.
I reverted this commit in our local fork, so fix it or not, either works.
>
> Also I am replyting to your emails while I should be working too..
> please create working gh account just for problems reporting that
> would save time for everyone :-)
>
No.
Sebastien
Re: Identified critical bug that breaks configure.sh
Posted by Tomek CEDRO <to...@cedro.info>.
On Fri, Jan 31, 2025 at 2:54 PM Sebastien Lorquet <se...@lorquet.fr> wrote:
> On 31/01/2025 13:56, Tomek CEDRO wrote:
> > Can you please report that Issue on GitHub? 🙂
>
> I dont have github accounts anymore.
The mentioned commit was part of PR
https://github.com/apache/nuttx/pull/14552/files
It changes tools/process_config.sh like this:
else
local key_config="$(echo "$line" | cut -d= -f1)="
sed -i "/$key_config/d" $output_file
echo "$line" >> $output_file
fi
done < "$input_file"
There is no `sed -e` invocation here. There is one on line 30 but that
was not changed. Also there is no tools/configure-nuttx.sh script in
our repo.
Also I am replyting to your emails while I should be working too..
please create working gh account just for problems reporting that
would save time for everyone :-)
--
CeDeROM, SQ7MHZ, http://www.tomek.cedro.info
Re: Identified critical bug that breaks configure.sh
Posted by Sebastien Lorquet <se...@lorquet.fr>.
On 31/01/2025 13:56, Tomek CEDRO wrote:
> Can you please report that Issue on GitHub? 🙂
I dont have github accounts anymore.
It's a custom board, we're not shipping devboards in industrial access
control.
Sebastien
Re: Identified critical bug that breaks configure.sh
Posted by Tomek CEDRO <to...@cedro.info>.
On Fri, Jan 31, 2025 at 1:45 PM Sebastien Lorquet <se...@lorquet.fr> wrote:
> I have the default sed from debian 12
>
> slo@slolin:~/Sources/product-env/nuttx$ LANG=C sed --version
> sed (GNU sed) 4.9
> Packaged by Debian
> Copyright (C) 2022 Free Software Foundation, Inc.
> License GPLv3+: GNU GPL version 3 or later
> <https://gnu.org/licenses/gpl.html>.
> This is free software: you are free to change and redistribute it.
> There is NO WARRANTY, to the extent permitted by law.
>
> Written by Jay Fenlason, Tom Lord, Ken Pizzini,
> Paolo Bonzini, Jim Meyering, and Assaf Gordon.
>
> This sed program was built with SELinux support.
> SELinux is disabled on this system.
>
> GNU sed home page: <https://www.gnu.org/software/sed/>.
> General help using GNU software: <https://www.gnu.org/gethelp/>.
> E-mail bug reports to: <bu...@gnu.org>.
>
> (I hope it's not a fix that was required by a microsoft version of sed
> in a wsl so something cursed like that.)
gsed 4.9 here too, native build for FreeBSD:
% gsed --version
gsed (GNU sed) 4.9
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Written by Jay Fenlason, Tom Lord, Ken Pizzini,
Paolo Bonzini, Jim Meyering, and Assaf Gordon.
This sed program was built without SELinux support.
GNU sed home page: <https://www.gnu.org/software/sed/>.
General help using GNU software: <https://www.gnu.org/gethelp/>.
E-mail bug reports to: <bu...@gnu.org>.
% uname -a
FreeBSD XXX 14.2-RELEASE FreeBSD 14.2-RELEASE
releng/14.2-n269506-c8918d6c7412 GENERIC amd64
For Linuxish builds I have Debian 12 too deboostrapped into jails that
use native Linux ELF emulation on FreeBSD to I can just use Debian 12
userland running on FreeBSD :-)
What configuration do you use? I have that problem only when building
for sim on bare FreeBSD.
Can you please report that Issue on GitHub? :-)
--
CeDeROM, SQ7MHZ, http://www.tomek.cedro.info
Re: Identified critical bug that breaks configure.sh
Posted by Sebastien Lorquet <se...@lorquet.fr>.
Why, why should we suffer for trivial bugs like that? Life is hard enough.
I have the default sed from debian 12
slo@slolin:~/Sources/product-env/nuttx$ LANG=C sed --version
sed (GNU sed) 4.9
Packaged by Debian
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later
<https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Written by Jay Fenlason, Tom Lord, Ken Pizzini,
Paolo Bonzini, Jim Meyering, and Assaf Gordon.
This sed program was built with SELinux support.
SELinux is disabled on this system.
GNU sed home page: <https://www.gnu.org/software/sed/>.
General help using GNU software: <https://www.gnu.org/gethelp/>.
E-mail bug reports to: <bu...@gnu.org>.
(I hope it's not a fix that was required by a microsoft version of sed
in a wsl so something cursed like that.)
Sebastien
On 31/01/2025 13:23, Tomek CEDRO wrote:
> I have this problem on BSD too and need to replace with gsed (GNU Sed
> variant). Would be nice to fix :-)
> Tomek
>
> On Fri, Jan 31, 2025 at 11:30 AM Sebastien Lorquet<se...@lorquet.fr> wrote:
>> After git bisect over more than 2000 revisions, I found that this commit
>> broke configure.sh for my board
>>
>>
>> slo@slolin:~/Sources/product-env/nuttx$ git bisect good
>> 27685aa179ad7608394fb8a97e11ef532888a75d is the first bad commit
>> commit 27685aa179ad7608394fb8a97e11ef532888a75d
>> Author: yezhonghui<ye...@xiaomi.com>
>> Date: Tue Sep 24 20:05:27 2024 +0800
>>
>> Fix config repeat define in include deconfig mode
>>
>> Signed-off-by: yezhonghui<ye...@xiaomi.com>
>>
>> tools/process_config.sh | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> Congratulations the-one-we-should-not-name
>>
>> Now please fix it
>>
>>
>> ../product/tools/configure-nuttx.sh
>> Copy files
>> sed: -e expression n°1, caractère 11: caractères inutiles après la commande
>> sed: -e expression n°1, caractère 16: commande inconnue: « O »
>> sed: -e expression n°1, caractère 17: caractères inutiles après la commande
>> sed: -e expression n°1, caractère 14: commande inconnue: « S »
>> sed: -e expression n°1, caractère 11: caractères inutiles après la commande
>> sed: -e expression n°1, caractère 13: commande inconnue: « O »
>> sed: -e expression n°1, caractère 17: commande inconnue: « O »
>> sed: -e expression n°1, caractère 11: commande inconnue: « m »
>> sed: -e expression n°1, caractère 14: commande inconnue: « Z »
>> sed: -e expression n°1, caractère 15: commande inconnue: « O »
>> Select CONFIG_HOST_LINUX=y
>>
>>
>> Sebastien
>>
>>
>
Re: Identified critical bug that breaks configure.sh
Posted by Tomek CEDRO <to...@cedro.info>.
I have this problem on BSD too and need to replace with gsed (GNU Sed
variant). Would be nice to fix :-)
Tomek
On Fri, Jan 31, 2025 at 11:30 AM Sebastien Lorquet <se...@lorquet.fr> wrote:
>
> After git bisect over more than 2000 revisions, I found that this commit
> broke configure.sh for my board
>
>
> slo@slolin:~/Sources/ccv5-env/nuttx$ git bisect good
> 27685aa179ad7608394fb8a97e11ef532888a75d is the first bad commit
> commit 27685aa179ad7608394fb8a97e11ef532888a75d
> Author: yezhonghui <ye...@xiaomi.com>
> Date: Tue Sep 24 20:05:27 2024 +0800
>
> Fix config repeat define in include deconfig mode
>
> Signed-off-by: yezhonghui <ye...@xiaomi.com>
>
> tools/process_config.sh | 2 ++
> 1 file changed, 2 insertions(+)
>
> Congratulations the-one-we-should-not-name
>
> Now please fix it
>
>
> ../product/tools/configure-nuttx.sh
> Copy files
> sed: -e expression n°1, caractère 11: caractères inutiles après la commande
> sed: -e expression n°1, caractère 16: commande inconnue: « O »
> sed: -e expression n°1, caractère 17: caractères inutiles après la commande
> sed: -e expression n°1, caractère 14: commande inconnue: « S »
> sed: -e expression n°1, caractère 11: caractères inutiles après la commande
> sed: -e expression n°1, caractère 13: commande inconnue: « O »
> sed: -e expression n°1, caractère 17: commande inconnue: « O »
> sed: -e expression n°1, caractère 11: commande inconnue: « m »
> sed: -e expression n°1, caractère 14: commande inconnue: « Z »
> sed: -e expression n°1, caractère 15: commande inconnue: « O »
> Select CONFIG_HOST_LINUX=y
>
>
> Sebastien
>
>
--
CeDeROM, SQ7MHZ, http://www.tomek.cedro.info