Makefile: mehrere Sourcefiles kompilieren klappt nicht unter Linux??

[Hannes (Gast)]

Hallo Liebe Forengemeinde,

ich habe ein kleines Problem mit dem µC-programmieren unter Linux.

Ich nutze Debian 7 mit gcc 4.7.2.

Das Problem klingt recht simpel:

Ich habe ein µC Projekt wo ich an einem AtMega8 ein normal 16x2 Display 
betreiben will. Ich benutze dazu die Anleitung hier aus dem Wiki, also 
mit den Dateien lcd-routines.c und lcd-routines.h. Das Problem ist nun 
aber das ich beim kompilieren immer den Fehler bekomme "undefined 
reference to" bei den "lcd-Befehlen" wie zB lcd_init(); oder 
lcd_string("");. Ich weiß das es funktioniert wenn ich die beiden 
Dateien direkt in meine main.c einfüge, doch die feine Art ist das ja 
wohl nicht.
Ich habe trotzdem im Makefile bei "SRC" die lcd-routines.c mit 
angegeben. Ich habe es mittlerweile Makefiles von verschiedenen Seiten 
(unteranderem mit der Vorlage und auch mit der Anleitung hier im WIKI) 
probiert.

Ich vermute das trotzdem bei den Makefiles bzw. beim Einbinden der 
lcd-routines.c der Fehler liegt.

Woran kann das liegen? Habe mit Makefiles noch nicht sehr viel Erfahrung 
bzw. habe nicht sehr oft mit mehreren C-Source-Dateien gearbeite.

MfG

Hannes

[Tux (Gast)]

Zeig doch mal dein Makefile, alles andere ist wildes Raten ...

[Peter II (Gast)]

Hannes schrieb:
> Ich vermute das trotzdem bei den Makefiles bzw. beim Einbinden der
> lcd-routines.c der Fehler liegt.

dann poste doch mal die Ausgaben von make (vorher make clean machen). 
eventuell auch das Makefile.

[__f__ (Gast)]

versuch doch erstmal deine sources über die kommandozeile zu 
kompilieren(ich geh davon aus es sind nicht so viele sources?).
Und dann schaust du ob du die Befehle, mit denen du manuell kompiliert 
hast, so "ähnlich" im makefile findest
http://www.ijon.de/comp/tutorials/makefile.html

[Hannes (Gast)]

Das ist die Ausgabe im Terminal

