Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 678594 - sys-boot/vboot-utils-72_p20181229 : host/.../openssl_compat.h:16:20: error: static declaration of RSA_get0_key follows non-static declaration
Summary: sys-boot/vboot-utils-72_p20181229 : host/.../openssl_compat.h:16:20: error: s...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Zac Medico
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks: libressl-support
  Show dependency tree
 
Reported: 2019-02-22 16:33 UTC by Toralf Förster
Modified: 2020-01-13 01:07 UTC (History)
3 users (show)

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


Attachments
emerge-info.txt (emerge-info.txt,16.31 KB, text/plain)
2019-02-22 16:33 UTC, Toralf Förster
Details
emerge-history.txt (emerge-history.txt,166.40 KB, text/plain)
2019-02-22 16:33 UTC, Toralf Förster
Details
environment (environment,77.36 KB, text/plain)
2019-02-22 16:33 UTC, Toralf Förster
Details
etc.portage.tbz2 (etc.portage.tbz2,18.00 KB, application/x-bzip)
2019-02-22 16:33 UTC, Toralf Förster
Details
logs.tbz2 (logs.tbz2,527 bytes, application/x-bzip)
2019-02-22 16:33 UTC, Toralf Förster
Details
sys-boot:vboot-utils-72_p20181229:20190222-043959.log (sys-boot:vboot-utils-72_p20181229:20190222-043959.log,60.27 KB, text/plain)
2019-02-22 16:33 UTC, Toralf Förster
Details
openssl's changes to v1.1 give us easy if statements to find. (vboot-libre.patch,426 bytes, patch)
2019-02-23 18:55 UTC, pagorman
Details | Diff
emerge-info.txt (emerge-info.txt,16.33 KB, text/plain)
2019-06-13 18:13 UTC, Toralf Förster
Details
emerge-history.txt (emerge-history.txt,1.63 KB, text/plain)
2019-06-13 18:13 UTC, Toralf Förster
Details
environment (environment,77.84 KB, text/plain)
2019-06-13 18:13 UTC, Toralf Förster
Details
etc.portage.tbz2 (etc.portage.tbz2,10.77 KB, application/x-bzip)
2019-06-13 18:13 UTC, Toralf Förster
Details
logs.tbz2 (logs.tbz2,530 bytes, application/x-bzip)
2019-06-13 18:13 UTC, Toralf Förster
Details
sys-boot:vboot-utils-72_p20181229-r1:20190613-180250.log (sys-boot:vboot-utils-72_p20181229-r1:20190613-180250.log,63.40 KB, text/plain)
2019-06-13 18:13 UTC, Toralf Förster
Details
emerge --info (vboot-utils-72_p20181229-r1_emerge.info,4.36 KB, text/plain)
2019-06-14 10:18 UTC, Alpha Bravo
Details
new version ebuild (vboot-utils-76.ebuild,2.74 KB, text/plain)
2019-11-15 18:21 UTC, Steve Arnold
Details
musl patch for vboot-utils-76 (vboot-utils-76-musl-fts.patch,645 bytes, text/plain)
2019-11-15 18:22 UTC, Steve Arnold
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Toralf Förster gentoo-dev 2019-02-22 16:33:13 UTC
x86_64-pc-linux-gnu-gcc -O2 -pipe -march=native -DCHROMEOS_ENVIRONMENT -Wall   -DNDEBUG -MMD -MF /var/tmp/portage/sys-boot/vboot-utils-72_p20181229/work/build/host/lib/util_misc.o.d -fPIE -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -DHAVE_EXECINFO_H -DHAVE_LIBZIP  -Ifirmware/include -Ifirmware/lib/include -Ifirmware/lib/cgptlib/include -Ifirmware/lib/cryptolib/include -Ifirmware/lib/tpm_lite/include -Ifirmware/2lib/include -Ifirmware/lib20/include -Ihost/include -Ihost/lib/include -Ihost/lib21/include -Ifirmware/lib21/include -c -o /var/tmp/portage/sys-boot/vboot-utils-72_p20181229/work/build/host/lib/util_misc.o host/lib/util_misc.c
In file included from host/lib/util_misc.c:21:
host/include/openssl_compat.h:16:20: error: static declaration of ‘RSA_get0_key’ follows non-static declaration
 static inline void RSA_get0_key(const RSA *rsa, const BIGNUM **n,
                    ^~~~~~~~~~~~
In file included from host/lib/util_misc.c:9:

  -------------------------------------------------------------------

  This is an unstable amd64 chroot image at a tinderbox (==build bot)
  name: 17.0-desktop-gnome_libressl_20190218-073051

  -------------------------------------------------------------------

  Please see the tracker bug for details.

gcc-config -l:
 [1] x86_64-pc-linux-gnu-8.2.0 *

Available Python interpreters, in order of preference:
  [1]   python3.7
  [2]   python3.6
  [3]   python2.7 (fallback)
Available Ruby profiles:
  [1]   ruby24 (with Rubygems)
  [2]   ruby25 (with Rubygems)
  [3]   ruby26 (with Rubygems) *
Available Rust versions:
  [1]   rust-1.32.0 *
java-config:
The following VMs are available for generation-2:
*)	IcedTea JDK 3.10.0 [icedtea-bin-8]
Available Java Virtual Machines:
  [1]   icedtea-bin-8  system-vm

