rlarranaga / bafprp

Automatically exported from code.google.com/p/bafprp
0 stars 0 forks source link

ld: unknown option: -obin/bafprp on on Os X #34

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
On OS X when I launch the compilation:
make -f Makefile_NoSQL

I get the following error:
ld: unknown option: -obin/bafprp

It's caused by a missing space between the -o option of the linker and the bin 
path.

Here is a fix which works on Linux too.

--- a/Makefile_NoSQL
+++ b/Makefile_NoSQL
@@ -66,11 +66,11 @@ rebuild: clean all

 release: $(OBJ_LINUX)
        mkdir -p $(BIN_DIR)
-       $(CPP) $^ $(LDFLAGS) -s -o$(BIN_DIR)/$(BINARY)
+       $(CPP) $^ $(LDFLAGS) -s -o $(BIN_DIR)/$(BINARY)^M

 debug: $(DOBJ_LINUX)
        mkdir -p $(BIN_DIR)
-       $(CPP) $^ $(LDFLAGS) -o$(BIN_DIR)/$(BINARY)_d
+       $(CPP) $^ $(LDFLAGS) -o $(BIN_DIR)/$(BINARY)_d^M

 install: release
        cp $(BIN_DIR)/$(BINARY) /usr/bin/$(BINARY)

Original issue reported on code.google.com by chab...@gmail.com on 29 Mar 2011 at 11:43

GoogleCodeExporter commented 8 years ago
Thanks for reporting this, Ill get it fixed in my next commit

Original comment by charless...@gmail.com on 30 Mar 2011 at 2:25

GoogleCodeExporter commented 8 years ago
Fixed the issue, I have not tried building yet but what harm can a new space do 
haha.

Original comment by charless...@gmail.com on 29 Aug 2011 at 2:53

GoogleCodeExporter commented 8 years ago
On my side I have tested it on OS X and Linux.

Original comment by chab...@gmail.com on 29 Aug 2011 at 2:03

GoogleCodeExporter commented 8 years ago
thanks for trying it out, Ill just mark this verified then :)

Original comment by charless...@gmail.com on 29 Aug 2011 at 2:45