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

Issue 709590 link

Starred by 2 users
Status: Verified
Owner:
█ yunlian@chromium.org
Last visit > 30 days ago
Closed: May 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 674175



Sign in to add a comment

libarchive cannot be built with gcc for target.

Project Member Reported by yunlian@chromium.org, Apr 7 2017 
CC=x86_64-cros-linux-gnu-gcc emerge-caroline libarchive

configure:14609: checking for xmlInitParser in -lxml2
configure:14634: x86_64-cros-linux-gnu-gcc -o conftest -O2 -pipe -O2 -pipe -march=corei7 -g -fno-exceptions -fno-unwind-tables   -fno-asynchronous-unwind-tables  -clang-syntax  -I/usr/include/libxml2 -Wl,-O1 -Wl,-O2 -Wl,--as-needed conftest.c -lxml2  -llzma -lbz2 -lz  -lxml2 -lz -lm -ldl >&5
cc1: error: include location "/usr/include/libxml2" is unsafe for cross-compilation [-Werror=poison-system-directories]

Comment 1 by vapier@chromium.org, Apr 7 2017 

are we losing cross-compiler coverage because of the clang migration ?  sounds like we need poison logic added to clang like we have with gcc.  that -I path should have been rejected by clang too for the same reason.
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 8 2017 

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

commit e9966914d140c14e2c5e741ff49a81e697f1cdec
Author: Yunlian Jiang <yunlian@google.com>
Date: Sat Apr 08 22:56:36 2017

libarchive: upgraded package to upstream

Upgraded app-arch/libarchive to version 3.2.2-r1 for all

This upgrade enables us to build libarchive with gcc for
target packages.

BUG= chromium:709590 
TEST=`cbuildbot chromiumos-sdk` passes (builds sdk and 3 boards)
      CC=x86_64-cros-linux-gnu-gcc emerge-caroline libarchive passes.

Change-Id: I689a51018e6983e034c24d2966b43c0457bf89f4
Reviewed-on: https://chromium-review.googlesource.com/471907
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[delete] https://crrev.com/c27f3e071bdb467ae7281961b9ddd4fc131917d2/app-arch/libarchive/libarchive-3.1.2-r1.ebuild
[modify] https://crrev.com/e9966914d140c14e2c5e741ff49a81e697f1cdec/app-arch/libarchive/metadata.xml
[add] https://crrev.com/e9966914d140c14e2c5e741ff49a81e697f1cdec/app-arch/libarchive/files/CVE-2017-5601.patch
[delete] https://crrev.com/c27f3e071bdb467ae7281961b9ddd4fc131917d2/app-arch/libarchive/files/libarchive-3.1.2-CVE-2013-0211.patch
[modify] https://crrev.com/e9966914d140c14e2c5e741ff49a81e697f1cdec/app-arch/libarchive/Manifest
[add] https://crrev.com/e9966914d140c14e2c5e741ff49a81e697f1cdec/app-arch/libarchive/libarchive-3.2.2-r1.ebuild

Comment 3 by yunlian@chromium.org, May 12 2017

Status: Verified (was: Untriaged)
►

Sign in to add a comment