You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by Sijie Guo <gu...@gmail.com> on 2020/01/20 21:19:21 UTC

[DISCUSS] Homebrew libpulsar

Hi all,

It is my first time to realize that there is a libpulsar entry in homebrew.
(https://github.com/apache/pulsar/issues/6091). Are Pulsar committers
responsible for updating this? Or homebrew team is updating that entry?

If the committers are responsible for updating it, it should have been
discussed in the mailing list before and we need to add an entry in release
doc about this.

Thanks,
Sijie

Re: [DISCUSS] Homebrew libpulsar

Posted by Sijie Guo <gu...@gmail.com>.
If that's the case, we should document the process in the release process
documentation. Can you help update the wiki page?

- Sijie

On Tue, Jan 21, 2020 at 1:23 PM Matteo Merli <ma...@gmail.com> wrote:

> In general, Homebrew will only accept updates to the formula once a
> particular software version is already released.
> We could certainly include it as a post-step in our release process,
> to do after the official release announcement is done.
>
>
>
> --
> Matteo Merli
> <ma...@gmail.com>
>
> On Tue, Jan 21, 2020 at 11:08 AM Sijie Guo <gu...@gmail.com> wrote:
> >
> > Thanks Dave.
> >
> > So the question will be - shall the release manager be responsible for
> > updating the homebrew entry? If so, we need to include it in the release
> > procedure.
> >
> > Thanks,
> > Sijie
> >
> > On Mon, Jan 20, 2020 at 2:03 PM Dave Fisher <wa...@apache.org> wrote:
> >
> > > Hi -
> > >
> > > I’ve played with Homebrew before and from the git log I can see that
> > > Matteo created the initial version, but for the most part the Homebrew
> > > community has maintained the script.
> > >
> > > Here is the current script:
> > >
> > > Formula/libpulsar.rb
> > > class Libpulsar < Formula
> > >   desc "Apache Pulsar C++ library"
> > >   homepage "https://pulsar.apache.org/"
> > >   url "
> > >
> https://www.apache.org/dyn/closer.cgi?path=pulsar/pulsar-2.4.2/apache-pulsar-2.4.2-src.tar.gz
> > > "
> > >   sha256
> "4b543932db923aa135c4d54b9122bcbdfc67bd73de641f9fffbc9a4ddf3430ae"
> > >   revision 1
> > >
> > >   bottle do
> > >     cellar :any
> > >     sha256
> > > "13c37b77dee18f4bba454484b4426a6f3dad27f902e0793a56a6358897ab4f3f" =>
> > > :catalina
> > >     sha256
> > > "ff0090b77842ac6034c4a425438f8d5b401164da4bada7ac11c4963dcdaa3a28" =>
> > > :mojave
> > >     sha256
> > > "d73f612edf73a351ab7f90776c24670c4c77b85fd95e37999e863ff27daf89ef" =>
> > > :high_sierra
> > >   end
> > >
> > >   depends_on "cmake" => :build
> > >   depends_on "pkg-config" => :build
> > >   depends_on "boost"
> > >   depends_on "openssl@1.1"
> > >   depends_on "protobuf"
> > >   depends_on "snappy"
> > >   depends_on "zstd"
> > >
> > >   def install
> > >     cd "pulsar-client-cpp" do
> > >       system "cmake", ".", *std_cmake_args,
> > >                       "-DBUILD_TESTS=OFF",
> > >                       "-DBUILD_PYTHON_WRAPPER=OFF",
> > >
>  "-DBoost_INCLUDE_DIRS=#{Formula["boost"].include}",
> > >
> > > "-DProtobuf_INCLUDE_DIR=#{Formula["protobuf"].include}",
> > >
> > > "-DProtobuf_LIBRARIES=#{Formula["protobuf"].lib}/libprotobuf.dylib"
> > >       system "make", "pulsarShared", "pulsarStatic"
> > >       system "make", "install"
> > >     end
> > >   end
> > >
> > >   test do
> > >     (testpath/"test.cc").write <<~EOS
> > >       #include <pulsar/Client.h>
> > >
> > >       int main (int argc, char **argv) {
> > >         pulsar::Client client("pulsar://localhost:6650");
> > >         return 0;
> > >       }
> > >     EOS
> > >     system ENV.cxx, "test.cc", "-L#{lib}", "-lpulsar", "-o", "test"
> > >     system "./test"
> > >   end
> > > end
> > >
> > > Here is the log w/o the bottling.
> > >
> > > % git log master -- Formula/libpulsar.rb
> > > commit e4ed6752b0cab4c98ae72f81b2bd7160412bb938
> > > Author: FX Coudert <fx...@gmail.com>
> > > Date:   Thu Dec 26 16:17:50 2019 +0100
> > >
> > >     libpulsar: revision for boost
> > >
> > > commit 07bd5db82526c43a0e5004f690f3afbda4b89c7c
> > > Author: Matteo Merli <mm...@apache.org>
> > > Date:   Thu Dec 19 11:38:56 2019 -0800
> > >
> > >     libpulsar 2.4.2
> > >
> > >     Closes #48088.
> > >
> > >     Signed-off-by: FX Coudert <fx...@gmail.com>
> > >
> > > commit 3e6adba69f61da1b6fee5ba49199d299878f0db1
> > > Author: Austin Shalit <au...@gmail.com>
> > > Date:   Wed Nov 27 23:32:07 2019 -0800
> > >
> > >     libpulsar: revision bump
> > >
> > > commit fc6e1f2d6179379740466631eb0dffd56b404f5c
> > > Author: Rui Chen <rc...@meetup.com>
> > > Date:   Thu Oct 3 13:05:09 2019 -0400
> > >
> > >     libpulsar: revision for protobuf
> > >
> > > commit 11b3aa3dbe8a76716784cda9f351a28b49e9dfd0
> > > Author: Henry Fredrick Schreiner <he...@cern.ch>
> > > Date:   Mon Sep 2 20:47:07 2019 -0400
> > >
> > >     libpulsar 2.4.1: revision for boost 1.71.0
> > >
> > > commit 0d14181d6cbdd044eeec9b78e1d4b7d766ff5f1d
> > > Author: Henry Fredrick Schreiner <he...@cern.ch>
> > > Date:   Sun Sep 1 19:56:04 2019 -0400
> > >
> > >     libpulsar: Keep from linking to snappy
> > >
> > >     Closes #43761.
> > >
> > >     Signed-off-by: FX Coudert <fx...@gmail.com>
> > >
> > > commit d9288916f84c02730c9ba800eb12550cb1b5b865
> > > Author: FX Coudert <fx...@gmail.com>
> > > Date:   Thu Aug 29 10:28:00 2019 +0200
> > >
> > >     libpulsar: move to OpenSSL 1.1
> > >
> > > commit 84d41c55af723989ccfd61161fc014c69f62da5c
> > > Author: Matteo Merli <mm...@apache.org>
> > > Date:   Wed Aug 28 11:26:50 2019 -0700
> > >
> > >     libpulsar 2.4.0
> > >
> > >     Closes #43555.
> > >
> > >     Signed-off-by: FX Coudert <fx...@gmail.com>
> > >
> > > commit 7e24d72cbe366889612c2acc9d230d7263db4122
> > > Author: Chongyu Zhu <i...@lembacon.com>
> > > Date:   Wed May 29 15:24:44 2019 +0800
> > >
> > >     libpulsar: revision for protobuf
> > >
> > > commit 42a18870f92e42a787692c840d00cb5731ae6a5c
> > > Author: FX Coudert <fx...@gmail.com>
> > > Date:   Fri Apr 12 21:19:13 2019 +0200
> > >
> > >     libpulsar: revision and patch for boost
> > >
> > > commit 167f3474dd8ca620ab3e9d12513e88e499fe1ca2
> > > Author: Steven Peters <sc...@openrobotics.org>
> > > Date:   Sat Jun 1 15:36:17 2019 -0700
> > >
> > >     libpulsar 2.3.2
> > >
> > >     Closes #40591.
> > >
> > >     Signed-off-by: Steven Peters <sc...@openrobotics.org>
> > >
> > > commit 1b3b8c15319503ff8a33e9627c7a46d5f8d2546f
> > > Author: FX Coudert <fx...@gmail.com>
> > > Date:   Mon Apr 15 17:40:36 2019 +0200
> > >
> > >     libpulsar 2.3.1
> > >
> > >     Closes #38947.
> > >
> > >     Signed-off-by: FX Coudert <fx...@gmail.com>
> > >
> > > commit 3c0b62cc0288f6d5dfb0ed9f682d6e245d1e3cfe
> > > Author: Steven Peters <sc...@openrobotics.org>
> > > Date:   Mon Mar 4 12:41:16 2019 -0800
> > >
> > >     libpulsar: revision for boost
> > >
> > > commit cac1d53d2254004b2082a2d501f772fcaf77b8c0
> > > Author: Chongyu Zhu <i...@lembacon.com>
> > > Date:   Sat Mar 2 23:50:28 2019 +0800
> > >
> > >     libpulsar: revision for protobuf
> > >
> > > commit fcc845573eef8564c7fac9ddba8a9e660b2d06f7
> > > Author: Matteo Merli <mm...@apache.org>
> > > Date:   Wed Feb 20 22:32:16 2019 -0800
> > >
> > >     libpulsar 2.3.0
> > >
> > >     Closes #37151.
> > >
> > >     Signed-off-by: FX Coudert <fx...@gmail.com>
> > >
> > > commit 24c06c4c49374136d484ef8a000b1c1626f2bf79
> > > Author: Viktor Szakats <co...@vszakats.net>
> > > Date:   Fri Jan 18 01:50:31 2019 +0000
> > >
> > >     libpulsar: use common/short form of apache url
> > >
> > > commit c2f34710ae2d378fbea076661016b87b31ce35af
> > > Author: Matteo Merli <mm...@apache.org>
> > > Date:   Wed Oct 31 15:28:10 2018 -0700
> > >
> > >     libpulsar 2.2.1 (new formula)
> > >
> > >     Closes #35809.
> > >
> > >     Signed-off-by: FX Coudert <fx...@gmail.com>
> > >
> > > > On Jan 20, 2020, at 1:19 PM, Sijie Guo <gu...@gmail.com> wrote:
> > > >
> > > > Hi all,
> > > >
> > > > It is my first time to realize that there is a libpulsar entry in
> > > homebrew.
> > > > (https://github.com/apache/pulsar/issues/6091). Are Pulsar
> committers
> > > > responsible for updating this? Or homebrew team is updating that
> entry?
> > > >
> > > > If the committers are responsible for updating it, it should have
> been
> > > > discussed in the mailing list before and we need to add an entry in
> > > release
> > > > doc about this.
> > > >
> > > > Thanks,
> > > > Sijie
> > >
> > >
>

Re: [DISCUSS] Homebrew libpulsar

Posted by Matteo Merli <ma...@gmail.com>.
In general, Homebrew will only accept updates to the formula once a
particular software version is already released.
We could certainly include it as a post-step in our release process,
to do after the official release announcement is done.



--
Matteo Merli
<ma...@gmail.com>

On Tue, Jan 21, 2020 at 11:08 AM Sijie Guo <gu...@gmail.com> wrote:
>
> Thanks Dave.
>
> So the question will be - shall the release manager be responsible for
> updating the homebrew entry? If so, we need to include it in the release
> procedure.
>
> Thanks,
> Sijie
>
> On Mon, Jan 20, 2020 at 2:03 PM Dave Fisher <wa...@apache.org> wrote:
>
> > Hi -
> >
> > I’ve played with Homebrew before and from the git log I can see that
> > Matteo created the initial version, but for the most part the Homebrew
> > community has maintained the script.
> >
> > Here is the current script:
> >
> > Formula/libpulsar.rb
> > class Libpulsar < Formula
> >   desc "Apache Pulsar C++ library"
> >   homepage "https://pulsar.apache.org/"
> >   url "
> > https://www.apache.org/dyn/closer.cgi?path=pulsar/pulsar-2.4.2/apache-pulsar-2.4.2-src.tar.gz
> > "
> >   sha256 "4b543932db923aa135c4d54b9122bcbdfc67bd73de641f9fffbc9a4ddf3430ae"
> >   revision 1
> >
> >   bottle do
> >     cellar :any
> >     sha256
> > "13c37b77dee18f4bba454484b4426a6f3dad27f902e0793a56a6358897ab4f3f" =>
> > :catalina
> >     sha256
> > "ff0090b77842ac6034c4a425438f8d5b401164da4bada7ac11c4963dcdaa3a28" =>
> > :mojave
> >     sha256
> > "d73f612edf73a351ab7f90776c24670c4c77b85fd95e37999e863ff27daf89ef" =>
> > :high_sierra
> >   end
> >
> >   depends_on "cmake" => :build
> >   depends_on "pkg-config" => :build
> >   depends_on "boost"
> >   depends_on "openssl@1.1"
> >   depends_on "protobuf"
> >   depends_on "snappy"
> >   depends_on "zstd"
> >
> >   def install
> >     cd "pulsar-client-cpp" do
> >       system "cmake", ".", *std_cmake_args,
> >                       "-DBUILD_TESTS=OFF",
> >                       "-DBUILD_PYTHON_WRAPPER=OFF",
> >                       "-DBoost_INCLUDE_DIRS=#{Formula["boost"].include}",
> >
> > "-DProtobuf_INCLUDE_DIR=#{Formula["protobuf"].include}",
> >
> > "-DProtobuf_LIBRARIES=#{Formula["protobuf"].lib}/libprotobuf.dylib"
> >       system "make", "pulsarShared", "pulsarStatic"
> >       system "make", "install"
> >     end
> >   end
> >
> >   test do
> >     (testpath/"test.cc").write <<~EOS
> >       #include <pulsar/Client.h>
> >
> >       int main (int argc, char **argv) {
> >         pulsar::Client client("pulsar://localhost:6650");
> >         return 0;
> >       }
> >     EOS
> >     system ENV.cxx, "test.cc", "-L#{lib}", "-lpulsar", "-o", "test"
> >     system "./test"
> >   end
> > end
> >
> > Here is the log w/o the bottling.
> >
> > % git log master -- Formula/libpulsar.rb
> > commit e4ed6752b0cab4c98ae72f81b2bd7160412bb938
> > Author: FX Coudert <fx...@gmail.com>
> > Date:   Thu Dec 26 16:17:50 2019 +0100
> >
> >     libpulsar: revision for boost
> >
> > commit 07bd5db82526c43a0e5004f690f3afbda4b89c7c
> > Author: Matteo Merli <mm...@apache.org>
> > Date:   Thu Dec 19 11:38:56 2019 -0800
> >
> >     libpulsar 2.4.2
> >
> >     Closes #48088.
> >
> >     Signed-off-by: FX Coudert <fx...@gmail.com>
> >
> > commit 3e6adba69f61da1b6fee5ba49199d299878f0db1
> > Author: Austin Shalit <au...@gmail.com>
> > Date:   Wed Nov 27 23:32:07 2019 -0800
> >
> >     libpulsar: revision bump
> >
> > commit fc6e1f2d6179379740466631eb0dffd56b404f5c
> > Author: Rui Chen <rc...@meetup.com>
> > Date:   Thu Oct 3 13:05:09 2019 -0400
> >
> >     libpulsar: revision for protobuf
> >
> > commit 11b3aa3dbe8a76716784cda9f351a28b49e9dfd0
> > Author: Henry Fredrick Schreiner <he...@cern.ch>
> > Date:   Mon Sep 2 20:47:07 2019 -0400
> >
> >     libpulsar 2.4.1: revision for boost 1.71.0
> >
> > commit 0d14181d6cbdd044eeec9b78e1d4b7d766ff5f1d
> > Author: Henry Fredrick Schreiner <he...@cern.ch>
> > Date:   Sun Sep 1 19:56:04 2019 -0400
> >
> >     libpulsar: Keep from linking to snappy
> >
> >     Closes #43761.
> >
> >     Signed-off-by: FX Coudert <fx...@gmail.com>
> >
> > commit d9288916f84c02730c9ba800eb12550cb1b5b865
> > Author: FX Coudert <fx...@gmail.com>
> > Date:   Thu Aug 29 10:28:00 2019 +0200
> >
> >     libpulsar: move to OpenSSL 1.1
> >
> > commit 84d41c55af723989ccfd61161fc014c69f62da5c
> > Author: Matteo Merli <mm...@apache.org>
> > Date:   Wed Aug 28 11:26:50 2019 -0700
> >
> >     libpulsar 2.4.0
> >
> >     Closes #43555.
> >
> >     Signed-off-by: FX Coudert <fx...@gmail.com>
> >
> > commit 7e24d72cbe366889612c2acc9d230d7263db4122
> > Author: Chongyu Zhu <i...@lembacon.com>
> > Date:   Wed May 29 15:24:44 2019 +0800
> >
> >     libpulsar: revision for protobuf
> >
> > commit 42a18870f92e42a787692c840d00cb5731ae6a5c
> > Author: FX Coudert <fx...@gmail.com>
> > Date:   Fri Apr 12 21:19:13 2019 +0200
> >
> >     libpulsar: revision and patch for boost
> >
> > commit 167f3474dd8ca620ab3e9d12513e88e499fe1ca2
> > Author: Steven Peters <sc...@openrobotics.org>
> > Date:   Sat Jun 1 15:36:17 2019 -0700
> >
> >     libpulsar 2.3.2
> >
> >     Closes #40591.
> >
> >     Signed-off-by: Steven Peters <sc...@openrobotics.org>
> >
> > commit 1b3b8c15319503ff8a33e9627c7a46d5f8d2546f
> > Author: FX Coudert <fx...@gmail.com>
> > Date:   Mon Apr 15 17:40:36 2019 +0200
> >
> >     libpulsar 2.3.1
> >
> >     Closes #38947.
> >
> >     Signed-off-by: FX Coudert <fx...@gmail.com>
> >
> > commit 3c0b62cc0288f6d5dfb0ed9f682d6e245d1e3cfe
> > Author: Steven Peters <sc...@openrobotics.org>
> > Date:   Mon Mar 4 12:41:16 2019 -0800
> >
> >     libpulsar: revision for boost
> >
> > commit cac1d53d2254004b2082a2d501f772fcaf77b8c0
> > Author: Chongyu Zhu <i...@lembacon.com>
> > Date:   Sat Mar 2 23:50:28 2019 +0800
> >
> >     libpulsar: revision for protobuf
> >
> > commit fcc845573eef8564c7fac9ddba8a9e660b2d06f7
> > Author: Matteo Merli <mm...@apache.org>
> > Date:   Wed Feb 20 22:32:16 2019 -0800
> >
> >     libpulsar 2.3.0
> >
> >     Closes #37151.
> >
> >     Signed-off-by: FX Coudert <fx...@gmail.com>
> >
> > commit 24c06c4c49374136d484ef8a000b1c1626f2bf79
> > Author: Viktor Szakats <co...@vszakats.net>
> > Date:   Fri Jan 18 01:50:31 2019 +0000
> >
> >     libpulsar: use common/short form of apache url
> >
> > commit c2f34710ae2d378fbea076661016b87b31ce35af
> > Author: Matteo Merli <mm...@apache.org>
> > Date:   Wed Oct 31 15:28:10 2018 -0700
> >
> >     libpulsar 2.2.1 (new formula)
> >
> >     Closes #35809.
> >
> >     Signed-off-by: FX Coudert <fx...@gmail.com>
> >
> > > On Jan 20, 2020, at 1:19 PM, Sijie Guo <gu...@gmail.com> wrote:
> > >
> > > Hi all,
> > >
> > > It is my first time to realize that there is a libpulsar entry in
> > homebrew.
> > > (https://github.com/apache/pulsar/issues/6091). Are Pulsar committers
> > > responsible for updating this? Or homebrew team is updating that entry?
> > >
> > > If the committers are responsible for updating it, it should have been
> > > discussed in the mailing list before and we need to add an entry in
> > release
> > > doc about this.
> > >
> > > Thanks,
> > > Sijie
> >
> >

Re: [DISCUSS] Homebrew libpulsar

Posted by Sijie Guo <gu...@gmail.com>.
Thanks Dave.

So the question will be - shall the release manager be responsible for
updating the homebrew entry? If so, we need to include it in the release
procedure.

Thanks,
Sijie

On Mon, Jan 20, 2020 at 2:03 PM Dave Fisher <wa...@apache.org> wrote:

> Hi -
>
> I’ve played with Homebrew before and from the git log I can see that
> Matteo created the initial version, but for the most part the Homebrew
> community has maintained the script.
>
> Here is the current script:
>
> Formula/libpulsar.rb
> class Libpulsar < Formula
>   desc "Apache Pulsar C++ library"
>   homepage "https://pulsar.apache.org/"
>   url "
> https://www.apache.org/dyn/closer.cgi?path=pulsar/pulsar-2.4.2/apache-pulsar-2.4.2-src.tar.gz
> "
>   sha256 "4b543932db923aa135c4d54b9122bcbdfc67bd73de641f9fffbc9a4ddf3430ae"
>   revision 1
>
>   bottle do
>     cellar :any
>     sha256
> "13c37b77dee18f4bba454484b4426a6f3dad27f902e0793a56a6358897ab4f3f" =>
> :catalina
>     sha256
> "ff0090b77842ac6034c4a425438f8d5b401164da4bada7ac11c4963dcdaa3a28" =>
> :mojave
>     sha256
> "d73f612edf73a351ab7f90776c24670c4c77b85fd95e37999e863ff27daf89ef" =>
> :high_sierra
>   end
>
>   depends_on "cmake" => :build
>   depends_on "pkg-config" => :build
>   depends_on "boost"
>   depends_on "openssl@1.1"
>   depends_on "protobuf"
>   depends_on "snappy"
>   depends_on "zstd"
>
>   def install
>     cd "pulsar-client-cpp" do
>       system "cmake", ".", *std_cmake_args,
>                       "-DBUILD_TESTS=OFF",
>                       "-DBUILD_PYTHON_WRAPPER=OFF",
>                       "-DBoost_INCLUDE_DIRS=#{Formula["boost"].include}",
>
> "-DProtobuf_INCLUDE_DIR=#{Formula["protobuf"].include}",
>
> "-DProtobuf_LIBRARIES=#{Formula["protobuf"].lib}/libprotobuf.dylib"
>       system "make", "pulsarShared", "pulsarStatic"
>       system "make", "install"
>     end
>   end
>
>   test do
>     (testpath/"test.cc").write <<~EOS
>       #include <pulsar/Client.h>
>
>       int main (int argc, char **argv) {
>         pulsar::Client client("pulsar://localhost:6650");
>         return 0;
>       }
>     EOS
>     system ENV.cxx, "test.cc", "-L#{lib}", "-lpulsar", "-o", "test"
>     system "./test"
>   end
> end
>
> Here is the log w/o the bottling.
>
> % git log master -- Formula/libpulsar.rb
> commit e4ed6752b0cab4c98ae72f81b2bd7160412bb938
> Author: FX Coudert <fx...@gmail.com>
> Date:   Thu Dec 26 16:17:50 2019 +0100
>
>     libpulsar: revision for boost
>
> commit 07bd5db82526c43a0e5004f690f3afbda4b89c7c
> Author: Matteo Merli <mm...@apache.org>
> Date:   Thu Dec 19 11:38:56 2019 -0800
>
>     libpulsar 2.4.2
>
>     Closes #48088.
>
>     Signed-off-by: FX Coudert <fx...@gmail.com>
>
> commit 3e6adba69f61da1b6fee5ba49199d299878f0db1
> Author: Austin Shalit <au...@gmail.com>
> Date:   Wed Nov 27 23:32:07 2019 -0800
>
>     libpulsar: revision bump
>
> commit fc6e1f2d6179379740466631eb0dffd56b404f5c
> Author: Rui Chen <rc...@meetup.com>
> Date:   Thu Oct 3 13:05:09 2019 -0400
>
>     libpulsar: revision for protobuf
>
> commit 11b3aa3dbe8a76716784cda9f351a28b49e9dfd0
> Author: Henry Fredrick Schreiner <he...@cern.ch>
> Date:   Mon Sep 2 20:47:07 2019 -0400
>
>     libpulsar 2.4.1: revision for boost 1.71.0
>
> commit 0d14181d6cbdd044eeec9b78e1d4b7d766ff5f1d
> Author: Henry Fredrick Schreiner <he...@cern.ch>
> Date:   Sun Sep 1 19:56:04 2019 -0400
>
>     libpulsar: Keep from linking to snappy
>
>     Closes #43761.
>
>     Signed-off-by: FX Coudert <fx...@gmail.com>
>
> commit d9288916f84c02730c9ba800eb12550cb1b5b865
> Author: FX Coudert <fx...@gmail.com>
> Date:   Thu Aug 29 10:28:00 2019 +0200
>
>     libpulsar: move to OpenSSL 1.1
>
> commit 84d41c55af723989ccfd61161fc014c69f62da5c
> Author: Matteo Merli <mm...@apache.org>
> Date:   Wed Aug 28 11:26:50 2019 -0700
>
>     libpulsar 2.4.0
>
>     Closes #43555.
>
>     Signed-off-by: FX Coudert <fx...@gmail.com>
>
> commit 7e24d72cbe366889612c2acc9d230d7263db4122
> Author: Chongyu Zhu <i...@lembacon.com>
> Date:   Wed May 29 15:24:44 2019 +0800
>
>     libpulsar: revision for protobuf
>
> commit 42a18870f92e42a787692c840d00cb5731ae6a5c
> Author: FX Coudert <fx...@gmail.com>
> Date:   Fri Apr 12 21:19:13 2019 +0200
>
>     libpulsar: revision and patch for boost
>
> commit 167f3474dd8ca620ab3e9d12513e88e499fe1ca2
> Author: Steven Peters <sc...@openrobotics.org>
> Date:   Sat Jun 1 15:36:17 2019 -0700
>
>     libpulsar 2.3.2
>
>     Closes #40591.
>
>     Signed-off-by: Steven Peters <sc...@openrobotics.org>
>
> commit 1b3b8c15319503ff8a33e9627c7a46d5f8d2546f
> Author: FX Coudert <fx...@gmail.com>
> Date:   Mon Apr 15 17:40:36 2019 +0200
>
>     libpulsar 2.3.1
>
>     Closes #38947.
>
>     Signed-off-by: FX Coudert <fx...@gmail.com>
>
> commit 3c0b62cc0288f6d5dfb0ed9f682d6e245d1e3cfe
> Author: Steven Peters <sc...@openrobotics.org>
> Date:   Mon Mar 4 12:41:16 2019 -0800
>
>     libpulsar: revision for boost
>
> commit cac1d53d2254004b2082a2d501f772fcaf77b8c0
> Author: Chongyu Zhu <i...@lembacon.com>
> Date:   Sat Mar 2 23:50:28 2019 +0800
>
>     libpulsar: revision for protobuf
>
> commit fcc845573eef8564c7fac9ddba8a9e660b2d06f7
> Author: Matteo Merli <mm...@apache.org>
> Date:   Wed Feb 20 22:32:16 2019 -0800
>
>     libpulsar 2.3.0
>
>     Closes #37151.
>
>     Signed-off-by: FX Coudert <fx...@gmail.com>
>
> commit 24c06c4c49374136d484ef8a000b1c1626f2bf79
> Author: Viktor Szakats <co...@vszakats.net>
> Date:   Fri Jan 18 01:50:31 2019 +0000
>
>     libpulsar: use common/short form of apache url
>
> commit c2f34710ae2d378fbea076661016b87b31ce35af
> Author: Matteo Merli <mm...@apache.org>
> Date:   Wed Oct 31 15:28:10 2018 -0700
>
>     libpulsar 2.2.1 (new formula)
>
>     Closes #35809.
>
>     Signed-off-by: FX Coudert <fx...@gmail.com>
>
> > On Jan 20, 2020, at 1:19 PM, Sijie Guo <gu...@gmail.com> wrote:
> >
> > Hi all,
> >
> > It is my first time to realize that there is a libpulsar entry in
> homebrew.
> > (https://github.com/apache/pulsar/issues/6091). Are Pulsar committers
> > responsible for updating this? Or homebrew team is updating that entry?
> >
> > If the committers are responsible for updating it, it should have been
> > discussed in the mailing list before and we need to add an entry in
> release
> > doc about this.
> >
> > Thanks,
> > Sijie
>
>

Re: [DISCUSS] Homebrew libpulsar

Posted by Dave Fisher <wa...@apache.org>.
Hi -

I’ve played with Homebrew before and from the git log I can see that Matteo created the initial version, but for the most part the Homebrew community has maintained the script.

Here is the current script:

Formula/libpulsar.rb
class Libpulsar < Formula
  desc "Apache Pulsar C++ library"
  homepage "https://pulsar.apache.org/"
  url "https://www.apache.org/dyn/closer.cgi?path=pulsar/pulsar-2.4.2/apache-pulsar-2.4.2-src.tar.gz"
  sha256 "4b543932db923aa135c4d54b9122bcbdfc67bd73de641f9fffbc9a4ddf3430ae"
  revision 1

  bottle do
    cellar :any
    sha256 "13c37b77dee18f4bba454484b4426a6f3dad27f902e0793a56a6358897ab4f3f" => :catalina
    sha256 "ff0090b77842ac6034c4a425438f8d5b401164da4bada7ac11c4963dcdaa3a28" => :mojave
    sha256 "d73f612edf73a351ab7f90776c24670c4c77b85fd95e37999e863ff27daf89ef" => :high_sierra
  end

  depends_on "cmake" => :build
  depends_on "pkg-config" => :build
  depends_on "boost"
  depends_on "openssl@1.1"
  depends_on "protobuf"
  depends_on "snappy"
  depends_on "zstd"

  def install
    cd "pulsar-client-cpp" do
      system "cmake", ".", *std_cmake_args,
                      "-DBUILD_TESTS=OFF",
                      "-DBUILD_PYTHON_WRAPPER=OFF",
                      "-DBoost_INCLUDE_DIRS=#{Formula["boost"].include}",
                      "-DProtobuf_INCLUDE_DIR=#{Formula["protobuf"].include}",
                      "-DProtobuf_LIBRARIES=#{Formula["protobuf"].lib}/libprotobuf.dylib"
      system "make", "pulsarShared", "pulsarStatic"
      system "make", "install"
    end
  end

  test do
    (testpath/"test.cc").write <<~EOS
      #include <pulsar/Client.h>

      int main (int argc, char **argv) {
        pulsar::Client client("pulsar://localhost:6650");
        return 0;
      }
    EOS
    system ENV.cxx, "test.cc", "-L#{lib}", "-lpulsar", "-o", "test"
    system "./test"
  end
end

Here is the log w/o the bottling.

% git log master -- Formula/libpulsar.rb
commit e4ed6752b0cab4c98ae72f81b2bd7160412bb938
Author: FX Coudert <fx...@gmail.com>
Date:   Thu Dec 26 16:17:50 2019 +0100

    libpulsar: revision for boost

commit 07bd5db82526c43a0e5004f690f3afbda4b89c7c
Author: Matteo Merli <mm...@apache.org>
Date:   Thu Dec 19 11:38:56 2019 -0800

    libpulsar 2.4.2
    
    Closes #48088.
    
    Signed-off-by: FX Coudert <fx...@gmail.com>

commit 3e6adba69f61da1b6fee5ba49199d299878f0db1
Author: Austin Shalit <au...@gmail.com>
Date:   Wed Nov 27 23:32:07 2019 -0800

    libpulsar: revision bump

commit fc6e1f2d6179379740466631eb0dffd56b404f5c
Author: Rui Chen <rc...@meetup.com>
Date:   Thu Oct 3 13:05:09 2019 -0400

    libpulsar: revision for protobuf

commit 11b3aa3dbe8a76716784cda9f351a28b49e9dfd0
Author: Henry Fredrick Schreiner <he...@cern.ch>
Date:   Mon Sep 2 20:47:07 2019 -0400

    libpulsar 2.4.1: revision for boost 1.71.0

commit 0d14181d6cbdd044eeec9b78e1d4b7d766ff5f1d
Author: Henry Fredrick Schreiner <he...@cern.ch>
Date:   Sun Sep 1 19:56:04 2019 -0400

    libpulsar: Keep from linking to snappy
    
    Closes #43761.
    
    Signed-off-by: FX Coudert <fx...@gmail.com>

commit d9288916f84c02730c9ba800eb12550cb1b5b865
Author: FX Coudert <fx...@gmail.com>
Date:   Thu Aug 29 10:28:00 2019 +0200

    libpulsar: move to OpenSSL 1.1

commit 84d41c55af723989ccfd61161fc014c69f62da5c
Author: Matteo Merli <mm...@apache.org>
Date:   Wed Aug 28 11:26:50 2019 -0700

    libpulsar 2.4.0
    
    Closes #43555.
    
    Signed-off-by: FX Coudert <fx...@gmail.com>

commit 7e24d72cbe366889612c2acc9d230d7263db4122
Author: Chongyu Zhu <i...@lembacon.com>
Date:   Wed May 29 15:24:44 2019 +0800

    libpulsar: revision for protobuf

commit 42a18870f92e42a787692c840d00cb5731ae6a5c
Author: FX Coudert <fx...@gmail.com>
Date:   Fri Apr 12 21:19:13 2019 +0200

    libpulsar: revision and patch for boost

commit 167f3474dd8ca620ab3e9d12513e88e499fe1ca2
Author: Steven Peters <sc...@openrobotics.org>
Date:   Sat Jun 1 15:36:17 2019 -0700

    libpulsar 2.3.2
    
    Closes #40591.
    
    Signed-off-by: Steven Peters <sc...@openrobotics.org>

commit 1b3b8c15319503ff8a33e9627c7a46d5f8d2546f
Author: FX Coudert <fx...@gmail.com>
Date:   Mon Apr 15 17:40:36 2019 +0200

    libpulsar 2.3.1
    
    Closes #38947.
    
    Signed-off-by: FX Coudert <fx...@gmail.com>

commit 3c0b62cc0288f6d5dfb0ed9f682d6e245d1e3cfe
Author: Steven Peters <sc...@openrobotics.org>
Date:   Mon Mar 4 12:41:16 2019 -0800

    libpulsar: revision for boost

commit cac1d53d2254004b2082a2d501f772fcaf77b8c0
Author: Chongyu Zhu <i...@lembacon.com>
Date:   Sat Mar 2 23:50:28 2019 +0800

    libpulsar: revision for protobuf

commit fcc845573eef8564c7fac9ddba8a9e660b2d06f7
Author: Matteo Merli <mm...@apache.org>
Date:   Wed Feb 20 22:32:16 2019 -0800

    libpulsar 2.3.0
    
    Closes #37151.
    
    Signed-off-by: FX Coudert <fx...@gmail.com>

commit 24c06c4c49374136d484ef8a000b1c1626f2bf79
Author: Viktor Szakats <co...@vszakats.net>
Date:   Fri Jan 18 01:50:31 2019 +0000

    libpulsar: use common/short form of apache url

commit c2f34710ae2d378fbea076661016b87b31ce35af
Author: Matteo Merli <mm...@apache.org>
Date:   Wed Oct 31 15:28:10 2018 -0700

    libpulsar 2.2.1 (new formula)
    
    Closes #35809.
    
    Signed-off-by: FX Coudert <fx...@gmail.com>

> On Jan 20, 2020, at 1:19 PM, Sijie Guo <gu...@gmail.com> wrote:
> 
> Hi all,
> 
> It is my first time to realize that there is a libpulsar entry in homebrew.
> (https://github.com/apache/pulsar/issues/6091). Are Pulsar committers
> responsible for updating this? Or homebrew team is updating that entry?
> 
> If the committers are responsible for updating it, it should have been
> discussed in the mailing list before and we need to add an entry in release
> doc about this.
> 
> Thanks,
> Sijie