[
https://issues.apache.org/jira/browse/AVRO-1769?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15072916#comment-15072916
]
Douglas Creager commented on AVRO-1769:
---------------------------------------
This looks good, though the FindJansson module is pretty old. Jansson ships a
pkg-config file, so you could use {{pkg_check_modules(JANSSON jansson>=2.3)}}
instead, like we do for lzma/xz. That has the added benefit of having CMake
verify that the Jansson that we find meets our >=2.3 version requirement.
> Depend on external libjansson instead of a bundled copy
> -------------------------------------------------------
>
> Key: AVRO-1769
> URL: https://issues.apache.org/jira/browse/AVRO-1769
> Project: Avro
> Issue Type: Improvement
> Components: c
> Affects Versions: 1.7.7
> Reporter: Martin Kleppmann
> Assignee: Martin Kleppmann
> Attachments: AVRO-1769.patch.gz
>
>
> To fix AVRO-1691 we need to update to a newer version of Jansson. However,
> rather than updating the version of Jansson that is bundled with Avro-C, this
> would be a good time to switch to a dynamically linked version. As
> [~edenhill] pointed out, libjansson is widely available in distributions
> nowadays.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)