Actual compilation

Makefile

@@ -228,10 +228,12 @@ define PARSE_KEYMAP
         KB_SP := $(CURRENT_KB)/$$(CURRENT_SP)
     endif
     KB_SP := $(BOLD)$$(KB_SP)$(NO_COLOR)
+    MAKE_VARS := KEYBOARD=$$(CURRENT_KB) SUBPROJECT=$$(CURRENT_SP) KEYMAP=$$(CURRENT_KM)
+    MAKE_VARS += VERBOSE=$(VERBOSE) COLOR=$(COLOR) SILENT=false
     COMMAND_$$(COMMAND) := \
         printf "Compiling $$(KB_SP) with $(BOLD)$$(CURRENT_KM)$(NO_COLOR)" | \
         $(AWK) '{ printf "%-118s", $$$$0;}'; \
-    LOG=$$$$(echo "$$(MAKE) -c $(ROOT_DIR) -f build_keyboard.mk VERBOSE=$(VERBOSE) COLOR=$(COLOR) SILENT=true" 2>&1) ; \
+        LOG=$$$$($$(MAKE) -C $(ROOT_DIR) -f build_keyboard.mk $$(MAKE_VARS) 2>&1) ; \
         if [ $$$$? -gt 0 ]; \
             then $$(PRINT_ERROR_PLAIN); \
         elif [ "$$$$LOG" != "" ] ; \
@@ -259,9 +261,11 @@ $(SUBPROJECTS): %: %-allkm
 
 .PHONY: %
 %: 
+	cmp --version >/dev/null 2>&1; if [ $$? -gt 0 ]; then printf "$(MSG_NO_CMP)"; exit 1; fi;
 	$(eval $(call PARSE_RULE,$@))
 	$(foreach COMMAND,$(COMMANDS),$(RUN_COMMAND))
 	
+
 .PHONY: all
 all: all-keyboards 
 

build_keyboard.mk

@@ -4,44 +4,9 @@ endif
 
 .DEFAULT_GOAL := all
 
-space := $(subst ,, )
-ESCAPED_ABS_PATH = $(subst $(space),_SPACE_,$(abspath $1))
-starting_makefile := $(call ESCAPED_ABS_PATH,$(firstword $(MAKEFILE_LIST)))
-mkfile_path := $(call ESCAPED_ABS_PATH,$(lastword $(MAKEFILE_LIST))))
-abs_tmk_root := $(patsubst %/,%,$(dir $(mkfile_path)))
-
-ifneq (,$(findstring /keyboards/,$(starting_makefile)))
-	possible_keyboard:=$(patsubst %/,%,$(dir $(patsubst $(abs_tmk_root)/keyboards/%,%,$(starting_makefile))))
-	ifneq (,$(findstring /keymaps/,$(possible_keyboard)))
-		KEYMAP_DIR:=$(lastword $(subst /keymaps/, ,$(possible_keyboard)))
-		KEYBOARD_DIR:=$(firstword $(subst /keymaps/, ,$(possible_keyboard)))
-		ifneq (,$(findstring /,$(KEYBOARD_DIR)))
-			# SUBPROJECT_DIR:=$(lastword $(subst /, ,$(KEYBOARD_DIR)))
-			# KEYBOARD_DIR:=$(firstword $(subst /, ,$(KEYBOARD_DIR)))
-			tmk_root = ../../..
-		else
-			tmk_root = ../../../..
-		endif
-	else
-		KEYMAP_DIR:=default
-		KEYBOARD_DIR:=$(possible_keyboard)
-		ifneq (,$(findstring /,$(KEYBOARD_DIR)))
-			# SUBPROJECT_DIR:=$(lastword $(subst /, ,$(KEYBOARD_DIR)))
-			# KEYBOARD_DIR:=$(firstword $(subst /, ,$(KEYBOARD_DIR)))
-			tmk_root = ../../..
-		else
-			tmk_root = ../..
-		endif
-	endif
-else
-	tmk_root = .
-endif
-# $(info $(KEYBOARD_DIR))
-# $(info $(KEYMAP_DIR))
-# $(info $(SUBPROJECT_DIR))
 
 # Directory common source filess exist
-TOP_DIR = $(tmk_root)
+TOP_DIR = .
 TMK_DIR = tmk_core
 TMK_PATH = $(TOP_DIR)/$(TMK_DIR)
 LIB_PATH = $(TOP_DIR)/lib
@@ -50,92 +15,49 @@ QUANTUM_DIR = quantum
 QUANTUM_PATH = $(TOP_DIR)/$(QUANTUM_DIR)
 
 
