confluentinc / libserdes

Avro Serialization/Deserialization C/C++ library with Confluent schema-registry support
Apache License 2.0
5 stars 64 forks source link

Fixed compatibility with Avro master branch #17

Closed mparry closed 2 years ago

mparry commented 5 years ago

... while retaining backwards compatibility with released versions. However, the trade-off is that C++11 is now required.

(An alternative would be to switch to explicitly saying shared_ptr instead of auto_ptr, but C++11 seems like a reasonable requirement to impose, given that the underlying Avro library has already done so.)

ghost commented 5 years ago

It looks like @mparry hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence. Wikipedia

You can read and sign our full Contributor License Agreement here.

Once you've signed reply with [clabot:check] to prove it.

Appreciation of efforts,

clabot

swarog46 commented 3 years ago

@edenhill, may I ask you to take a look, please?

edenhill commented 2 years ago

@mparry Could you sign the CLA so we can get this merged?

Thanks

mparry commented 2 years ago

We signed a group CLA some time ago - see my other PR #16 - but it didn't seem to work. Not sure what to try next!

mparry commented 2 years ago

[clabot:check]

edenhill commented 2 years ago

Thank you!