[Buildroot] [PATCH 1/1] package/avro: new package

Titouan Christophe titouan.christophe at railnova.eu
Wed Dec 11 10:29:59 UTC 2019


Hello Gilles and all,

On 12/10/19 8:46 PM, Gilles Talis wrote:
> Hello Christophe,
> 
> Thanks for your contribution
> 
> I would be very interested to have other people's opinion on this one,
> but I think it'd be neater to follow the protobuf "architecture":
> - one package for protobuf
> - one package for python-protobuf
> 
> Similarly, you could create:
> - one package for avro-c
> - one package for python-avro
> (another package for avro-cpp eventually

Because I used the single archive (see your next remark below), I draw 
inspiration from the alsa-lib package, which provides both the C lib and 
a python package.

> 
> Moreover, these packages seem to have been nicely separated upstream:
> - avro-c: https://www-eu.apache.org/dist/avro/avro-1.9.1/c/avro-c-1.9.1.tar.gz
> - avro-python3:
> https://www-eu.apache.org/dist/avro/avro-1.9.1/py3/avro-python3-1.9.1.tar.gz
> 

Shame on me, I did not even notice that these nicely separated archives 
were distributed upstream. I'll change that and send a new version.

Using these separated archives would therefore make much more sense in 
their respective "sections" (python-avro, etc...)

> I have not run an exact diff between the avro-src-xxx.tar.gz and the
> other source packages, but if they match, I would recommend using the
> "protobuf" method.
> 
> Thanks
> Gilles.
> 

Thanks for your review !

Titouan



More information about the buildroot mailing list