emerge -qpvO sys-boot/vboot-utils
[ebuild  N    ] sys-boot/vboot-utils-72_p20181229  USE="libressl -minimal -static"
Comment 1 Toralf Förster gentoo-dev 2019-02-22 16:33:17 UTC
Created attachment 566082 [details]
emerge-info.txt
Comment 2 Toralf Förster gentoo-dev 2019-02-22 16:33:23 UTC
Created attachment 566084 [details]
emerge-history.txt
Comment 3 Toralf Förster gentoo-dev 2019-02-22 16:33:30 UTC
Created attachment 566086 [details]
environment
Comment 4 Toralf Förster gentoo-dev 2019-02-22 16:33:34 UTC
Created attachment 566088 [details]
etc.portage.tbz2
Comment 5 Toralf Förster gentoo-dev 2019-02-22 16:33:39 UTC
Created attachment 566090 [details]
logs.tbz2
Comment 6 Toralf Förster gentoo-dev 2019-02-22 16:33:45 UTC
Created attachment 566092 [details]
sys-boot:vboot-utils-72_p20181229:20190222-043959.log
Comment 7 pagorman 2019-02-23 18:55:20 UTC
Created attachment 566218 [details, diff]
openssl's changes to v1.1 give us easy if statements to find.

builds on amd64.
Comment 8 pagorman 2019-02-23 18:55:44 UTC
builds, that is, with a patch.
Comment 9 pagorman 2019-02-23 19:12:09 UTC
I'm not sure if its relevant... but this patch seems best applied by the ebuild in the "src_prepare" section, rather than in a PATCHES list.
Comment 10 Alpha Bravo 2019-06-10 18:56:07 UTC
Same issue on armv7a-unknown-linux-musleabihf. pagorman's patch only works after reverting https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=db7f353c9db529295c7c1fa1aa66751bbc9c1594
Comment 11 Larry the Git Cow gentoo-dev 2019-06-13 01:52:48 UTC
The bug has been closed via the following commit(s):

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

commit c24cbd37c6d967028dcb5ed93568b6da56eecda0
Author:     Jory Pratt <anarchy@gentoo.org>
AuthorDate: 2019-06-13 01:52:17 +0000
Commit:     Jory Pratt <anarchy@gentoo.org>
CommitDate: 2019-06-13 01:52:40 +0000

    sys-boot/vboot-utils: fix libressl support, musl dep fix
    
    Closes: https://bugs.gentoo.org/678594
    Closes: https://bugs.gentoo.org/687820
    Package-Manager: Portage-2.3.67, Repoman-2.3.14
    Signed-off-by: Jory Pratt <anarchy@gentoo.org>

 .../vboot-utils/files/vboot-utils-72_p20181229-libressl.patch | 11 +++++++++++
 ...72_p20181229.ebuild => vboot-utils-72_p20181229-r1.ebuild} |  2 ++
 2 files changed, 13 insertions(+)
Comment 12 Toralf Förster gentoo-dev 2019-06-13 18:13:29 UTC
it is still an issue at unstable amd64 tinderbox image 17.1-desktop-gnome-systemd_20190608-211904
Comment 13 Toralf Förster gentoo-dev 2019-06-13 18:13:33 UTC
Created attachment 579734 [details]
emerge-info.txt
Comment 14 Toralf Förster gentoo-dev 2019-06-13 18:13:36 UTC
Created attachment 579736 [details]
emerge-history.txt
Comment 15 Toralf Förster gentoo-dev 2019-06-13 18:13:39 UTC
Created attachment 579738 [details]
environment
Comment 16 Toralf Förster gentoo-dev 2019-06-13 18:13:41 UTC
Created attachment 579740 [details]
etc.portage.tbz2
Comment 17 Toralf Förster gentoo-dev 2019-06-13 18:13:44 UTC
Created attachment 579742 [details]
logs.tbz2
Comment 18 Toralf Förster gentoo-dev 2019-06-13 18:13:47 UTC
Created attachment 579744 [details]
sys-boot:vboot-utils-72_p20181229-r1:20190613-180250.log
Comment 19 Alpha Bravo 2019-06-14 10:13:22 UTC
For me (musl, unstable arm) the libressl support is fixed by that commit.