-ifdef keyboard
-	KEYBOARD ?= $(keyboard)
-endif
-ifdef KEYBOARD_DIR
-	KEYBOARD ?= $(KEYBOARD_DIR)
-endif
-ifndef KEYBOARD
-	KEYBOARD=planck
-endif
-
 MASTER ?= left
 ifdef master
 	MASTER = $(master)
 endif
 
+KEYBOARD_PATH := keyboards/$(KEYBOARD)
+KEYBOARD_C := $(KEYBOARD_PATH)/$(KEYBOARD).c
 
-# converts things to keyboards/subproject
+ifneq ("$(wildcard $(KEYBOARD_C))","")
-ifneq (,$(findstring /,$(KEYBOARD)))
+    include $(KEYBOARD_PATH)/rules.mk
-	TEMP:=$(KEYBOARD)
-	KEYBOARD:=$(firstword $(subst /, ,$(TEMP)))
-	SUBPROJECT:=$(lastword $(subst /, ,$(TEMP)))
-endif
-
-KEYBOARD_PATH = $(TOP_DIR)/keyboards/$(KEYBOARD)
-
-ifdef sub
-	SUBPROJECT=$(sub)
-endif
-ifdef subproject
-	SUBPROJECT=$(subproject)
-endif
-
-ifneq ("$(wildcard $(KEYBOARD_PATH)/$(KEYBOARD).c)","")
-	KEYBOARD_FILE = keyboards/$(KEYBOARD)/$(KEYBOARD).c
-	ifneq ($(call ESCAPED_ABS_PATH,$(KEYBOARD_PATH)/Makefile),$(starting_makefile))
-		-include $(KEYBOARD_PATH)/Makefile
-	endif
 else 
-$(error "$(KEYBOARD_PATH)/$(KEYBOARD).c" does not exist)
+    $(error "$(KEYBOARD_C)" does not exist)
 endif
 
-ifdef SUBPROJECT_DEFAULT
-	SUBPROJECT?=$(SUBPROJECT_DEFAULT)
-endif
 
-ifdef SUBPROJECT
+ifneq ($(SUBPROJECT),)
-	SUBPROJECT_PATH = $(TOP_DIR)/keyboards/$(KEYBOARD)/$(SUBPROJECT)
+    SUBPROJECT_PATH := keyboards/$(KEYBOARD)/$(SUBPROJECT)
-	ifneq ("$(wildcard $(SUBPROJECT_PATH)/$(SUBPROJECT).c)","")
+    SUBPROJECT_C := $(SUBPROJECT_PATH)/$(SUBPROJECT).c
+    ifneq ("$(wildcard $(SUBPROJECT_C))","")
         OPT_DEFS += -DSUBPROJECT_$(SUBPROJECT)
-		SUBPROJECT_FILE = keyboards/$(KEYBOARD)/$(SUBPROJECT)/$(SUBPROJECT).c
+        include $(SUBPROJECT_PATH)/rules.mk
-		ifneq ($(call ESCAPED_ABS_PATH,$(SUBPROJECT_PATH)/Makefile),$(starting_makefile))
-			-include $(SUBPROJECT_PATH)/Makefile
-		endif
     else 
-$(error "$(SUBPROJECT_PATH)/$(SUBPROJECT).c" does not exist)
+        $(error "$(SUBPROJECT_PATH)/$(SUBPROJECT).c" does not exist)
     endif
 endif
 
-ifdef keymap
+MAIN_KEYMAP_PATH := $(KEYBOARD_PATH)/keymaps/$(KEYMAP)
-	KEYMAP ?= $(keymap)
+MAIN_KEYMAP_C := $(MAIN_KEYMAP_PATH)/keymap.c
-endif
+SUBPROJ_KEYMAP_PATH := $(SUBPROJECT_PATH)/keymaps/$(KEYMAP)
-ifdef KEYMAP_DIR
+SUBPROJ_KEYMAP_C := $(SUBPROJ_KEYMAP_PATH)/keymap.c
-	KEYMAP ?= $(KEYMAP_DIR)
+ifneq ("$(wildcard $(SUBPROJ_KEYMAP_C))","")
-endif
+    -include $(SUBPROJ_KEYMAP_PATH)/Makefile
-ifndef KEYMAP
+    KEYMAP_C := $(SUBPROJ_KEYMAP_C)
-	KEYMAP = default
+    KEYMAP_PATH := $(SUBPROJ_KEYMAP_PATH)
-endif
+else ifneq ("$(wildcard $(MAIN_KEYMAP_C))","")
-KEYMAP_PATH = $(KEYBOARD_PATH)/keymaps/$(KEYMAP)
+    -include $(MAIN_KEYMAP_PATH)/Makefile
-ifneq ("$(wildcard $(KEYMAP_PATH)/keymap.c)","")
+    KEYMAP_C := $(MAIN_KEYMAP_C)
-	KEYMAP_FILE = keyboards/$(KEYBOARD)/keymaps/$(KEYMAP)/keymap.c
+    KEYMAP_PATH := $(MAIN_KEYMAP_PATH)
-	ifneq ($(call ESCAPED_ABS_PATH,$(KEYMAP_PATH)/Makefile),$(starting_makefile))
-		-include $(KEYMAP_PATH)/Makefile
-	endif
 else
