Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 808099

Summary: net-libs/pjproject-2.10-r2 stores incorrect CFLAGS in /usr/lib64/pkgconfig/libpjproject.pc
Product: Gentoo Linux Reporter: capresucrike-1984
Component: Current packagesAssignee: Jaco Kroon <jaco>
Status: RESOLVED FIXED    
Severity: normal CC: proxy-maint, sam
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description capresucrike-1984 2021-08-14 04:53:00 UTC
Add some funky CFLAGS to complille pjproject like --param l1-cache-size=32 --param l1-cache-line-size=64 --param l2-cache-size=8192 and you will see that these get added to the pkgconfig file.

Reproducible: Always

Steps to Reproduce:
1. Comple pjproject with CFLAGS --param l1-cache-size=32 --param l1-cache-line-size=64 --param l2-cache-size=8192
2. Try to compile asterisk 
3. Asterisk fails because of incorrect CFLAGS from pjproject with misleading error  of missing SQLite and/or SQLite3
Comment 1 Jaco Kroon 2021-09-27 08:49:36 UTC
Ha!  This may also explain issues w.r.t. linking against 2.11.
Comment 2 Jaco Kroon 2022-06-09 14:52:22 UTC
For the record:

Cflags: -I${includedir} -O2 -march=native -pipe -frecord-gcc-switches -fno-common -I/usr/include -DPJ_AUTOCONF=1 -O2 -march=native -pipe -frecord-gcc-switches -fno-common -DNDEBUG=1 -DPJ_HAS_IPV6=1 -DPJMEDIA_HAS_SRTP=1 -DPJ_IS_BIG_ENDIAN=0 -DPJ_IS_LITTLE_ENDIAN=1 -fPIC


vs

Cflags: -I/usr/include -DPJ_AUTOCONF=1  -DPJ_IS_BIG_ENDIAN=0 -DPJ_IS_LITTLE_ENDIAN=1

Looks a bit better.  Libs: also looks cleaner.

PR to follow shortly (if all goes well).
Comment 3 Larry the Git Cow gentoo-dev 2022-06-15 13:32:14 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=74bc4476b50218718af7c43038176f1d69c50e61

commit 74bc4476b50218718af7c43038176f1d69c50e61
Author:     Jaco Kroon <jaco@uls.co.za>
AuthorDate: 2021-07-24 08:36:57 +0000
Commit:     Joonas Niilola <juippis@gentoo.org>
CommitDate: 2022-06-15 13:32:11 +0000

    net-libs/pjproject: Version 2.12.1.
    
    Upstream release.
    
    Remove the need for custom patches (which is still required but no
    longer applies, instead, rely on ./configure detecting openssl, we do
    depend on it, and only --disable-ssl works, passing --enable-ssl also
    effectively disables ssl).
    
    Compile tested asterisk 13, 16 and 18 (in-tree versions) against this.
    Would appreciate a double-check on this one.  One version from each is
    sufficient.
    
    Since the two libraries that were the target of parallel build failures
    are now one, I believe the parallel build issue is fixed too.
    
    Included patch for CVE-2022-31031
    
    Closes: https://bugs.gentoo.org/833765
    Closes: https://bugs.gentoo.org/817803
    Closes: https://bugs.gentoo.org/808099
    Closes: https://bugs.gentoo.org/834491
    Bug: https://bugs.gentoo.org/803614
    Bug: https://bugs.gentoo.org/765799
    Bug: https://bugs.gentoo.org/829894
    Package-Manager: Portage-3.0.20, Repoman-3.0.2
    Signed-off-by: Jaco Kroon <jaco@uls.co.za>
    Closes: https://github.com/gentoo/gentoo/pull/21761
    Signed-off-by: Joonas Niilola <juippis@gentoo.org>

 net-libs/pjproject/Manifest                        |   1 +
 .../files/pjproject-2.12.1-CVE-2022-31031.patch    |  41 +++++++
 net-libs/pjproject/pjproject-2.12.1.ebuild         | 125 +++++++++++++++++++++
 3 files changed, 167 insertions(+)