(In reply to Toralf Förster from comment #14)
> Created attachment 579736 [details]
> emerge-history.txt
Irrespective of this bug something seems to be wrong with that attachment, I don't think it tells what it should tell.
Comment 20 Alpha Bravo 2019-06-14 10:18:03 UTC
Created attachment 579770 [details]
emerge --info
Comment 21 Toralf Förster gentoo-dev 2019-06-14 16:24:40 UTC
(In reply to Alpha Bravo from comment #19)
> For me (musl, unstable arm) the libressl support is fixed by that commit.
> 
> (In reply to Toralf Förster from comment #14)
> > Created attachment 579736 [details]
> > emerge-history.txt
> Irrespective of this bug something seems to be wrong with that attachment, I
> don't think it tells what it should tell.

indeed
fixed by https://github.com/toralf/tinderbox/commit/b944957713fa999aee8b94d7fe87ecc1a3ae0d63
Comment 22 Toralf Förster gentoo-dev 2019-06-14 17:28:09 UTC
FWIW it fails here at different amd64 tinderbox images, last image was: 17.1-systemd_20190614-051827
Comment 23 Alpha Bravo 2019-06-20 12:45:39 UTC
On AArch64 it fails, too.
Comment 24 Alpha Bravo 2019-06-20 17:11:01 UTC
Problems might be solved with the ebuild / version bump from Bug #688396, feel free to test.
Comment 25 Steve Arnold archtester gentoo-dev 2019-09-12 18:41:03 UTC
Indeed, the vboot-utils-76.ebuild plus musl patch works for me on arm/amd64 with glibc or musl.
Comment 26 Laurent Pinchart 2019-10-12 20:32:21 UTC
I've hit the same issue when compiling without the libressl USE flag. The problem with the patch in https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c24cbd37c6d967028dcb5ed93568b6da56eecda0 is that when LIBRESSL_VERSION_NUMBER is unset, the condition

#if OPENSSL_VERSION_NUMBER < 0x10100000L || LIBRESSL_VERSION_NUMBER < 0x2080000L

always evaluates to true. Changing it to

#if OPENSSL_VERSION_NUMBER < 0x10100000L || (defined(LIBRESSL_VERSION_NUMBER) && LIBRESSL_VERSION_NUMBER < 0x2080000L)

fixes the issue on my system (but I haven't tested it with the libressl USE flag).
Comment 27 Steve Arnold archtester gentoo-dev 2019-11-15 18:20:04 UTC
This is still broken without using an overlay; both the adjusted patch in comment #26 and vboot-utils-76 (with a musl patch) are viable fixes.  If things are that stacked up I can push one or both of these...
Comment 28 Steve Arnold archtester gentoo-dev 2019-11-15 18:21:55 UTC
Created attachment 596264 [details]
new version ebuild
Comment 29 Steve Arnold archtester gentoo-dev 2019-11-15 18:22:40 UTC
Created attachment 596266 [details]
musl patch for vboot-utils-76
Comment 30 Alpha Bravo 2019-11-15 20:08:31 UTC
Maybe you could even push vboot-utils-79 from bug 688396, which has:
* seen some clean-up
* been fixed for building with USE="static"
* comes with a patch (vboot-utils-79-libzip.patch) that should fix the implementation of the new "libzip" USE-flag, which isn't fully respected otherwise

One issue with the vboot-utils-76.ebuild might be, that if USE="-minimal libzip", dev-libs/libzip will be listed two times in RDEPEND.
Comment 31 Alpha Bravo 2019-11-15 20:21:12 UTC
And regardless of the version I still can't obtain the source tarball directly from the SRC_URI, I have to manually download it - not sure as to why.
Comment 32 Larry the Git Cow gentoo-dev 2020-01-13 01:07:59 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=5e25b4aa1598f53a0712d6229df67f5564687285

commit 5e25b4aa1598f53a0712d6229df67f5564687285
Author:     Alpha Bravo <aos@ungwana.de>
AuthorDate: 2020-01-13 00:40:39 +0000
Commit:     Zac Medico <zmedico@gentoo.org>
CommitDate: 2020-01-13 01:07:44 +0000

    sys-boot/vboot-utils: Bump to version 80_p20200108
    
    Closes: https://bugs.gentoo.org/688396
    Closes: https://bugs.gentoo.org/687820
    Closes: https://bugs.gentoo.org/678594
    Package-Manager: Portage-2.3.84, Repoman-2.3.20
    Signed-off-by: Zac Medico <zmedico@gentoo.org>

 sys-boot/vboot-utils/Manifest                      |   1 +
 .../files/vboot-utils-80-musl-fts.patch            |  17 ++++
 sys-boot/vboot-utils/metadata.xml                  |   5 +-
 .../vboot-utils/vboot-utils-80_p20200108.ebuild    | 104 +++++++++++++++++++++
 4 files changed, 126 insertions(+), 1 deletion(-)