Makefile clean failed

[Alex -. (alex796)]

Hi,
Makefile und ich sind immer noch keine Freunde geworden, und ich schlage 
mich gerade mit dem clean Befehl herum.

Mein Makefile:

# CC and MACH are variables
CC=arm-none-eabi-gcc
MACH=cortex-m4
# CFLAGS will be options passed to the compiler
CFLAGS= -c -mcpu=$(MACH) -mthumb -std=gnu11 -Wall -O0

# Target: Dependencies
all: main.o led.o startup_stm32f303cctx.o
main.o: main.c
  $(CC) $(CFLAGS) -o $@ $^
  
led.o: led.c
  $(CC) $(CFLAGS) -o $@ $^

startup_stm32f303cctx.o: startup_stm32f303cctx.s
  $(CC) $(CFLAGS) -o $@ $^

clean:
  rm -rf *.o *.elf

"make all" funktioniert, es werden .o Dateien erzeugt.

"make clean" funktioniert nicht. Ausgabe auf der Konsole:
> rm -rf *.o *.elf
> process_begin: CreateProcess(NULL, rm -rf *.o *.elf, ...) failed.
> make (e=2): The system cannot find the file specified.
> make: *** [clean] Error 2

Kann jemand erkennen, was ich falsch mache?

[2⁵ (Gast)]

Arbeitest du evtl. unter Windows? Da gibts kein "rm"

[Alex -. (alex796)]

2⁵ schrieb:
> Arbeitest du evtl. unter Windows? Da gibts kein "rm"

Oh, guter Einwand. Ich arbeite mit Windows, und habe mich beim Makefile 
an einem Video gehalten.

Nach ein wenig Google, verwendet man bei Windows -del anstatt -rm

Vielen Dank für die schnelle Hilfe.

Gruß,

[🐧 DPA 🐧 (Gast)]

Make ist ein per POSIX standardisiertes tool. (Wobei ich nur die GNU 
Make Variante brauchbar finde). Von einem Build environment mit Make 
würde ich erwarten, das zumindest die meisten der von POSIX für POSIX 
Systeme vorgeschriebenen Tools auch da sind: 
https://pubs.opengroup.org/onlinepubs/9699919799/utilities/contents.html

Die bessere Lösung wäre also, eine Implementation der Tools zu 
installieren, oder gleich ein richtiges OS zu verwenden.

[Hannes J. (Firma: _⌨_) (pnuebergang)]

Alex -. schrieb:
>

> clean:
>   rm -rf *.o *.elf
>

Wenn das auf Linux wäre:

"-r" zusammen mit "*" Wildcards bei rm ist super-gefährlich.

Ein Tippfehler (* .o statt *.o) und du hast dir dein ganzes Projekt 
abgeräumt. Ohne "-r" "nur" dein aktuelles Verzeichnis.

Wenn das so in deinem Video stand, dann ist das Video Müll.

Bein jedem halbwegs vernünftigen Make:

clean:
        -$(RM) *.o *.elf
.PHONY: clean

Wobei:

Das führende "-" vor "$(RM)" wird nicht benötigt wenn RM richtig 
definiert ist (nicht-existierende Dateien ignoriert werden). Schadet 
aber auch nichts. Gib mal in einem leeren(!) Verzeichnis

make -p

ein um alle voreingestellten Regeln und Variablen zu sehen.

Weder "$(RM)", noch ".PHONY:" sind in POXIX spezifiziert, aber ein Make 
das solche Dinge heutzutage nicht kann würde ich persönlich entsorgen.

> Kann jemand erkennen, was ich falsch mache?

Na was wohl?

1. Handbuch nicht gelesen
2. Sich auf ein YT-Video verlassen
3. Windows

[Johannes S. (Gast)]

Oder weiterlernen und das makefile von cmake erzeugen lassen, das 
generiert dann plattformabhängig die richtigen Befehle.

[Rolf M. (rmagnus)]

Hannes J. schrieb:
> Bein jedem halbwegs vernünftigen Make:
> clean:
>         -$(RM) *.o *.elf
> .PHONY: clean

Falls das Projekt irgendwann in mehrere Verzeichnisse eingeteilt wird, 
muss man das rm dann allerdings für jedes Verzeichnis, in dem generierte 
Files liegen können, extra angeben. Und wenn die .o-Files in ein 
separates build-Verzeichnis gesteckt werden, muss man das entsprechend 
berücksichtigen.

> Wobei:
>
> Das führende "-" vor "$(RM)" wird nicht benötigt wenn RM richtig
> definiert ist (nicht-existierende Dateien ignoriert werden).

Die werden nur dann ignoriert, wenn du -f angibst, was bei dir oben aber 
fehlt. Ohne -f wird bei POSIX rm mit einer Fehlermeldung abgebrochen.

[Markus F. (mfro)]

Rolf M. schrieb:
> Falls das Projekt irgendwann in mehrere Verzeichnisse eingeteilt wird,
> muss man das rm dann allerdings für jedes Verzeichnis, in dem generierte
> Files liegen können, extra angeben. Und wenn die .o-Files in ein
> separates build-Verzeichnis gesteckt werden, muss man das entsprechend
> berücksichtigen.

Makefiles sollten das (und nur das) löschen, was sie erzeugen.
Also statt:

clean:
        -$(RM) *.o *.elf

lieber

clean:
        -$(RM) $(OBJS) $(EXE)

Dann gibt's auch keine bösen Überraschungen

[Hannes J. (Firma: _⌨_) (pnuebergang)]

Rolf M. schrieb:
> Falls das Projekt irgendwann in mehrere Verzeichnisse eingeteilt wird,
> muss man das rm dann allerdings für jedes Verzeichnis, in dem generierte
> Files liegen können, extra angeben.

Nein. Ein "-r" zusammen mit einem "*" in den Argumenten von rm ist 
trotzdem super-gefährlich. Es gab schon genug die plötzlich dumm aus der 
Wäsche geschaut haben. Das hat nichts mit Make zu tun, sondern mit der 
Shell-Syntax und dem Verhalten von rm.

> Und wenn die .o-Files in ein
> separates build-Verzeichnis gesteckt werden, muss man das entsprechend
> berücksichtigen.

Dann lässt man "*" weg. Denn gerade da braucht man es nicht:

$(RM) -r build

Ich rede hier von dem Unterschied ein paar Verzeichnisse kontrolliert zu 
löschen. Z.B.

$(RM) -r build cache tmp

und dem Fehler sich versehentlich das Projekt zu versenken:

$(RM) -r * .o

Am besten noch bevor man eine größere Änderung in git reingeschoben hat. 
Noch dazu dass das "-r" für die beabsichtigte Löschung "*.o" völlig 
unnütz ist und nicht benötigt wird.

>> Das führende "-" vor "$(RM)" wird nicht benötigt wenn RM richtig
>> definiert ist (nicht-existierende Dateien ignoriert werden).
>
> Die werden nur dann ignoriert, wenn du -f angibst, was bei dir oben aber
> fehlt. Ohne -f wird bei POSIX rm mit einer Fehlermeldung abgebrochen.

Ne, das passt schon. Die normale Definition von $(RM) ist "rm -f".

Markus F. schrieb:
> Makefiles sollten das (und nur das) löschen, was sie erzeugen.

Ja, und ich würde auch eine Doppel-Doppelpunkt-Regeln verwenden um 
Erzeugung und Cleanup im Makefile hintereinander zu haben. Aber ich 
wollte es nicht zu kompliziert machen.

Ansonsten geht es bei Dingen die nur teilweise oder gar nicht von Make 
erzeugt werden nicht ohne "*". Klassisches Beispiel sind Reste vom 
Editor

clean:
        $(RM) *.bak *~
.PHONY: clean

[Oliver S. (oliverso)]

🐧 DPA 🐧 schrieb:
> Die bessere Lösung wäre also, eine Implementation der Tools zu
> installieren, oder gleich ein richtiges OS zu verwenden.

https://www.gnu.org/software/coreutils/

Oliver

[Markus F. (mfro)]

Hannes J. schrieb:
> Ansonsten geht es bei Dingen die nur teilweise oder gar nicht von Make
> erzeugt werden nicht ohne "*". Klassisches Beispiel sind Reste vom
> Editor

make clean

sollte genau das tun, wozu es gedacht ist: eine "saubere" 
Ausgangssituation für ein folgendes make herstellen.

"Küche putzen" kann man heutzutage getrost git clean überlassen...

[Alex -. (alex796)]

Vielen Dank euch allen, ich werde mich näher und tiefgründig mit make 
befassen :)