-
Notifications
You must be signed in to change notification settings - Fork 129
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Doesn't seem to work on osx. #1
Comments
Am 16.07.2013 um 01:58 schrieb vito-c notifications@github.com:
well try this, without the arguments you cant see what failed:
very likely its the different naming scheme for shared libs on osx, .dynload or so - the Apple docs and related examples should help -m |
ok I'm working on adding osx compatibility in my fork and send you a pull request :) make: *** No rule to make target So it's a little unlcear what this is supposed to do: What command is supposed to be run by the test_json target? Thanks! |
Am 16.07.2013 um 04:00 schrieb vito-c notifications@github.com:
yeah, that -lprotobuf should be replace by the full pathname to the protobuf library, looks like a mistake but those are generic Makefile issues
|
took a little bit more mucking with but i got it working thanks for the quick responses and help :) Can you pull it down and run on nix to make sure I didn't completely foo bar it on nix too? |
@vito-c What was the solution? I have the same problem. I substituted the full path (/usr/local/lib) for -lprotobuf and that did not help? |
all: libjson2pb.so test_json clean: test_json: test.pb.o libjson2pb.so test_json.o test_json.o: test.pb.h json2pb.o: bin2ascii.h libjson2pb.so: json2pb.o test.pb.h test.pb.cc: test.proto |
does this work on osx?
jansson 2.4 is installed here in my /usr/local area ... i do see the json2pb.o so that might just mean it's failing on the unix
The text was updated successfully, but these errors were encountered: