New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 806958 link

Starred by 1 user
Status: Fixed
Owner:
█ rahulchaudhry@chromium.org
Last visit > 30 days ago
Closed: Feb 2018
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 800984



Sign in to add a comment

app-text/opensp does not build with llvm-next

Project Member Reported by rahulchaudhry@chromium.org, Jan 29 2018 
From chromiumos-sdk build here: https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/chromiumos-sdk/builds/2850

There are build error from app-text/opensp. All of them seem to be of the form:

opensp-1.5.2-r3: ../include/Ptr.h:72:11: error: ISO C++11 does not allow access declarations; use
opensp-1.5.2-r3:       using declarations instead
opensp-1.5.2-r3:   Ptr<T>::isNull;
opensp-1.5.2-r3:           ^
opensp-1.5.2-r3:   using
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 1 2018 

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

commit 118b1baf82f9cc5d27f3fe475d89aded00725d9c
Author: Rahul Chaudhry <rahulchaudhry@chromium.org>
Date: Thu Feb 01 02:01:03 2018

app-text/opensp: get opensp-1.5.2-r6.ebuild from gentoo.

The ebuild is from
  https://gitweb.gentoo.org/repo/gentoo.git/plain/app-text/opensp/opensp-1.5.2-r6.ebuild
with two changes:
  - KEYWORDS="*"
  - Revert DEPEND/RDEPEND change from
      https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=d4c53d2eb994e37a8624d4c4533a5abf9318d7a7
    This change introduced a new dependency to net-libs/libnsl when
    using glibc-2.26. net-libs/libnsl ebuild in gentoo uses EAPI="6",
    which in turn requires upgrading Portage to a newer version.

BUG= chromium:806958 
TEST='sudo emerge app-text/opensp' works with llvm-next.

Change-Id: I8fbcc2e807fb3888218ad095048561156a817b6c
Reviewed-on: https://chromium-review.googlesource.com/895910
Commit-Ready: Rahul Chaudhry <rahulchaudhry@chromium.org>
Tested-by: Rahul Chaudhry <rahulchaudhry@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[delete] https://crrev.com/ddc746ee490635b1c3ef6d432ad049d008870c64/app-text/opensp/files/opensp-1.5-gcc34.patch
[modify] https://crrev.com/118b1baf82f9cc5d27f3fe475d89aded00725d9c/app-text/opensp/metadata.xml
[add] https://crrev.com/118b1baf82f9cc5d27f3fe475d89aded00725d9c/app-text/opensp/files/opensp-1.5.2-c11-using.patch
[rename] https://crrev.com/118b1baf82f9cc5d27f3fe475d89aded00725d9c/app-text/opensp/opensp-1.5.2-r6.ebuild

Comment 2 by rahulchaudhry@chromium.org, Feb 1 2018

Status: Fixed (was: Assigned)
►

Sign in to add a comment