search

farsounder / protobuf-matlab

Automatically exported from code.google.com/p/protobuf-matlab
Other
26 stars 13 forks source link

Installation instructions incomplete and ambiguous. #1

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Hello,

First of all, many thanks for producing this library, so that protobuf messages 
can be used as a method to store/transmit data from matlab to other processes.  
We use this library for a matlab simulation with visualization in other 
processes. 

I'd like to raise an issue with the documentation on installation (README.txt) 
and compiling the protoc binary. If it doesn't end up in the actual 
documentation, then at least it will be available as an issue for others who 
may be facing the same problems.

Step 1 is correct and unambiguous.

Step 2 should clarify that src/farsounder/protobuf/compiler/matlab/* should 
actually end up as "src/farsounder" in the protobuf 2.4.0a tree as well, 
because now it can be interpreted as copying files to root or src.

Step 3 is correct, but please be aware that main.cc is in that directory and 
not in 'src' in the 2.4.0a tree. ( I made that mistake ).

Step 4 doesn't work yet, because Makefile.am hasn't been edited.

Explanation:
============
- Open Makefile.am in 'src'.
- There are 2 locations to add "matlab_generator.*" to:
     matlab_generator.h  after "python_generator.h"
     matlab_generator.cc after "python_generator.cc"
  Don't forget to add a backslash character after python_generator.*.

- Now run ./autogen.sh
- Then run ./configure
- Then run make
- Your updated 'protoc' is now in "src" or "src/.libs".

This does give you the required *.m files that you'll need to use protobuf 
messages in matlab. There's another issue with actual use that I hope the 
authors can comment on. This will be highlighted in another issue.

Original issue reported on code.google.com by gtoons...@gmail.com on 13 Oct 2011 at 4:20

GoogleCodeExporter commented 9 years ago 
Thanks for the comments. I've updated the README and added a Makefile with 
those changes. 

Please see 
http://code.google.com/p/protobuf-matlab/source/detail?r=8264ec14819c8b6b7bbbebd
56169b117e5c225bc

Original comment by evan.lap...@gmail.com on 21 Oct 2011 at 7:50