avr-gcc -g -mmcu=atmega8 -Wall -Wstrict-prototypes -Os -mcall-prologues -o launchcontroll.out -Wl,-Map,launchcontroll.map launchcontroll.o 
launchcontroll.o: In function `main':
/home/hannes/Arbeitsfläche/Launchcontrol/launchcontroll.c:49: undefined reference to `lcd_init'
/home/hannes/Arbeitsfläche/Launchcontrol/launchcontroll.c:50: undefined reference to `lcd_clear'
/home/hannes/Arbeitsfläche/Launchcontrol/launchcontroll.c:52: undefined reference to `lcd_setcursor'
/home/hannes/Arbeitsfläche/Launchcontrol/launchcontroll.c:53: undefined reference to `lcd_string'
collect2: error: ld returned 1 exit status
make: *** [launchcontroll.out] Fehler 1

Und Das Makefile ist das von hier:

http://www.mikrocontroller.net/articles/Beispiel_Makefile

[Hannes (Gast)]

Ich hatte vergessen zusagen:

Das Makefile habe ich natürlich an einen atmega8 bei MCU angepasst und 
den Programmer geändert.

[S. R. (svenska)]

In dem Beispiel-Makefile steht doch drin, dass du alle .c-Dateien 
auflisten musst. Woher soll "make" denn sonst wissen, welche Dateien es 
zusammenbauen muss?

Irgendwo ist deine main.c aufgelistet. Dahinter trägst du die lcd.c ein 
und probierst nochmal.

[Hannes (Gast)]

Ja das ist richtig. Hatte aber geschrieben, dass ich das makefile schon 
angepasst hatte unteranderem auch mit dem eintragen der lcd-routines.c 
bei "SRC = ..."

# MCU name
MCU = atmega8


# Processor frequency.
#     This will define a symbol, F_CPU, in all source code files equal to the 
#     processor frequency. You can then use this symbol in your source code to 
#     calculate timings. Do NOT tack on a 'UL' at the end, this will be done
#     automatically to create a 32-bit value in your source code.
#     Typical values are:
#    F_CPU =  1000000
#       F_CPU =  1843200
#       F_CPU =  2000000
#       F_CPU =  3686400
#       F_CPU =  4000000
#    F_CPU =  7372800
#       F_CPU =  8000000
#    F_CPU = 11059200
#       F_CPU = 14745600
F_CPU = 16000000
#       F_CPU = 18432000
#       F_CPU = 20000000
#       F_CPU = 8000000


# Output format. (can be srec, ihex, binary)
FORMAT = ihex


# Target file name (without extension).
TARGET = launchcontroll


# Object files directory
OBJDIR = obj


# List C source files here. (C dependencies are automatically generated.)
SRC = $(TARGET).c lcd.c


# List C++ source files here. (C dependencies are automatically generated.)
CPPSRC = 


# List Assembler source files here.
#     Make them always end in a capital .S.  Files ending in a lowercase .s
#     will not be considered source files but generated files (assembler
#     output from the compiler), and will be deleted upon "make clean"!
#     Even though the DOS/Win* filesystem matches both .s and .S the same,
#     it will preserve the spelling of the filenames, and gcc itself does
#     care about how the name is spelled on its command-line.
ASRC =


# Optimization level, can be [0, 1, 2, 3, s]. 
#     0 = turn off optimization. s = optimize for size.
#     (Note: 3 is not always the best optimization level. See avr-libc FAQ.)
OPT = s


# Debugging format.
#     Native formats for AVR-GCC's -g are dwarf-2 [default] or stabs.
#     AVR Studio 4.10 requires dwarf-2.
#     AVR [Extended] COFF format requires stabs, plus an avr-objcopy run.
DEBUG = dwarf-2

wollte das lange ding nicht einfügen aber ok

bei mir hab ich die lcd-routines.c als lcd.c gespeichert. Nicht das sich 
wer wundert das da lcd.c bei SRC steht. Hab alles Unter Debug jetzt für 
den Beitrag entfernt damit das hier im forum nicht so lang ist.

[Marc (gierig)]

#include "lcd.h"

 (oder wie die auch immer bei Dir heißt)
vergessen in deinem File von wo aus Du die Funktion aufrufst ?

Zeig doch bitte das ganze Projekt.

[Peter II (Gast)]

Marc D. schrieb:
> (oder was wie die auch immer heißt)

nein, dann kommt ein compiler Fehler und kein Linker Fehler.

[Hannes (Gast)]

//#define F_CPU 8000000UL   //the Mega8 runs with the internal Osc. 
              //on 8MHz

#define F_CPU 16000000UL  //external crystal on  16MHz
        
#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include "lcd-routines.h"

Hier mal der Kopf von der Hauptdatei

ist ja egal wie es heißt habs  ja drinnen.

Außerdem wenn ich die "lcd.c" in die main einfüge dann klappt es ja ... 
hab grade ein bißchen gelesen und bin auf die Vermutung Linkerfehler 
gekommen... könnte da das Problem liegen?

[Hannes (Gast)]

Der Rest des Programms kann es nicht sein dort habe ich zur zeit in der 
int main(void) nur lcd_init(), lcd_clear(), lcd_setcursor(0,0) und eine 
ausgabe mit lcd_string("irgendwas") sehen.

[S. R. (svenska)]

Der Linkerfehler kommt daher, dass beim Linken die lcd.o nicht mit der 
main.o zusammengelinkt wird und daher der Linker die in lcd.o 
vorhandenen Symbole nicht findet und die Referenzen aus main.o nicht 
auflösen kann.

Das heißt:
- obj/lcd.o wird nicht erzeugt
  ($CC $CFLAGS -c -o lcd.o lcd.c), oder
- obj/lcd.o wird nicht eingelinkt
  ($CC $CFLAGS $LFLAGS -o main main.o lcd.o).

Probiere mal "make obj/lcd.o", wenn er dir dann die lcd.o erzeugt, ist 
es der zweite Fall; andernfalls der erste Fall. In jedem Fall erkennt 
make die Abhängigkeit zwischen "targetcontroll" und "lcd.o" nicht.

(Übrigens schreibt man "control" nicht mit 2 L am Ende.)

[batman (Gast)]

Listet man da nicht anstatt der SRC die OBJ-Dateien auf und überläßt es 
dem make, die entsprechenden Sourcen zu kompilieren?
Ich hätte da im Makefile OBJ= .. lcd.o ..