skeeto / w64devkit

Portable C and C++ Development Kit for x64 (and x86) Windows
The Unlicense
2.7k stars 186 forks source link

in windows shell running make clean give "del: not found" error #20

Open meiry opened 1 year ago

meiry commented 1 year ago

Running make clean command gives error in windows 10 While trying to clean the lib from source

  1. open windows shell in raylib dir
  2. C:\raylib\w64devkit>set PATH=C:\raylib\w64devkit\bin;%PATH%
  3. cd c:\dev\my\raylib\raylib_git\raylib-4.0.0\src Then :
    c:\dev\my\raylib\raylib_git\raylib-4.0.0\src>make clean                                                                                                                                                                                      del *.o /s                                                                                                                                                                                                                                   sh: del: not found                                                                                                                                                                                                                           make: *** [Makefile:685: clean] Error 127

By the way solution is to convert to rm command in makefile : image

raysan5 commented 1 year ago

@meiry I'm afraid this issue is dependant on the SHELL used by the OS/Platform to run those commands. There is no easy solution, it's really OS/Platform dependant, I invested many hours of my life trying to find a solution that could work in all situations and I couldn't; this issue is the prove of it.

meiry commented 1 year ago

@raysan5
just add option to the make file so if its see's that flag it will use rm and not del something like MAKE_RM=1 make clean

Peter0x44 commented 1 year ago

@meiry try make SHELL=cmd clean that should also work

I see you are also using raylib 4.0, I believe this issue has been reviewed since can you try with the current version of the repo too?

Peter0x44 commented 1 year ago

this definitely isn't the appropriate place to open the issue, though

imaami commented 1 year ago

I don't think this can work, can it?

rm $(RAYLIB_RELEASE_PATH)

It needs the -r argument to delete a directory tree.