On Sat, 22 Mar 2025 03:43:58 GMT, SendaoYan <s...@openjdk.org> wrote:

>> Hi all,
>> 
>> There is an extra a.out file generated located in repository root directory 
>> after execute the configure command with llvm/clang compiler sometimes, not 
>> always. The a.out was generate by `$LINKER -Wl,-v 2>&1 | $HEAD -n 1`, shows 
>> as below. This PR  add a command to remove the extra a.out file. No risk.
>> 
>> 
>>> rm -rf a.out ; date ; ~/software/acc/x86_64/bin/clang -Wl,-v 2>&1 | head -n 
>>> 1 ; stat a.out ; ls -lh a.out ; file a.out
>> Thu Mar 20 02:46:13 PM CST 2025
>> GNU ld (GNU Binutils) 2.43.50.20240909
>>   File: a.out
>>   Size: 792 Blocks: 8 IO Block: 4096 regular file
>> Device: 10303h/66307d Inode: 2355308 Links: 1
>> Access: (0640/-rw-r-----) Uid: ( 995/yansendao) Gid: ( 992/yansendao)
>> Access: 2025-03-20 14:46:13.682967228 +0800
>> Modify: 2025-03-20 14:46:13.682967228 +0800
>> Change: 2025-03-20 14:46:13.682967228 +0800
>>  Birth: 2025-03-20 14:46:13.674966977 +0800
>> -rw-r----- 1 yansendao yansendao 792 Mar 20 14:46 a.out
>> a.out: data
>
> I have add a comment line do demonstrate why we need `RM -rf a.out`.

> Hello @sendaoYan, I don't have knowledge of these build files so I'll let the 
> build team decide if this change is needed. However, is the `-rf` needed in 
> that `rm` command? Would just `rm a.out` not work?

Hello @jaikiran There is a small probability that a.out will not be generated, 
so I use `-rf` paramater to avoid print error message when there is no a.out 
file. I think we should use `-f` instead of `-rf`.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/24125#issuecomment-2745086955

Reply via email to