Revert "Revert "Do not use PCRE2 in libselinux_vendor""

This reverts commit 832c8466e9.

Also make sure regex.c is compiled for all targets that libselinux is
built for.

Bug: 37919668
Bug: 37343404
Test: Build and boot

Change-Id: I6bcced2d3675899cf976ad3cd30e591539d2aaa1
Signed-off-by: Sandeep Patil <sspatil@google.com>
This commit is contained in:
Sandeep Patil 2017-05-03 07:21:51 -07:00
parent 43d548e365
commit a1b5897b8b

View file

@ -1,6 +1,4 @@
common_CFLAGS = [ common_CFLAGS = [
"-DUSE_PCRE2",
// Persistently stored patterns (pcre2) are architecture dependent. // Persistently stored patterns (pcre2) are architecture dependent.
// In particular paterns built on amd64 can not run on devices with armv7 // In particular paterns built on amd64 can not run on devices with armv7
// (32bit). Therefore, this feature stays off for now. // (32bit). Therefore, this feature stays off for now.
@ -26,7 +24,6 @@ cc_defaults {
"src/label.c", "src/label.c",
"src/label_support.c", "src/label_support.c",
"src/matchpathcon.c", "src/matchpathcon.c",
"src/regex.c",
"src/setrans_client.c", "src/setrans_client.c",
"src/sha1.c", "src/sha1.c",
], ],
@ -92,16 +89,9 @@ cc_defaults {
local_include_dirs: ["include"], local_include_dirs: ["include"],
export_include_dirs: ["include"], export_include_dirs: ["include"],
static: {
whole_static_libs: ["libpcre2"],
},
shared: {
shared_libs: ["libpcre2"],
},
} }
cc_library { cc_library_shared {
name: "libselinux_vendor", name: "libselinux_vendor",
defaults: ["libselinux_defaults"], defaults: ["libselinux_defaults"],
vendor: true, vendor: true,
@ -118,9 +108,11 @@ cc_library {
name: "libselinux", name: "libselinux",
defaults: ["libselinux_defaults"], defaults: ["libselinux_defaults"],
host_supported: true, host_supported: true,
cflags: ["-DUSE_PCRE2"],
srcs: [ srcs: [
"src/label_file.c", "src/label_file.c",
"src/regex.c",
], ],
target: { target: {
@ -166,6 +158,13 @@ cc_library {
version_script: "exported.map", version_script: "exported.map",
}, },
}, },
static: {
whole_static_libs: ["libpcre2"],
},
shared: {
shared_libs: ["libpcre2"],
},
} }
// If one attempts to comment out the sefcontext_compile target in the // If one attempts to comment out the sefcontext_compile target in the
@ -182,6 +181,7 @@ cc_library {
//cc_binary_host { //cc_binary_host {
// name: "sefcontext_compile", // name: "sefcontext_compile",
// cflags: common_CFLAGS, // cflags: common_CFLAGS,
// cflags: ["-DUSE_PCRE2"],
// srcs: ["utils/sefcontext_compile.c"], // srcs: ["utils/sefcontext_compile.c"],
// //
// static_libs: ["libselinux"], // static_libs: ["libselinux"],