Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 652196 - net-libs/pjproject ebuild needs to set #define PJMEDIA_HAS_VIDEO
Summary: net-libs/pjproject ebuild needs to set #define PJMEDIA_HAS_VIDEO
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Jaco Kroon
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-04-02 10:35 UTC by Szef
Modified: 2019-11-30 07:49 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Szef 2018-04-02 10:35:25 UTC
As mentioned here: https://trac.pjsip.org/repos/wiki/Getting-Started/Autoconf
to properly enable video support in pjproject, one should add PJMEDIA_HAS_VIDEO define in config_site.h.
That is never done in 2.7 branch. And many (all) video capabilities remain disabled.
For me following code in src_configure() in .ebuild resolves the issue:

if [[ ! -z ${VIDEO_FLAGS} ]]; then
        echo '#define PJMEDIA_HAS_VIDEO 1' >> pjlib/include/pj/config_site.h
fi

Something similar should be done in the portage.
Comment 1 Larry the Git Cow gentoo-dev 2019-11-30 07:49:52 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=90d194bc8a80a478c91f5196da9eac770a755ca2

commit 90d194bc8a80a478c91f5196da9eac770a755ca2
Author:     Jaco Kroon <jaco@uls.co.za>
AuthorDate: 2019-11-18 09:45:56 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2019-11-30 07:48:56 +0000

    net-libs/pjproject: Bump to 2.9.
    
    Drop libressl patches (upstream).
    Rebase configure for --enable-ssl.
    Pull in asterisk config_site.h file.
    Remove blocked on mediastreamer-bcg729 (bcg729 and mediastreamer-bcg729
    are the actual blockers and marked as such already).
    Use ${ED} instead of ${D} on install
    Correctly rm static libs if USE=-static-libs
    Set the PJMEDIA_HAS_VIDEO #define as per https://bugs.gentoo.org/652196
    Update package.use.mask for ppc to mask g729 for pjproject since
    net-libs/bcg729 isn't available (yet) on ppc.
    
    Closes: https://bugs.gentoo.org/609702
    Closes: https://bugs.gentoo.org/618352
    Closes: https://bugs.gentoo.org/652196
    Closes: https://bugs.gentoo.org/650312
    Closes: https://bugs.gentoo.org/653482
    Closes: https://bugs.gentoo.org/657014
    Closes: https://bugs.gentoo.org/686796
    Package-Manager: Portage-2.3.76, Repoman-2.3.16
    Signed-off-by: Jaco Kroon <jaco@uls.co.za>
    Closes: https://github.com/gentoo/gentoo/pull/13712
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 net-libs/pjproject/Manifest                        |   1 +
 .../pjproject/files/pjproject-2.9-config_site.h    |  74 +++++++++++++
 .../pjproject/files/pjproject-2.9-ssl-enable.patch | 100 +++++++++++++++++
 net-libs/pjproject/metadata.xml                    |   3 +-
 net-libs/pjproject/pjproject-2.9.ebuild            | 120 +++++++++++++++++++++
 .../linux/powerpc/ppc32/17.0/package.use.mask      |   3 +
 .../ppc64/17.0/32bit-userland/package.use.mask     |   3 +
 7 files changed, 303 insertions(+), 1 deletion(-)