New issue
Advanced search Search tips

Issue 749849 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jul 2017
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Build dev-libs/crypto++ with clang

Project Member Reported by manojgupta@chromium.org, Jul 27 2017

Issue description

Currently dev-libs/crypto++ is built with gcc. Allow it to be built with clang.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 28 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/cf0d5207e185caf49fd3d311149cac5a9eca4e9d

commit cf0d5207e185caf49fd3d311149cac5a9eca4e9d
Author: Manoj Gupta <manojgupta@google.com>
Date: Fri Jul 28 22:04:21 2017

crypto++: Remove forced build with gcc.

Latest crypto++ version can be built with clang.

CQ-DEPEND=CL:590671
BUG= chromium:749849 
TEST=sudo emerge crypto++ works with both gcc and clang.

Change-Id: Ic4a15cef1443419e9706d92f6303e207d059a55c
Reviewed-on: https://chromium-review.googlesource.com/590673
Commit-Ready: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Caroline Tice <cmtice@chromium.org>

[delete] https://crrev.com/3cff2a06f9f1cfd1d4560695968f48fff003dc6c/chromeos/config/env/dev-libs/crypto++

Project Member

Comment 2 by bugdroid1@chromium.org, Jul 28 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/portage-stable/+/94a9ace0a8d9830f6c19384364a3cd0b2281c26b

commit 94a9ace0a8d9830f6c19384364a3cd0b2281c26b
Author: Manoj Gupta <manojgupta@google.com>
Date: Fri Jul 28 22:04:20 2017

dev-libs/crypto++: Upgrade to latest gentoo stable 5.6.5-r1.

Latest gentoo stable version 5.6.5-r1 can be built with clang.

Ebuild changes:
1. EAPI 6->5
2. KEYWORDS -> *
3. Added src_prepare to handle PATCHES

BUG= chromium:749849 
TEST=sudo emerge crypto++ works with both gcc and clang.

Change-Id: I7cd3cdbdecec9a8c7c1c4c0a36cbaa73d95fd1d7
Reviewed-on: https://chromium-review.googlesource.com/590671
Commit-Ready: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[delete] https://crrev.com/f3c36240c356b0c4bdb98c5854353bfbd7a91ced/dev-libs/crypto++/files/crypto++-5.6.1-fix_build_system.patch
[delete] https://crrev.com/f3c36240c356b0c4bdb98c5854353bfbd7a91ced/dev-libs/crypto++/crypto++-5.6.1-r1.ebuild
[delete] https://crrev.com/f3c36240c356b0c4bdb98c5854353bfbd7a91ced/dev-libs/crypto++/files/crypto++-5.6.1-rijndael.patch
[delete] https://crrev.com/f3c36240c356b0c4bdb98c5854353bfbd7a91ced/dev-libs/crypto++/files/crypto++-5.6.1-r3-fix_build_system.patch
[delete] https://crrev.com/f3c36240c356b0c4bdb98c5854353bfbd7a91ced/dev-libs/crypto++/crypto++-5.6.1-r3.ebuild
[delete] https://crrev.com/f3c36240c356b0c4bdb98c5854353bfbd7a91ced/dev-libs/crypto++/files/crypto++-5.6.1-sunos.patch
[add] https://crrev.com/94a9ace0a8d9830f6c19384364a3cd0b2281c26b/dev-libs/crypto++/files/crypto++-5.6.4-nonative.patch
[delete] https://crrev.com/f3c36240c356b0c4bdb98c5854353bfbd7a91ced/dev-libs/crypto++/files/crypto++-5.6.1-gcc-4.7.patch
[add] https://crrev.com/94a9ace0a8d9830f6c19384364a3cd0b2281c26b/dev-libs/crypto++/crypto++-5.6.5-r1.ebuild
[modify] https://crrev.com/94a9ace0a8d9830f6c19384364a3cd0b2281c26b/dev-libs/crypto++/Manifest
[add] https://crrev.com/94a9ace0a8d9830f6c19384364a3cd0b2281c26b/dev-libs/crypto++/files/crypto++-5.6.5-CVE-2017-9434.patch

Status: Verified (was: Started)

Sign in to add a comment