Refactor Makefile

Author: AitorATuin

.gitignore

@@ -0,0 +1,2 @@
+*.swp
+*~

Makefile

@@ -1,57 +1,100 @@
-
-PYTHON_VERSION=2.7
-PYTHON_LIBS=FindPackage GetAvailable GuessLatest CheckDependencies DescribeProgram UseFlags Corrections
-PYTHON_SITE=lib/python$(PYTHON_VERSION)/site-packages
 PROGRAM=Scripts
 VERSION=git-$(shell date +%Y%m%d)
 goboPrograms?=/Programs
 PREFIX?=
-DESTDIR=$(goboPrograms)/$(PROGRAM)/$(VERSION)
+DESTDIR=$(PREFIX)/$(goboPrograms)/$(PROGRAM)/$(VERSION)
+
+INSTALL_FILE = install
+INSTALL_DIR = install -d
 
-man_files = $(shell cd bin; grep -l Parse_Options * | xargs -i echo share/man/man1/{}.1)
-exec_files = $(patsubst src/%.c,bin/%,$(wildcard src/*.c))
+PYTHON_VERSION=2.7
+PYTHON_LIBS=Corrections
+PYTHON_SITE=python$(PYTHON_VERSION)/site-packages
 
-all: python $(exec_files) manuals
+MAN_FILES = $(shell grep -l Parse_Options bin/* | xargs -i echo {}.1)
+EXEC_FILES = $(patsubst src/%.c,bin/%,$(wildcard src/*.c))
+SCRIPT_FILES = AddUser AttachProgram DeduceName FindPackage GoboPath install PrioritiseUpdates ScriptFunctions UnversionExecutables VersionExecutables Alien AugmentCommandNotFoundDatabase Dependencies FindQuick GrepQuick InstallPackage ProblemReport SignProgram UpdateKdeRecipe which Alien-Cabal CheckDependants DescribeProgram FixAttributes GrepReplace KillProcess RemoveBroken SuggestDuplicates UpdateSettings xmlcatalog Alien-CPAN CheckDependencies DetachProgram FixDirReferences GuessLatest ListProgramFiles RemoveEmpty SuggestUpdates UpdateXorgRecipe Alien-LuaRocks CleanModules DisableProgram FixInfo GuessProgramCase RemoveProgram SymlinkProgram UpgradeSystem Alien-PIP Corrections FilterColors GenBuildInformation HasCompatiblePackage MergeTree Rename SystemFind UseFlags Alien-RubyGems CreatePackage FilterLines GetAvailable Hashes NamingConventions RescueInstallPackage SystemInfo VerifyProgram
 
-debug: python
-	cd src; $(MAKE) debug
+.PHONY: all clean install
 
-python:
-	for f in $(PYTHON_LIBS); \
-	do libf=$(PYTHON_SITE)/$$f.py; \
-	   rm -f $$libf; ln -nfs ../../../bin/$$f $$libf; \
-	done
-	cd $(PYTHON_SITE) && \
+all: python_all
+	@$(MAKE) -C src
+
+python_all:
+	mkdir -p lib/$(PYTHON_SITE)
+	$(foreach PYTHON_LIB, $(PYTHON_LIBS), \
+		ln -nfs ../../../bin/$(PYTHON_LIB) lib/$(PYTHON_SITE)/$(PYTHON_LIB).py ; \
+	)
+	cp lib/python/*.py lib/$(PYTHON_SITE) 
+	cd lib/$(PYTHON_SITE) && \
 	for f in *.py; \
-	do python -c "import `basename $$f .py`"; \
+	do  python -c "import `basename $$f .py`"; \
 	done
 
-clean:
+python_clean:
+	rm -rf lib/$(dir $(PYTHON_SITE))
+
+clean: python_clean
+	@$(MAKE) -C src clean
+	@echo "Cleaning man pages"
+	$(foreach MAN_FILE, $(MAN_FILES), \
+		rm -f $(MAN_FILE) ; \
+	)
+	@echo "Cleaning binaries"
+	$(foreach EXE_FILE, $(EXEC_FILES), \
+		rm -f $(EXE_FILE) ; \
+	)
 	rm -rf Resources/FileHash*
-	find * -path "*~" -or -path "*/.\#*" -or -path "*.bak" | xargs rm -f
-	cd src && $(MAKE) clean
-	cd $(PYTHON_SITE) && rm -f *.pyc *.pyo
-	rm -f $(exec_files)
-	rm -rf share/man/man1
 
-manuals: $(man_files)
+python_install:
+	@echo "Installing python libraries"
+	mkdir -p lib/$(PYTHON_SITE)
+	cp -r lib/$(PYTHON_SITE) $(DESTDIR)/lib
+
+manuals: $(MAN_FILES)
 
-$(man_files): share/man/man1/%.1: bin/%
-	@mkdir -p share/man/man1
+$(MAN_FILES): %.1: %
+	@echo "Generating man page $@"
 	help2man --name=" " --source="GoboLinux" --no-info $< --output $@
 
-$(exec_files): bin/%: src/%
+$(EXEC_FILES): bin/%: src/%
 	cp -af $< $@
 	chmod a+x $@
 
-src/%: src/%.c
-	$(MAKE) -C src
+install_manuals: manuals
+	$(INSTALL_DIR) -d -m 755 $(DESTDIR)/share/man/man1
+	$(foreach MAN_FILE, $(MAN_FILES), \
+		$(INSTALL_FILE) -m 644 $(MAN_FILE) $(DESTDIR)/share/man/man1 ; \
+	)
+
+install_scripts:
+	@echo "Installing scripts"
+	$(INSTALL_DIR) -d -m 755 $(DESTDIR)/bin
+	$(foreach SCRIPT_FILE, $(SCRIPT_FILES), \
+	   	$(INSTALL_FILE) -m 755 bin/$(SCRIPT_FILE) $(DESTDIR)/bin ; \
+	)
+
+install_data:
+	@echo "Installing Data"
+	$(INSTALL_DIR) -d -m 755 $(DESTDIR)/Data 
+	cp -r Data $(DESTDIR)
+
+install_resources:
+	@echo "Installing Resources"
+	cp -r Resources $(DESTDIR) 
+	
+install_share_data:
+	@echo "Installing share data"
+	cp -rf share $(DESTDIR)
+
+install_functions:
+	@echo "Installing Functions"
+	cp -rf Functions $(DESTDIR)
 
-.PHONY: all python manuals debug clean
+prepare_install:
+	@echo "Installing $(PROGRAM) into $(DESTDIR)"
+	$(INSTALL_DIR) -m 755 $(DESTDIR)
 
-install: python $(exec_files) manuals
-	@echo "Installing Scripts into $(PREFIX)/$(DESTDIR)"
-	@install -d  -m 755 $(PREFIX)/$(DESTDIR)
-	@echo "Copying files ..."
-	@./bin/ListProgramFiles `pwd` | grep -v "^src" | grep -v "^Makefile" | \
-	cpio --pass-through --quiet --verbose --unconditional $(PREFIX)/$(DESTDIR)
+install: all prepare_install install_scripts install_data install_resources install_share_data install_functions python_install install_manuals
+	@$(MAKE) DESTDIR=$(DESTDIR) -C src install
+	@echo "Installed $(PROGRAM) into $(DESTDIR)"

lib/python2.7/site-packages/Alien.py renamed to lib/python/Alien.py

@@ -1,55 +1,74 @@
-
-MYCFLAGS = -O2 -Wall 
+DESTDIR?=
 CC = gcc
+CFLAGS = -O2 -Wall
+INSTALL_FILE=install -s
+INSTALL_DIR=install -d
 
-STATIC=-static
-ifeq ($(shell uname -s),Darwin)
-   STATIC=
-endif
-
+# Running on windows
 IS_CYGWIN=$(shell uname | grep -i cygwin &>/dev/null && echo yes || echo no)
 ifeq ($(IS_CYGWIN),yes)
    RM_EXE=-rm -f *.exe
 endif
 
+STATIC_CFLAGS = $(CFLAGS) -static
+ifneq ($(shell uname -s),Darwin)
+	STATIC_CFLAGS = $(CFLAGS) 
+endif
+
+# Add compiler options
+ifdef DEBUG
+	CFLAGS += -g -DDEBUG
+	STATIC_CFLAGS += -g -DDEBUG
+	INSTALL_FILE=install
+endif
+
+exec_files = $(patsubst %.c,%,$(wildcard *.c))
+
 dynamic_exec = BackgroundExec SuperUserName IsExecutable usleep LinkOrExpandAll List CommandNotFound GetSupportedFilesystems
 static_exec = RescueSymlinkProgram
 other_exec = FindDependencies Runner
 
 # first rule
-default: all
 
-.PHONY: all default
+.PHONY: all clean static debug install
 
 all: $(dynamic_exec) $(static_exec) $(other_exec)
 
 $(dynamic_exec): %: %.c
-	$(CC) $(MYCFLAGS) $< -o $@
-	@if [ -e "$@.exe" ]; then \
+	@echo "Compiling $@"
+	$(CC) $(CFLAGS) $< -o $@
+	if [ -e "$@.exe" ]; then \
 		mv $@.exe $@; \
 	fi
 
 $(static_exec): %: %.c
-	$(CC) $(MYCFLAGS) $< -o $@ $(STATIC)
-	@if [ -e "$@.exe" ]; then \
+	@echo "Compiling $@"
+	$(CC) $(STATIC_CFLAGS) $< -o $@
+	if [ -e "$@.exe" ]; then \
 		mv $@.exe $@; \
 	fi
 
 FindDependencies: %: %.c
-	$(CC) $(MYCFLAGS) $< -o $@ -DBUILD_MAIN
+	@echo "Compiling $@"
+	$(CC) $(CFLAGS) $< -o $@ -DBUILD_MAIN
 
 Runner: Runner.c FindDependencies.c
-	$(CC) $(MYCFLAGS) $^ -o $@
-	chmod 4755 $@
-
-debug: MYCFLAGS = -g -DDEBUG -Wall
-debug: all
-
-static: MYCFLAGS = $(STATIC)
-static: all
+	@echo "Compiling $@"
+	$(CC) $(CFLAGS) $^ -o $@
 
 clean:
+	@echo "Cleaning sources"
 	rm -f $(dynamic_exec) $(static_exec) $(other_exec) lib*.so lib*.so.* *.o
 	$(RM_EXE)
 
-.PHONY: all clean static debug install
+install: all
+	@echo "Installing compiled binaries"
+ifndef DESTDIR
+	$(error DESTDIR is not set)
+endif
+	$(INSTALL_DIR) -d -m 755 $(DESTDIR)/bin
+	$(foreach EXEC_FILE, $(exec_files), \
+		$(INSTALL_FILE) -m 755 $(EXEC_FILE) $(DESTDIR)/bin ; \
+	)
+	chmod 4755 $(DESTDIR)/bin/Runner
+