zookeeper-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [zookeeper] symat opened a new pull request #1113: ZOOKEEPER-3530: add new artifact for compiled c-client code
Date Thu, 10 Oct 2019 14:29:05 GMT
symat opened a new pull request #1113: ZOOKEEPER-3530: add new artifact for compiled c-client
code
URL: https://github.com/apache/zookeeper/pull/1113
 
 
   
   During the old ZooKeeper 3.4 ant builds (ant package-native), there was an artifact (zookeeper-<version>-lib.tar.gz)
created just for the C-client, with the following content:
   
   ```
   usr
     |--- bin
            |--- cli_mt
            |--- cli_st
            |--- load_gen
     |--- include
            |--- zookeeper
                   |--- proto.h
                   |--- recordio.h
                   |--- zookeeper.h
                   |--- zookeeper.jute.h
                   |--- zookeeper_log.h
                   |--- zookeeper_version.h
     |--- lib
            |--- libzookeeper_mt.a
            |--- libzookeeper_mt.la
            |--- libzookeeper_mt.so
            |--- libzookeeper_mt.so.2
            |--- libzookeeper_mt.so.2.0.0
            |--- libzookeeper_st.a
            |--- libzookeeper_st.la
            |--- libzookeeper_st.so
            |--- libzookeeper_st.so.2
            |--- libzookeeper_st.so.2.0.0
   ```
   
   Currently with maven, when we are generating a tarball during full-build then the C-client
is not getting archived. In [PR-1078](https://github.com/apache/zookeeper/pull/1078) we discussed
that we should re-introduce the apache-zookeeper-<version>-lib.tar.gz artifact.
   
   The goals of this PR are:
   - re-introduce the 'lib' artifact, with the same structure we had for the older zookeeper
3.4.x ant generated tar file
   - we should also add the LICENSE.txt file to the archive (it was missing from the 3.4.x
version tar.gz file)
   - the new artifact should be generated only when the full-build profile is set for maven
   - we should also update the README_packaging.md file
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message