Page 1 of 1

Tiny patch. Makefile "make clean" broken in msys/mingw.

Posted: Wed 01 Apr, 2015 9:46 pm
by insidious611
In Makefile.mingw, you seem to have forgotten that the "del" command doesn't exist within an msys shell, which is the preferred mingw dev environment.

Code: Select all

--- Makefile.mingw	2015-04-01 16:40:54 -0500
+++ Makefile.mingw	2015-04-01 16:41:11 -0500
@@ -33,9 +33,9 @@
 all : PCem.exe
 
 clean :
-	del *.o
-	del *.exe
-	del *.res
+	rm -f *.o
+	rm -f *.exe
+	rm -f *.res
 
 %.o : %.c
 	$(CC) $(CFLAGS) -c $<
this makes

Code: Select all

make -f Makefile.mingw clean

work properly.

Re: Tiny patch. Makefile "make clean" broken in msys/mingw.

Posted: Thu 02 Apr, 2015 7:33 am
by SarahWalker
insidious611 wrote:In Makefile.mingw, you seem to have forgotten that the "del" command doesn't exist within an msys shell, which is the preferred mingw dev environment.
Yes, but it's not the one I use. Your patch would break "make clean" for me.

Re: Tiny patch. Makefile "make clean" broken in msys/mingw.

Posted: Sat 04 Apr, 2015 9:14 am
by ender
If you use GNU make, rm should work regardless of system (while del is limited to Windows).

Re: Tiny patch. Makefile "make clean" broken in msys/mingw.

Posted: Tue 14 Apr, 2015 7:48 pm
by SarahWalker
Fixed in rev 224.