[ 
https://issues.apache.org/jira/browse/HDFS-9408?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15001169#comment-15001169
 ] 

Haohui Mai commented on HDFS-9408:
----------------------------------

Looks good to me overall.

{code}
+
+set(EMPTY_FILE_CC ${CMAKE_CURRENT_BINARY_DIR}${CMAKE_FILES_DIRECTORY}/empty.cc)
+file(WRITE ${EMPTY_FILE_CC} "")
{code}

Some comments on the hack will be useful.

{code}
+SET(HDFSPP_SUBLIBS bindings_c fs rpc reader proto common)
+IF(${CMAKE_SYSTEM_NAME} MATCHES "Linux")
+  SET(HDFSPP_SUBLIBS -Wl,--whole-archive ${HDFSPP_SUBLIBS} 
-Wl,--no-whole-archive)
+ENDIF(${CMAKE_SYSTEM_NAME} MATCHES "Linux")
{code}

It makes sense to play around with the order of the {{HDFSPP_SUBLIBS}} to avoid 
the linker flags hacks. It might make sense to substitute {{HDFSPP}} to 
{{LIBHDFSPP}} in the files.


> extend the build system to produce static and dynamic libhdfspp libs
> --------------------------------------------------------------------
>
>                 Key: HDFS-9408
>                 URL: https://issues.apache.org/jira/browse/HDFS-9408
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: hdfs-client
>            Reporter: Stephen
>            Assignee: Stephen
>             Fix For: HDFS-8707
>
>         Attachments: HDFS-9408.HDFS-8707.001.patch, 
> HDFS-9408.HDFS-8707.002.patch, HDFS-9408.HDFS-8707.003.patch, 
> HDFS-9408.HDFS-8707.004.patch
>
>
> Generate static and dynamic libhdfspp libraries for use by other applications.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to