Re: [PATCH 2/2] Add LoongArch64 basic support

[Jinyang He]

On 2024-07-18 22:53, Camm Maguire wrote:

> I cannot reproduce with make 4.3, and 4.4.1 does not appear to be
> available yet in debian unstable, nor on the cfarm machines.  You don't
> mean debian package make-dfsg 4.3-4.1, do you?
Yes, I built it on other distros. Only make 4.4.1 produced errors.
(I tried 4.3 and it not produced error.) Trying build "make 4.4.1"
and export PATH should reproduce this error.

I simply fixed it by

diff --git a/gcl/clcs/makefile b/gcl/clcs/makefile
index a3ee81f52..5ece1d1df 100644
--- a/gcl/clcs/makefile
+++ b/gcl/clcs/makefile
@@ -41,5 +41,8 @@ saved_full_gcl: ${LISP}

 clean:
        rm -f *.o *.fn saved_full_gcl$(EXE) saved_full_gcl cmpinclude.h 
*.c *.h *.data saved_clcs_gcl$(EXE)
+
+.PHONY: keep_temp_files
+keep_temp_files: $(addsuffix .c,$(FILES)) $(addsuffix .h,$(FILES)) 
$(addsuffix .data,$(FILES))
+
 #.INTERMEDIATE: saved_clcs_gcl
-#.PRECIOUS: $(addsuffix .c,$(FILES))

or (follows should work, I test it on amd64)

diff --git a/gcl/clcs/makefile b/gcl/clcs/makefile
index a3ee81f52..d836a0e74 100644
--- a/gcl/clcs/makefile
+++ b/gcl/clcs/makefile
@@ -41,5 +41,6 @@ saved_full_gcl: ${LISP}

 clean:
        rm -f *.o *.fn saved_full_gcl$(EXE) saved_full_gcl cmpinclude.h 
*.c *.h *.data saved_clcs_gcl$(EXE)
+
 #.INTERMEDIATE: saved_clcs_gcl
-#.PRECIOUS: $(addsuffix .c,$(FILES))
+.PRECIOUS: $(addsuffix .h,$(FILES)) $(addsuffix .data,$(FILES))

The compile process is complete. The ansi_test result is,

2 out of 21727 total tests failed: DECODE-UNIVERSAL-TIME.1, 
DECODE-UNIVERSAL-TIME.2.
real time       :    164.620 secs
run-gbc time    :     57.850 secs
child run time  :     92.790 secs
gbc time        :      2.650 secs
allocation      :      15997 Mbytes
NIL

Could you please update gcl27 package (after fix this if the fix is needed)?
I believe it will build OK.


Thanks,
Jinyang