Re: 回复： 回复： env.Textfi le problem

Hello liruncong,

I created a feature request issue for Textfile builder to automatically
translate to ninja: https://github.com/SCons/scons/issues/4156

On Mon, May 23, 2022 at 12:39 PM Bill Deegan <[email protected]>
wrote:

> ninja builder is marked as experimental.
> So one should not expect this to be fully functional, or simple and easy
> to use.
>
> On Sun, May 22, 2022 at 11:47 PM liruncong via Scons-users <
> [email protected]> wrote:
>
>> Hello Daniel Moody,
>> Thank you!
>> But in the future, the difficulty for users should be reduced, and it is
>> difficult for users to solve these problems by themselves!
>>
>>
>> ------------------ 原始邮件 ------------------
>> *发件人:* "Daniel Moody" <[email protected]>;
>> *发送时间:* 2022年5月23日(星期一) 中午1:53
>> *收件人:* "liruncong"<[email protected]>;
>> *抄送:* "scons-users"<[email protected]>;
>> *主题:* Re: [Scons-users] 回复： env.Textfile problem
>>
>> Hello liruncong,
>>
>> For issue number 1:
>> with scons only, I ran the build with --debug=explain and the output
>> stated the order of the object files is changing every build. I fixed it
>> with a sorted list like this:
>> @@ -816,7 +816,7 @@ def DoBuilding(target, objects):
>>          for group in Projects:
>>              local_group(group, objects)
>>
>> -        program = Env.Program(target, objects)
>> +        program = Env.Program(target, sorted(objects))
>>
>>      EndBuilding(target, program)
>>
>> For issue number 2:
>> this is unavoidable. As I explained, ninja can not know if those objects
>> need to be rebuilt because only scons can know this, so it must call to
>> scons to check. This is the output you will see, ninja is asking scons if
>> these files need to be built again because of some scons change. Ninja can
>> not know this, so it must always ask. SCons will see that the files do not
>> need to change and it should not touch the files if they are already built.
>> Ninja will see after it is done asking scons that the files have not been
>> touched also and it will know not to rebuild any files which depended on
>> those files which it asked scons to build.
>>
>> Alternatively, you do not need scons to build these files directly. If
>> you convert the creation of the files to a command that ninja can execute
>> on the command line then ninja will not need to callback to scons. I showed
>> an example in the PR: https://github.com/liruncong/NinJaTest/pull/2
>>
>> For issue number 3:
>> This is expected behavior, ninja "-n" option is dry-run which means it
>> won't build anything, just test graph execution. I was using "-n" as an
>> example of passing a ninja argument to ninja from scons.
>>
>>
>> On Wed, May 18, 2022 at 7:01 AM liruncong2018 <[email protected]>
>> wrote:
>>
>>> Hi,
>>> After updated,
>>> 1) When ninja is not enabled, the last link is always executed when the
>>> compilation is repeated.
>>>     log:
>>> ___________________________________________
>>> scons: Reading SConscript files ...
>>> Newlib version:2.4.0
>>> scons: done reading SConscript files.
>>> scons: Building targets ...
>>> scons: building associated VariantDir targets: build
>>> arm-none-eabi-gcc -o rtthread-beaglebone.elf -Wall -march=armv7-a
>>> -mtune=cortex-a8 -ftree-vectorize -ffast-math -mfpu=vfpv3-d16
>>> -mfloat-abi=softfp
>>> -Wl,--gc-sections,-Map=rtthread-beaglebone.map,-cref,-u,Reset_Handler -T
>>> beaglebone_ram.lds -lm -lc @build/__link_objects.txt
>>> scons: done building targets.
>>> ___________________________________________
>>> 2) When ninja is enabled, when recompiling, it always prints something
>>> that doesn't seem to actually execute the steps
>>>     log:
>>> ___________________________________________
>>> [1/56] Starting scons daemon...[2K
>>> [2/56] Defer to SCons to build build\__cpp_path.txt[2K
>>> [3/56] Defer to SCons to build build\__define_options.txt[2K
>>> [4/56] Defer to SCons to build build\__link_objects.txt
>>> scons: done building targets.
>>> ___________________________________________
>>> Judging from the file time, these txt
>>> files(__cpp_path.txt/__define_options.txt/__link_objects.txt) are not
>>> actually generated.
>>>
>>> The following is the log using NINJA_CMD_ARGS, printing the same message
>>> every time:
>>> ___________________________________________
>>> D:\work\gitee\rt-thread_ninja\bsp\beaglebone>scons
>>> -j%NUMBER_OF_PROCESSORS% --implicit-cache NINJA_CMD_ARGS="-n" --verbose
>>> --experimental=ninja
>>> scons: Reading SConscript files ...
>>> Newlib version:2.4.0
>>> scons: done reading SConscript files.
>>> scons: Building targets ...
>>> Generating: build.ninja...
>>> No updates detected, build.ninja not regenerated.
>>> Executing: run_ninja_env.bat -n
>>> D:\work\gitee\rt-thread_ninja\bsp\beaglebone>set SystemDrive=C:
>>> D:\work\gitee\rt-thread_ninja\bsp\beaglebone>set SystemRoot=C:\Windows
>>> D:\work\gitee\rt-thread_ninja\bsp\beaglebone>set
>>> TEMP=C:\Users\ARCHYI~1\AppData\Local\Temp
>>> D:\work\gitee\rt-thread_ninja\bsp\beaglebone>set
>>> TMP=C:\Users\ARCHYI~1\AppData\Local\Temp
>>> D:\work\gitee\rt-thread_ninja\bsp\beaglebone>set
>>> USERPROFILE=C:\Users\archyizimi
>>> D:\work\gitee\rt-thread_ninja\bsp\beaglebone>set
>>> COMSPEC=C:\Windows\system32\cmd.exe
>>> D:\work\gitee\rt-thread_ninja\bsp\beaglebone>set
>>> PATH=D:\MYBIN\env\tools\gnu_gcc\arm_gcc\mingw\bin;D:\MYBIN\scons-local-4.3.1\;D:\MYBIN;D:\MYBIN\Anaconda3\;C:\Program
>>> Files\Microsoft Visual Studio\2022\Enterprise\VC\Tools\Llvm\bin;C:\Program
>>> Files\Microsoft Visual
>>> Studio\2022\Enterprise\VC\Tools\MSVC\14.32.31326\bin\HostX86\x86;C:\Program
>>> Files\Microsoft Visual
>>> Studio\2022\Enterprise\Common7\IDE\VC\VCPackages;C:\Program Files\Microsoft
>>> Visual
>>> Studio\2022\Enterprise\Common7\IDE\CommonExtensions\Microsoft\TestWindow;C:\Program
>>> Files\Microsoft Visual
>>> Studio\2022\Enterprise\Common7\IDE\CommonExtensions\Microsoft\TeamFoundation\Team
>>> Explorer;C:\Program Files\Microsoft Visual
>>> Studio\2022\Enterprise\MSBuild\Current\bin\Roslyn;C:\Program
>>> Files\Microsoft Visual Studio\2022\Enterprise\Team Tools\Performance
>>> Tools;C:\Program Files (x86)\Microsoft SDKs\Windows\v10.0A\bin\NETFX 4.8
>>> Tools\;C:\Program Files (x86)\Windows
>>> Kits\10\bin\10.0.19041.0\\x86;C:\Program Files (x86)\Windows
>>> Kits\10\bin\\x86;C:\Program Files\Microsoft Visual
>>> Studio\2022\Enterprise\\MSBuild\Current\Bin\amd64;C:\Windows\Microsoft.NET\Framework\v4.0.30319;C:\Program
>>> Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\;C:\Program
>>> Files\Microsoft Visual
>>> Studio\2022\Enterprise\Common7\Tools\;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Windows\System32\OpenSSH\;C:\Program
>>> Files\Microsoft SQL Server\150\Tools\Binn\;C:\Program Files\IDM Computer
>>> Solutions\UltraEdit;C:\Program Files\IDM Computer
>>> Solutions\UltraCompare;C:\Program Files\TortoiseGit\bin;C:\Program
>>> Files\dotnet\;C:\Program
>>> Files\Git\cmd;C:\Users\archyizimi\AppData\Local\Microsoft\WindowsApps;C:\Users\archyizimi\.dotnet\tools;C:\Program
>>> Files\Microsoft Visual
>>> Studio\2022\Enterprise\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin;C:\Program
>>> Files\Microsoft Visual
>>> Studio\2022\Enterprise\Common7\IDE\CommonExtensions\Microsoft\CMake\Ninja;C:\Program
>>> Files\Microsoft Visual
>>> Studio\2022\Enterprise\Common7\IDE\VC\Linux\bin\ConnectionManagerExe;c:\MinGW\bin
>>> D:\work\gitee\rt-thread_ninja\bsp\beaglebone>set
>>> PATHEXT=.COM;.EXE;.BAT;.CMD
>>> D:\work\gitee\rt-thread_ninja\bsp\beaglebone>D:\MYBIN\Anaconda3\lib\site-packages\ninja\data\bin\ninja.exe
>>> -f D:\work\gitee\rt-thread_ninja\bsp\beaglebone\build.ninja -n
>>> [1/56] Starting scons daemon... [2K
>>> [2/56] Defer to SCons to build build\__cpp_path.txt [2K
>>> [3/56] Defer to SCons to build build\__link_objects.txt [2K
>>> [4/56] Defer to SCons to build build\__define_options.txt [2K
>>> [5/56] Compiling build\applications\board.o [2K
>>> [6/56] Compiling build\applications\main.o [2K
>>> [7/56] Compiling build\drivers\gpio.o [2K
>>> [8/56] Compiling build\kernel\components\dfs\src\dfs_posix.o [2K
>>> [9/56] Compiling build\kernel\components\dfs\src\dfs_file.o [2K
>>> [10/56] Compiling build\drivers\uart.o [2K
>>> [11/56] Compiling build\kernel\components\dfs\filesystems\devfs\devfs.o
>>> [2K
>>> [12/56] Compiling build\kernel\components\dfs\src\dfs.o [2K
>>> [13/56] Compiling build\kernel\components\dfs\src\dfs_fs.o [2K
>>> [14/56] Compiling build\kernel\components\drivers\ipc\pipe.o [2K
>>> [15/56] Compiling build\kernel\components\drivers\ipc\completion.o [2K
>>> [16/56] Compiling build\kernel\components\drivers\ipc\dataqueue.o [2K
>>> [17/56] Compiling build\kernel\components\drivers\ipc\ringbuffer.o [2K
>>> [18/56] Compiling build\kernel\components\drivers\ipc\ringblk_buf.o [2K
>>> [19/56] Compiling build\kernel\components\drivers\ipc\waitqueue.o [2K
>>> [20/56] Compiling build\kernel\components\finsh\msh.o [2K
>>> [21/56] Compiling build\kernel\components\drivers\serial\serial.o [2K
>>> [22/56] Compiling build\kernel\components\drivers\ipc\workqueue.o [2K
>>> [23/56] Compiling build\kernel\components\drivers\misc\pin.o [2K
>>> [24/56] Compiling build\kernel\components\finsh\cmd.o [2K
>>> [25/56] Compiling build\kernel\components\libc\compilers\common\time.o
>>> [2K
>>> [26/56] Compiling build\kernel\components\finsh\shell.o [2K
>>> [27/56] Compiling build\kernel\components\libc\compilers\common\stdlib.o
>>> [2K
>>> [28/56] Building build\kernel\libcpu\arm\am335x\context_gcc.o [2K
>>> [29/56] Compiling
>>> build\kernel\components\libc\compilers\newlib\syscalls.o [2K
>>> [30/56] Compiling build\kernel\components\finsh\msh_file.o [2K
>>> [31/56] Building build\kernel\libcpu\arm\am335x\start_gcc.o [2K
>>> [32/56] Building build\kernel\libcpu\arm\am335x\cp15_gcc.o [2K
>>> [33/56] Compiling build\kernel\libcpu\arm\am335x\interrupt.o [2K
>>> [34/56] Compiling build\kernel\libcpu\arm\am335x\cpu.o [2K
>>> [35/56] Compiling build\kernel\libcpu\arm\am335x\stack.o [2K
>>> [36/56] Compiling build\kernel\libcpu\arm\am335x\mmu.o [2K
>>> [37/56] Compiling build\kernel\libcpu\arm\am335x\trap.o [2K
>>> [38/56] Compiling build\kernel\src\clock.o [2K
>>> [39/56] Compiling build\kernel\libcpu\arm\common\backtrace.o [2K
>>> [40/56] Compiling build\kernel\libcpu\arm\common\div0.o [2K
>>> [41/56] Compiling build\kernel\libcpu\arm\common\showmem.o [2K
>>> [42/56] Building build\kernel\libcpu\arm\am335x\vector_gcc.o [2K
>>> [43/56] Compiling build\kernel\src\device.o [2K
>>> [44/56] Compiling build\kernel\src\components.o [2K
>>> [45/56] Compiling build\kernel\src\idle.o [2K
>>> [46/56] Compiling build\kernel\src\irq.o [2K
>>> [47/56] Compiling build\kernel\src\ipc.o [2K
>>> [48/56] Compiling build\kernel\src\kservice.o [2K
>>> [49/56] Compiling build\kernel\src\mempool.o [2K
>>> [50/56] Compiling build\kernel\src\object.o [2K
>>> [51/56] Compiling build\kernel\src\thread.o [2K
>>> [52/56] Compiling build\kernel\src\mem.o [2K
>>> [53/56] Compiling build\kernel\src\scheduler.o [2K
>>> [54/56] Compiling build\kernel\src\timer.o [2K
>>> [55/56] Building rtthread-beaglebone.elf [2K
>>> [56/56] Building rtthread.bin
>>> scons: done building targets.
>>> ___________________________________________
>>>
>>>
>>> ------------------ 原始邮件 ------------------
>>> *发件人:* "Daniel Moody" <[email protected]>;
>>> *发送时间:* 2022年5月18日(星期三) 上午6:11
>>> *收件人:* "liruncong2018"<[email protected]>;
>>> *抄送:* "SCons users mailing list"<[email protected]>;
>>> *主题:* Re: [Scons-users] 回复： env.Textfile problem
>>>
>>> Also I added a new feature to my branch, now on the command line you can
>>> pass arguments to ninja invocation from scons command line with
>>> NINJA_CMD_ARGS=
>>>
>>> So for example scons ... NINJA_CMD_ARGS="-n" will mean you will  see
>>> ninja execute with the dry run arg '-n'.
>>>
>>> On Tue, May 17, 2022 at 5:09 PM Daniel Moody <[email protected]>
>>> wrote:
>>>
>>>> Hello liruncong,
>>>>
>>>> I reproduced the problem. The problem is that scons does not update the
>>>> ninja file if the existing ninja file looks exactly the same. Because the
>>>> define option -DTEST2 is hidden a generated file, scons detects the ninja
>>>> file is the same and does not update. Ninja also detects the the ninja file
>>>> is the same, and does not call on scons to regenerate the defineOptsFile.
>>>>
>>>> Originally you brought up the complaint that ninja always rebuilds some
>>>> files and you think it should just say (ninja:no work to do). I added a new
>>>> feature described previously to deal with this, but it seems it will not
>>>> work with generated files like defineOptsFile. I reverted this feature in
>>>> the ninja_integration_branch, you will have to see ninja calling to scons
>>>> to check if some files should build, only scons can make the decision and
>>>> ninja will detect if scons updated the files and if it should then rebuild
>>>> things which depended on those files.
>>>>
>>>> On Tue, May 17, 2022 at 7:02 AM liruncong2018 <[email protected]>
>>>> wrote:
>>>>
>>>>> Hello Daniel Moody,
>>>>> Thanks for the reminder, I corrected the recompile problem (
>>>>> https://github.com/liruncong/NinJaTest/tree/EnvTextFile).
>>>>> But the problem of not recompiling after modifying defineOptsFile, I
>>>>> still have no idea.
>>>>>
>>>>>
>>>>> ------------------ 原始邮件 ------------------
>>>>> *发件人:* "Daniel Moody" <[email protected]>;
>>>>> *发送时间:* 2022年5月17日(星期二) 下午2:03
>>>>> *收件人:* "SCons users mailing list"<[email protected]>;
>>>>> *抄送:* "liruncong2018"<[email protected]>;
>>>>> *主题:* Re: [Scons-users] 回复： env.Textfile problem
>>>>>
>>>>> hello liruncong,
>>>>>
>>>>> First you should understand why it is recompiling every time with just
>>>>> scons. Are your @files being generated differently because of ordering of
>>>>> the contents? Can you pick one file which is always recompiled and use this
>>>>> as the target to build with --debug=explain option?
>>>>>
>>>>>
>>>>>
>>>>> On Sun, May 15, 2022 at 4:20 AM liruncong2018 via Scons-users <
>>>>> [email protected]> wrote:
>>>>>
>>>>>> Hi,
>>>>>> I originally wanted to reproduce the problem in the test project, but
>>>>>> I did find another problem, scons is recompiled every time.
>>>>>> Test code: liruncong/NinJaTest at EnvTextFile (github.com)
>>>>>> <https://github.com/liruncong/NinJaTest/tree/EnvTextFile>
>>>>>> However, this problem was not found in my actual project.
>>>>>> The test project uses the gcc toolchain, and my actual project uses
>>>>>> the armclang (clang) toolchain.
>>>>>>
>>>>>> When ninja is not enabled, it is all recompiled every time you
>>>>>> compile. After enabling ninja, the first two executions are all recompiles,
>>>>>> and the third start is normal (ninja: no work to do.).
>>>>>> After normal (three times later), modify defineOptsFile as follows,
>>>>>> scons (enable ninja) does not recompile, which should be problematic.
>>>>>> defineOptsFile = env.Textfile("build/__define_options.txt", [
>>>>>>      "-DTEST1",
>>>>>>      "-DTEST2"  # new added
>>>>>> ])
>>>>>>
>>>>>> ------------------ 原始邮件 ------------------
>>>>>> *发件人:* "Bill Deegan" <[email protected]>;
>>>>>> *发送时间:* 2022年5月15日(星期天) 上午7:20
>>>>>> *收件人:* "SCons users mailing list"<[email protected]>;
>>>>>> *抄送:* "liruncong2018"<[email protected]>;
>>>>>> *主题:* Re: [Scons-users] env.Textfile problem
>>>>>>
>>>>>> Not enough information to help you.
>>>>>>
>>>>>> What's your scons logic look like for this?
>>>>>>
>>>>>> On Sat, May 14, 2022 at 3:04 AM liruncong2018 via Scons-users <
>>>>>> [email protected]> wrote:
>>>>>>
>>>>>>> Hi,
>>>>>>> The source(env.Textfile second parameter) change did not trigger
>>>>>>> recompilation when using env.Textfile.
>>>>>>> scons version: Commits · dmoody256/scons (github.com)
>>>>>>> <https://github.com/dmoody256/scons/commits/ninja_integration_branch>
>>>>>>> commit: force utf-8
>>>>>>> <https://github.com/dmoody256/scons/commit/c046aa853c177506543c0510562121b04f189e87>
>>>>>>> _______________________________________________
>>>>>>> Scons-users mailing list
>>>>>>> [email protected]
>>>>>>> https://pairlist4.pair.net/mailman/listinfo/scons-users
>>>>>>>
>>>>>> _______________________________________________
>>>>>> Scons-users mailing list
>>>>>> [email protected]
>>>>>> https://pairlist4.pair.net/mailman/listinfo/scons-users
>>>>>>
>>>>> _______________________________________________
>> Scons-users mailing list
>> [email protected]
>> https://pairlist4.pair.net/mailman/listinfo/scons-users
>>
>

_______________________________________________
Scons-users mailing list
[email protected]
https://pairlist4.pair.net/mailman/listinfo/scons-users