Patch to issue with linking error to i2c library as well as cleanup Makefile

Author: Alex Mous

MPU6050.cpp

@@ -2,17 +2,6 @@
 //Copyright (c) 2019, Alex Mous
 //Licensed under the CC BY-NC SA 4.0
 
-//Includes
-#include <iostream>
-#include <unistd.h>
-#include <fcntl.h>
-#include <sys/ioctl.h>
-#include <sys/types.h>
-#include <time.h>
-#include <linux/i2c-dev.h>
-#include <cmath>
-#include <thread>
-
 //Include the header file for this class
 #include "MPU6050.h"
 

MPU6050.h

@@ -41,8 +41,10 @@
 #include <sys/ioctl.h>
 #include <sys/types.h>
 #include <time.h>
-#include <linux/i2c-dev.h>
-#include <i2c/smbus.h>
+extern "C" {
+	#include <linux/i2c-dev.h>
+	#include <i2c/smbus.h>
+}
 #include <cmath>
 #include <thread>
 

Makefile

@@ -1,16 +1,35 @@
-all: MPU6050.cpp
-	g++ -fPIC -c MPU6050.cpp -o MPU6050.o
-	g++ -shared -o libMPU6050.so MPU6050.o
-	install -m 755 -p libMPU6050.so /usr/lib/
-	install -m 644 -p MPU6050.h /usr/include/
-	rm -f MPU6050.o
-	rm -f libMPU6050.so
+CC=g++
 
-clean:
+DEPS=MPU6050.h
+CFLAGS=-fPIC -Wall
+
+LIBS=-li2c
+LFLAGS=-shared
+
+OBJ=MPU6050.o
+OLIB=libMPU6050.so
+
+
+
+%.o: %.cpp $(DEPS)
+	$(CC) -c $< -o $@ $(CFLAGS)
+
+all: $(OBJ)
+	$(CC) -o $(OLIB) $< $(LIBS) $(LFLAGS)
+
+install: all $(OLIB) $(DEPS)
+	install -m 755 -p $(OLIB) /usr/lib/
+	install -m 644 -p $(DEPS) /usr/include/
+
+uninstall:
 	rm -f /usr/include/MPU6050.h
 	rm -f /usr/lib/libMPU6050.so
+
+clean:
 	rm -f Example
+	rm -f Example.o
+	rm -f MPU6050.o
+	rm -f libMPU6050.so
 
-example:
-	g++ Example.cpp -o Example -lMPU6050 -pthread
-	./Example
+example: Example.o
+	$(CC) $< -o Example -lMPU6050 -pthread