bugTransport Sample Protocol - Bugs: bug #24421, Can't build on solaris 2.8

 
 

bug #24421: Can't build on solaris 2.8

Submitted by:  MARTINSONS <martinsons>
Submitted on:  Wed 01 Oct 2008 09:01:22 AM UTC  
 
Category: Build SystemSeverity: 3 - Normal
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open
Release: 0.8.3Fixed Release: Unknown

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Wed 01 Oct 2008 01:18:51 PM UTC, comment #1:

Thank you very much for the detailed report.

Some comments for the one who will handle the bug:
the extra
TARGET_LINK_LIBRARIES(bb_tsp_provider ${ONCRPC_LIBRARY})
TARGET_LINK_LIBRARIES(bb_tsp_provider ${SOCKET_LIBRARY})
should be harmless on any platform since xxx_LIBRARY
should be void on non-concerned platforms.
This is harmless if there is at least one non void lib
in the target_link_libraries arguments.

so this could be (for the bb provider case):
TARGET_LINK_LIBRARIES(${_EXE} bb_tsp_provider ${SOCKET_LIBRARY} ${ONCRPC_LIBRARY})

Concerning Lex/Flex I don't think we NEED flex
standard lex should be OK, however I do not have
solaris lex at hand to test for supported options.

If you want to force a particular lex flavour usage
you should run ccmake and edit the LEX_PROGRAM entry,
it should be avoided to try to override the cmake
discovered value from within CMakeLists.txt.

concerning RPATH handling I think this should be handled
as an option because builtin rpath have drawbacks, in particular
if you update the libraries you depends on.

I may try to incorporate those change in CVS unless
someone want to go for it :=)

Eric NOULARD <erk>
Project Administrator
Wed 01 Oct 2008 09:01:22 AM UTC, original submission:

There is some problem to build TSP on Solaris 2.8 .

To fix that some modifications are needed :

-The include of stdint.h in bb_core.h must be removed.
-Then some definitions and link to libraries must be added in CMakeLists:
IF(CMAKE_SYSTEM_NAME STREQUAL "SunOS")
ADD_DEFINITIONS(-D_POSIX_PTHREAD_SEMANTICS)
TARGET_LINK_LIBRARIES(bb_tsp_provider ${ONCRPC_LIBRARY})
TARGET_LINK_LIBRARIES(bb_tsp_provider ${SOCKET_LIBRARY})
ENDIF(CMAKE_SYSTEM_NAME STREQUAL "SunOS")

in [TSP_SOURCE_DIR]/src/providers/ for bb_provider,generic_reader,res_reader and stub.For the consumers, just the links to libraries must be added for generic,res_writer,stdout,stdout_callback,test_group and tutorial.

-Concerning the ascii_writer consumer ,the option -o is not recognized by the system so the path to the flex command must be added:
IF(CMAKE_SYSTEM_NAME STREQUAL "SunOS")
SET(LEX_PROGRAM "/usr/local/bin/flex"
ENDIF(CMAKE_SYSTEM_NAME STREQUAL "SunOS")

-Finally,the RPATH must be forced to find some libraries,write the following in the main CMakeList:

# use, i.e. don't skip the full RPATH for the build tree
SET(CMAKE_SKIP_BUILD_RPATH FALSE)

# when building, don't use the install RPATH already
# (but later on when installing)
SET(CMAKE_BUILD_WITH_INSTALL_RPATH FALSE)

# the RPATH to be used when installing
SET(CMAKE_INSTALL_RPATH "${CMAKE_INSTALL_PREFIX}/lib")

# add the automatically determined parts of the RPATH
# which point to directories outside the build tree to the #install RPATH
SET(CMAKE_INSTALL_RPATH_USE_LINK_PATH TRUE)

Now, unless i miss something, everithing is working.

MARTINSONS <martinsons>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by erk (Posted a comment)
  • -unavailable- added by martinsons (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    No Changes Have Been Made to This Item

    Back to the top


    Powered by Savane 3.1-cleanup1