GitHub user Ormod opened a pull request:

    https://github.com/apache/zookeeper/pull/44

    Fix RPM package creation on recent distros

    The first commit comes from: 
http://mail-archives.apache.org/mod_mbox/zookeeper-user/201212.mbox/%[email protected]%3E
    
    The other two are issues I hit next when trying to build an RPM package. 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Ormod/zookeeper packaging

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zookeeper/pull/44.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #44
    
----
commit 10ea386b12c84b2c77fb7eb703f096e2ee8c2e9a
Author: Cédric Lejeune <[email protected]>
Date:   2015-09-09T15:22:22Z

    packaging: Fixes for RPM based packaging
    
    The install stage was removing BUILDROOT content:
    
    [rpm] + rm -rf /tmp/zkpython_build_rpm/BUILD
    
    Since BUILD and BUILDROOT are actually the same folder, everything is
    removed before being used.
    
    These commits were originally submitted at:
    
    
http://mail-archives.apache.org/mod_mbox/zookeeper-user/201212.mbox/%[email protected]%3E
    
    This commit simply reproduces them and credits Cédric Lejeune
    as the original author.

commit e80573748078e66b7c73e4413888014330511607
Author: Hannu Valtonen <[email protected]>
Date:   2015-09-09T15:26:08Z

    packaging: Remove "redhat" from rpm tool paths to work on newer distros
    
    All even semi-recent versions of Centos/RHEL originally had both
    defined until newer Fedoras for example no longer have the redhat
    directory at all, failing to build the RPM package.

commit fe10b24141e1952fb3a712363b10d2d8a9dd91f1
Author: Hannu Valtonen <[email protected]>
Date:   2015-09-11T12:50:59Z

    popd: remove invalid, unneeded argument
    
    Recent versions of bash actually return invalid argument and error
    out.
    
    While at it clean some whitespace in the file.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to