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 System Severity:  3 - Normal
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open
Release:  0.8.3 Fixed 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 Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

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

    Only logged-in users can vote.

     

     

     

    No changes have been made to this item

    Back to the top


    Powered by Savane 3.5