rxi
build.sh tweaks
- support environmentally-declared CFLAGS and LDFLAGS
- indicate build failure with an error exit code
Any reason this shouldn't just be done as a makefile instead? Would be nice to have it more declarative.
Totally agree it should be a Makefile, just wanted to make it a less intrusive change.
As a GNU / Linux user, I agree about the use of a makefile; it's my standard procedure for small to medium projects.
But what I'm wondering is the following: since it's an app that can be used on multiple environments, would not it make more sense to use something like Meson?
About CMake, I have no idea how to use it, but Meson is Python and I already know how to use it so...yeah.
Any reason this shouldn't just be done as a makefile instead? Would be nice to have it more declarative. @technomancy
CFLAGS ?= -Wall -O3 -g -std=gnu11
LDFLAGS ?= -lSDL2 -lm
CPPFLAGS ?= -Isrc
src/lib/stb/%.o: CPPFLAGS =
SRCS := $(wildcard src/*.c src/api/*.c src/lib/*.c src/lib/stb/*.c src/lib/lua52/*.c)
OBJS := $(patsubst %.c,%.o,$(SRCS))
ifeq ($(OS),Windows_NT)
OUT := lite.exe
CC := x86_64-w64-mingw32-gcc
src/lib/lua52/%.o: CPPFLAGS = -DLUA_USE_POPEN
CPPFLAGS += -Iwinlib/SDL2-2.0.10/x86_64-w64-mingw32/include
LDFLAGS += -Lwinlib/SDL2-2.0.10/x86_64-w64-mingw32/lib -lmingw32 -lSDL2main -mwindows
OBJS += res.res
else
OUT := lite
src/lib/lua52/%.o: CPPFLAGS = -DLUA_USE_POSIX
endif
$(OUT): $(OBJS)
$(LINK.o) $^ $(LOADLIBES) $(LDLIBS) $(OUTPUT_OPTION)
res.res: res.rc
x86_64-w64-mingw32-windres $< -O coff $(OUTPUT_OPTION)
clean:
-rm -f $(OUT) $(OBJS)
.PHONY: clean
(I haven't tested the Windows part, though.)
Any reason this shouldn't just be done as a makefile instead? Would be nice to have it more declarative. @technomancy
CFLAGS ?= -Wall -O3 -g -std=gnu11 LDFLAGS ?= -lSDL2 -lm CPPFLAGS ?= -Isrc src/lib/stb/%.o: CPPFLAGS = SRCS := $(wildcard src/*.c src/api/*.c src/lib/*.c src/lib/stb/*.c src/lib/lua52/*.c) OBJS := $(patsubst %.c,%.o,$(SRCS)) ifeq ($(OS),Windows_NT) OUT := lite.exe CC := x86_64-w64-mingw32-gcc src/lib/lua52/%.o: CPPFLAGS = -DLUA_USE_POPEN CPPFLAGS += -Iwinlib/SDL2-2.0.10/x86_64-w64-mingw32/include LDFLAGS += -Lwinlib/SDL2-2.0.10/x86_64-w64-mingw32/lib -lmingw32 -lSDL2main -mwindows OBJS += res.res else OUT := lite src/lib/lua52/%.o: CPPFLAGS = -DLUA_USE_POSIX endif $(OUT): $(OBJS) $(LINK.o) $^ $(LOADLIBES) $(LDLIBS) $(OUTPUT_OPTION) res.res: res.rc x86_64-w64-mingw32-windres $< -O coff $(OUTPUT_OPTION) clean: -rm -f $(OUT) $(OBJS) .PHONY: clean(I haven't tested the Windows part, though.)
Is it possible for you to add an install section, I can't figure out how to install it post compilation.
I've added Makefile in #257, but I haven't tested the Windows part. It should work on Linux/MacOS, though in order to properly install the package, there are more changes needed.