-	ifeq ("$(wildcard $(SUBPROJECT_PATH)/keymaps/$(KEYMAP)/keymap.c)","")
+    $(error "$(MAIN_KEYMAP_C)/keymap.c" does not exist)
-$(error "$(KEYMAP_PATH)/keymap.c" does not exist)
-	else
-		KEYMAP_PATH = $(SUBPROJECT_PATH)/keymaps/$(KEYMAP)
-		KEYMAP_FILE = keyboards/$(KEYBOARD)/$(SUBPROJECT)/keymaps/$(KEYMAP)/keymap.c
-		ifneq ($(call ESCAPED_ABS_PATH,$(KEYMAP_PATH)/Makefile),$(starting_makefile))
-			-include $(KEYMAP_PATH)/Makefile
-		endif
-	endif
 endif
 
-ifdef SUBPROJECT
+ifneq ($(SUBPROJECT),)
 	TARGET ?= $(KEYBOARD)_$(SUBPROJECT)_$(KEYMAP)
 else
 	TARGET ?= $(KEYBOARD)_$(KEYMAP)
@@ -149,36 +71,27 @@ BUILD_DIR = $(TOP_DIR)/.build
 OBJDIR = $(BUILD_DIR)/obj_$(TARGET)
 
 
-
 ifneq ("$(wildcard $(KEYMAP_PATH)/config.h)","")
 	CONFIG_H = $(KEYMAP_PATH)/config.h
 else
 	CONFIG_H = $(KEYBOARD_PATH)/config.h
-	ifdef SUBPROJECT
+	ifneq ($(SUBPROJECT),)
-		ifneq ("$(wildcard $(SUBPROJECT_PATH)/$(SUBPROJECT).c)","")
+		ifneq ("$(wildcard $(SUBPROJECT_C))","")
 			CONFIG_H = $(SUBPROJECT_PATH)/config.h
 		endif
 	endif
 endif
 
 # # project specific files
-SRC += $(KEYBOARD_FILE) \
+SRC += $(KEYBOARD_C) \
-	$(KEYMAP_FILE) \
+	$(KEYMAP_C) \
 	$(QUANTUM_DIR)/quantum.c \
 	$(QUANTUM_DIR)/keymap_common.c \
 	$(QUANTUM_DIR)/keycode_config.c \
 	$(QUANTUM_DIR)/process_keycode/process_leader.c
 
-ifdef SUBPROJECT
+ifneq ($(SUBPROJECT),)
-	SRC += $(SUBPROJECT_FILE)
+	SRC += $(SUBPROJECT_C)
-endif
-
-ifdef SUBPROJECT
-	SRC += $(SUBPROJECT_FILE)
-endif
-
-ifdef SUBPROJECT
-	SRC += $(SUBPROJECT_FILE)
 endif
 
 ifndef CUSTOM_MATRIX
@@ -229,7 +142,7 @@ endif
 
 # Search Path
 VPATH += $(KEYMAP_PATH)
-ifdef SUBPROJECT
+ifneq ($(SUBPROJECT),)
 	VPATH += $(SUBPROJECT_PATH)
 endif
 VPATH += $(KEYBOARD_PATH)

message.mk

@@ -33,3 +33,4 @@ PRINT_ERROR_PLAIN = ($(SILENT) ||printf " $(ERROR_STRING)" | $(AWK_STATUS)) && $
 PRINT_WARNING_PLAIN = ($(SILENT) || printf " $(WARN_STRING)" | $(AWK_STATUS)) && $(TAB_LOG_PLAIN)
 PRINT_OK = $(SILENT) || printf " $(OK_STRING)" | $(AWK_STATUS)
 BUILD_CMD = LOG=$$($(CMD) 2>&1) ; if [ $$? -gt 0 ]; then $(PRINT_ERROR); elif [ "$$LOG" != "" ] ; then $(PRINT_WARNING); else $(PRINT_OK); fi;
+MSG_NO_CMP = $(ERROR_COLOR)Error:$(NO_COLOR)$(BOLD) cmp command not found, please install diffutils\n$(NO_COLOR)