BewareMyPower opened a new pull request, #17064:
URL: https://github.com/apache/pulsar/pull/17064

   ### Motivation
   
   For branch-2.11 and master branches, the rpm and deb packaging is
   broken.
   
   > /usr/bin/env: 'python3': No such file or directory
   
   It's caused by [PIP-155](https://github.com/apache/pulsar/pull/15376),
   which removes the Python2 support. In addition, the Dockerfiles of `rpm`
   and `deb` subdirectories of `pulsar-client-cpp/pkg` don't install Boost
   for Python3 well. But the rpm build disables the
   `BUILD_PYTHON_WRAPPER` option while the deb build doesn't.
   
   ### Modifications
   
   - Fix Dockerfiles under `pkg` and `rpm` subdirectories and speed up the
     `docker build` process:
      - Install python3 to fix the bug
      - Download the CMake 3.24.0 binaries directly instead of building
        from source
      - Upgrade Boost to 1.79 so that only headers are required (the
        boost::regex module is required for CentOS 7 because the default
        GCC is 4.8 that there is something wrong with std::regex)
      - Add `-j8` option when compiling some large dependencies that uses 8
        threads to speed the compilation
      - Remove the `-v` option of `tar` to avoid showing too many info
   - Check the `BUILD_IMAGE` environment variable in `docker-build-xxx.sh`
     under `rpm` and `deb` subdirectories. If it's defined, build the image
     rather than pull the image from DockerHub before executing the build
     script. Because currently only a few committers have the permission to
     push the Docker images.
   - Add two workflows to build RPM and DEB packages if the files under
     `pulsar-client-cpp` changed. They build the docker images and run the
     build script for RPM and DEB packaging.
   
   ### Documentation
   
   Check the box below or label this PR directly.
   
   Need to update docs? 
   
   - [ ] `doc-required` 
   (Your PR needs to update docs and you will update later)
     
   - [x] `doc-not-needed` 
   I will update the docs in Wiki pages. The dos in the repository don't need 
update.
     
   - [ ] `doc` 
   (Your PR contains doc changes)
   
   - [ ] `doc-complete`
   (Docs have been already added)


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to