Skip to content
Snippets Groups Projects
  • David M. Lee's avatar
    d4e25a45
    install_prereq: Build jansson from source, when necessary · d4e25a45
    David M. Lee authored
    When r383579 was committed, it made Jansson a required dependency.
    
    While libjansson-dev and jansson-devel are available on recent
    distros, some older (but still supported) distros don't have
    it. There's a pull request[1] to get it into repoforge, but that still
    doesn't help everyone. (And helps no one until the pull request is
    merged and packages are built).
    
    This patch adds Jansson install from source to the install_unpackaged()
    function. There are a few gotcha's, which makes this change not
    completely trivial.
    
     * Since Jansson may be installed by a package, don't install from
       source if a package installation can be found
       * libresample may also be installed via package, so I added a
         similar check to that.
     * Since Jansson installs into /usr/local, this patch also adds
       /usr/local/lib to /etc/ld.so.conf.d so that the library can be
       found.
       * The alternative was to install into /usr, but then it gets
         complicated having to deal with EL's /usr/lib{32,64} shenanigans.
    
     [1]: https://github.com/repoforge/rpms/pull/250
    
    Review: https://reviewboard.asterisk.org/r/2414/
    
    
    git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@384488 65c4cc65-6c06-0410-ace0-fbb531ad65f3
    d4e25a45
    History
    install_prereq: Build jansson from source, when necessary
    David M. Lee authored
    When r383579 was committed, it made Jansson a required dependency.
    
    While libjansson-dev and jansson-devel are available on recent
    distros, some older (but still supported) distros don't have
    it. There's a pull request[1] to get it into repoforge, but that still
    doesn't help everyone. (And helps no one until the pull request is
    merged and packages are built).
    
    This patch adds Jansson install from source to the install_unpackaged()
    function. There are a few gotcha's, which makes this change not
    completely trivial.
    
     * Since Jansson may be installed by a package, don't install from
       source if a package installation can be found
       * libresample may also be installed via package, so I added a
         similar check to that.
     * Since Jansson installs into /usr/local, this patch also adds
       /usr/local/lib to /etc/ld.so.conf.d so that the library can be
       found.
       * The alternative was to install into /usr, but then it gets
         complicated having to deal with EL's /usr/lib{32,64} shenanigans.
    
     [1]: https://github.com/repoforge/rpms/pull/250
    
    Review: https://reviewboard.asterisk.org/r/2414/
    
    
    git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@384488 65c4cc65-6c06-0410-ace0-fbb531ad65f3