Stupid makefile!

[squeech]

Hello all,

I am trying to compile together a program using three files:
-Two files for a class called CPU (cpu.h and cpu.cpp)
-A driver file with the main function in it (driver.cpp)

This is my first attempt at writing my own makefile for an abstract data type like this so forgive the maturity level, but here is the makefile I had written for this program:

# Makefile for the CPU overclocking program

 CXX = g++
 CXXFLAGS = -g -c
 LD = g++

 ocsim: cpu.o driver.o
 ${LD} cpu.o driver.o -o ocsim

 cpu.o: cpu.h cpu.cpp
 ${CXX} ${CXXFLAGS} cpu.cpp

 driver.o: driver cpu.h
 ${CXX} ${CXXFLAGS} driver.cpp

 clean:
 rm -rf ocsim *.o core

I want to use the g++ compiler and the executable to be named "ocsim". when I try to run make with this file I get an error that states:

"Makefile:n: *** missing separator. Stop."

Where "n" is a number that varies each time I try the command.

Any help is GREATLY appreciated!

[Martee]

One rather annoying thing about makefiles is that there are certain places where you _must_ use tabs, not spaces. So the lines

ocsim: cpu.o driver.o
${LD} cpu.o driver.o -o ocsim

must be written as

ocsim: cpu.o driver.o
(TAB GOES HERE!!!)${LD} cpu.o driver.o -o ocsim

... replacing the obvious text with a tab. The same thing applies to the other rules. Also, in your rule for driver.o, I think you want to say 'driver.cpp', and not 'driver'.

[squeech]

Aha! That is slightly annoying.

However, I can tell you that I'll never forget it after that!

You were right about the driver.o rule, thanks for catchin' that.