Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Feb 5, 2013

A couple small fixes to the cpp docs and example

@joshmoore
Copy link
Member

👍 Thanks, @JesseCorrington. Changes tested by copying yourcode.cpp and Makefile to target directory and compiling. That works fine, so I'd say this is ready to go for 4.4.6

In terms of usability, there might be changes you want to make moving forward:

# Building is fine works fine
@icecpp-develop target/OMERO.cpp-4.4.4-DEV-ice33-darwin-gcc-4.2.1-64dbg $ make
g++  -c -o yourcode.o yourcode.cpp -I./include -I/usr/local/opt/zeroc-ice33/include 

# For some reason, the libpath for lib/libomero_client.dylib is unhappy
@icecpp-develop target/OMERO.cpp-4.4.4-DEV-ice33-darwin-gcc-4.2.1-64dbg $ make run
g++ -o yourcode yourcode.o -L./lib -L/usr/local/opt/zeroc-ice33/lib -L/usr/local/opt/zeroc-ice33/lib64 -lIce -lIceUtil -lGlacier2 -lomero_client -lstdc++
ld: warning: directory '/usr/local/opt/zeroc-ice33/lib64' following -L not found
LD_LIBRARY_PATH=":/usr/local/opt/zeroc-ice33/lib:/usr/local/opt/zeroc-ice33/lib64:./lib" ./yourcode --Ice.Config=../etc/ice.config
dyld: Library not loaded: libomero_client.dylib
  Referenced from: target/OMERO.cpp-4.4.4-DEV-ice33-darwin-gcc-4.2.1-64dbg/./yourcode
  Reason: image not found
make: *** [run] Trace/BPT trap

# This fixes it.
@icecpp-develop target/OMERO.cpp-4.4.4-DEV-ice33-darwin-gcc-4.2.1-64dbg $ ln -s lib/libomero_client.dylib .

# This may just be an issue of my git directory being a symlink.
@icecpp-develop target/OMERO.cpp-4.4.4-DEV-ice33-darwin-gcc-4.2.1-64dbg $ make run
LD_LIBRARY_PATH=":/usr/local/opt/zeroc-ice33/lib:/usr/local/opt/zeroc-ice33/lib64:./lib" ./yourcode --Ice.Config=../etc/ice.config
terminate called after throwing an instance of 'Ice::FileException'
  what():  PropertiesI.cpp:291: Ice::FileException:
file exception: No such file or directory
path: ../etc/ice.config

I'm happy to file a ticket or you can.

joshmoore added a commit that referenced this pull request Feb 6, 2013
@joshmoore joshmoore merged commit 50254df into ome:dev_4_4 Feb 6, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants