Revert "Make com_android_i18n namespace visible"

Revert submission 1299494-i18nApex

Reason for revert: Breaking aosp_x86-eng on aosp-master
Reverted Changes:
I30fc3735b:Move ICU from ART APEX to i18n APEX
Icb7e98b5c:Calling @IntraCoreApi from core-icu4j should not c...
Ic7de63fe3:Move core-icu4j into I18n APEX
I65b97bdba:Make com_android_i18n namespace visible
Ia4c83bc15:Move v8 and libpac into i18n APEX
I10e6d4948:Move core-icu4j into i18n APEX
I8d989cad7:Move ICU from ART APEX into i18n APEX
I72216ca12:Move ICU into i18n APEX
Ief9dace85:Add shared library into i18n APEX and add the requ...
I7d97a10ba:Move libpac into i18n APEX
I90fff9c55:Move ICU from ART APEX into i18n APEX

Change-Id: I8e170a25fb334c12af5c2fe4a1fb4afafd56d096

Make com_android_i18n namespace visible

libnativeloader calls android_get_exported_namespace to link
this namepsace to classloader namespace.

Bug: 138994281
Test: CtsJniTestCases
Change-Id: I65b97bdba462a11d7389b84d873c4daa9c08b45d

Merge changes from topic "lc_sphal"

* changes:
  Allow SPHAL namespace to access system in case of legacy
  Move libz from VNDKSP to LLNDK in case of VNDK Lite

Add legacy test configuration

And deprecated --legacy option.

Instead, to run linkerconfig with legacy mode, run it without -v option.
(-v is for setting vndk version).
For example,
  $ linkerconfig -r <root> -t <target>

Legacy mode:
  ro.vndk.version is not set and VNDK is not available.
  but other apexes are available.

Bug: 140599044
Test: ./rundiff.sh
Change-Id: I81cba2d6c6466737fcb4dd33407837b016c42b78

Allow SPHAL namespace to access system in case of legacy

In current there is limited link from sphal to platform in case of
legacy device due to absence of VNDK related library list. Anyway, SPHAL
namespace requires access to libraries under /system/lib which is
VNDK-SP in treblelized device. This change adds extra access to system
libs from SPHAL namespace in case of legacy device.

Bug: 152932503
Test: m -j passed
Change-Id: I93fc82a36cf3c2ee33c9bd1f2bb8371bb2d6d700

Move libz from VNDKSP to LLNDK in case of VNDK Lite

In case of VNDK Lite libz should be listed in LLNDK rather than VNDKSP.
This process has been missed while we move the logic to linkerconfig.
This change updates library list based on the library file name in case
of VNDK lite device.

Bug: 141908078
Test: m -j passed
Change-Id: I22eb76619c1242ab26c1d611a08b98177b08285c
Merged-In: I22eb76619c1242ab26c1d611a08b98177b08285c

Add more configurations for rundiff tests

stage0:
  no apexes. this configuration corresponds to the one for
  apexd-bootstrap process.

stage1:
  with bootstrap apexes. this corresponds to the one right after
  apexd-bootstrap finishes

stage2:
  with all apexes. this corresponds to the one for every other
  processes.

product-enabled:
  ro.product.vndk.version is set.

vndk-lite:
  ro.vndk.lite is set to true.

Bug: 140599044
Test: ./rundiff.sh
Change-Id: If7dc6aedf05e2078643e52559c580de425693dc5

Refactor rundiff.sh

(copied from reverted d283fb913db051ef0653c9ea1af5d2d865dbbbf0)

. Add --update to update golden_output
. Do not rely on ANDROID_BUILD_TOP (required to be run on cloud)

Bug: n/a
Test: ./rundiff.sh # no changes
Change-Id: I098dabd2c5cd06f18a806cf66f593ea930f02945

Revert "Make diff test as presubmit"

This reverts commit d283fb913db051ef0653c9ea1af5d2d865dbbbf0.

Reason for revert: flaky (b/152265282)

Change-Id: Ie73fd1967384281f3e727b2482bf1ee7ffec38d5

Make diff test as presubmit

so that changes to the output can be tracked.

Bug: 140599044
Test: atest --host linkerconfig_diff_test
Change-Id: Ib7a026ca5cb794a614279ce2eee53c6ef8219d1f

Update testdata/golden_output

libselinux.so is added to system_stub_libraries in
74d75a773d200542ac3439fc166aa87ef40e0104

Bug: 151053366
Test: ./rundiff.sh
Change-Id: Id69eded5f144958676d2b4b72774efa4fb82211f

Merge "Make libseliux a stub library"

Make libseliux a stub library

libselinux is currently being copied to APEXes. This is risky because
the library is not designed to be portable; part of it is tied to the
specific version of the Android that it was developed for.

This change fixes the problem by declaring that the library supports
a stub with the list of C APIs that are included in the stub. Then there
is only one copy of libselinux in /system/lib and other APEXes use the
copy by dynamically linking to it.

Bug: 151053366
Test: m com.android.adbd. It doesn't include libselinux in it.
Test: m com.android.adbd-deps-info. then inspect
out/soong/com.android.adbd-deps-info.txt. The dependency to libselinux
is shown as '(external)'.

Change-Id: I445db2d0c5103b8b3267caa094f5d91259f1f491

Update test data

Update test data based on the latest update - removing libstatssocket.so
from stub libraries.

Test: ./rundiff.sh passed
Change-Id: I47c2e0104e89f3dbfed15e2e586f416f988b578c

Remove libstatssocket from variableloader.

Need this so libstatssocket can be added to statsd apex

Bug: 145923416
Test: m
Change-Id: I5630321d26e208b540c408f82d3f9759dd1a9c14
Merged-In: I5630321d26e208b540c408f82d3f9759dd1a9c14

Remove cronet apex from visible_apexes am: a8eeddac31

Change-Id: I84d40640f1e3130500ca03e5015555043be6af0b

Generates jni.config.txt am: 8e5cc8d49e

Change-Id: I14823f7d785294f63d75bef6a094ae19b66c934a

Remove cronet apex from visible_apexes

It is now automatically set as visible because it has jni_libs.

Bug: 146420818
Bug: 143733063
Test: CronetApiTest
Change-Id: I7d506e63aece907d7cc00b405aca23a196f9a3db

Generates jni.config.txt

This file is consumed by libnativeloader so that JNI libraries in an
APEX can be loaded from classloader-namespace created for java libraries
in the same apex.

Bug: 143733063
Bug: 146420818
Test: ./rundiff.sh
Change-Id: I4b80d5c1c2a10cd6b644187e6ddd416c22cddf27

Merge "Make APEXes with JNI libs "visible" in "system" section" am: b263b8a15e

Change-Id: I4563d296129763cdbefe0d90a49c03ceef7b1380

Merge "Make APEXes with JNI libs "visible" in "system" section"

Make sphal and vndk namespace available from legacy devices am: 8a550bb09b

Change-Id: Icd04634f47edbaef5e95d5aef682512fe6740c37

Make sphal and vndk namespace available from legacy devices

linkerconfig failed to complete the execution from ARC++ which was
caused by missing vndk variables because ARC++ is legacy device. This
change allows linkerconfig to generate ld.config.txt for SWCodec by
allowing SPHal and VNDK namespaces even from legacy devices

Bug: 149914078
Test: m -j passed
Test: Tested from ARC++ w/ betty
Change-Id: I5412434f0d2b3b69b916ca512828498a7eba72a3

Replace the hardcoded ART namespace config with an autogenerated one. am: 0e9de4a5d7

Change-Id: Ifb50a8da35d31a5dc531ac940985216f409267bd

Rename ART APEX directory to what it is on device. am: b7e0ae0c97

Change-Id: Id5070761eb98f45c78f08245af0f35e7470341e7

Exit on failure in rundiff.sh. am: 73cccc439c

Change-Id: I86a4678d6d3bf1ce3928e5d1c2611d8ca9febdea

Update golden linkerconfig from change in https://r.android.com/1242397. am: 957847ebd3

Change-Id: I9b0554ee04db0fbf4c9d343d44816b904aadbbeb

Replace the hardcoded ART namespace config with an autogenerated one.

Test: Build and boot
Test: system/linkerconfig/rundiff.sh
Bug: 147082642
Change-Id: If6e45da6823180c8ef36f066753bdba167f64df5

Rename ART APEX directory to what it is on device.

Preparation to make an InitializeWithApex call in art.cc produce the
right paths.

Test: system/linkerconfig/rundiff.sh
Bug: 147082642
Change-Id: If6116a906dcd0ec983dec8746ab004cb0c835f58

Exit on failure in rundiff.sh.

In particular, don't show a diff created with an old binary if
linkerconfig failed to compile.

Test: system/linkerconfig/rundiff.sh
  (with and without a syntax error in a linkerconfig .cc file)
Bug: 140599044
Change-Id: I1449de75d20e8a40f26bac087048f869a0f468a5

Update golden linkerconfig from change in https://r.android.com/1242397.

Test: system/linkerconfig/rundiff.sh
Bug: 140599044
Change-Id: Ia4cdf12bcc37564951cbfc7ea286fd3bc1a6a933

Merge "Add golden output of linkerconfig" am: 2c87cccfbd

Change-Id: I549a7f2f90e9b1a9e78f56ac1b0684588740bd98

Merge "Add golden output of linkerconfig"

Make APEXes with JNI libs "visible" in "system" section

It could be done by explicitly listing them when building "system"
section. This change does it automatically by checking if an apex has
jni libs in it.

Bug: 149363889
Test: atest --host linkerconfig_contents_fulltest
      build/boots

Change-Id: Ibb23268ab3e5dcfe0a8f8f1c98f7594c083eafac

Merge "Add libadbd_fs to list of stub dependencies." am: e04eef3242

Change-Id: I283fdcd61aec1b236fe91cff9c54a66898be2587

Merge "Add libadbd_fs to list of stub dependencies."

Merge "[adbwifi] Add adbd apex exported libs to system namespace." am: c005be7589

Change-Id: I5440261e02501e6fac27fee126485a22dea9de3a

Merge "[adbwifi] Add adbd apex exported libs to system namespace."

Merge "Trim root path from output" am: b7c162435b

Change-Id: If7db732fa5a1e267c777c0610608dc3492a77749

Merge "Trim root path from output"

Merge "Define vndk_product namespace for product apps" am: 6eb18354c1

Change-Id: Ifde5ffcae2bdad31e068f2fabfbe9a1838b14a21

Merge "Define vndk_product namespace for product apps"

Merge "Make /linkerconfig/<apexname> directories accessible to others" am: b52f719faf

Change-Id: Id755473867942a98d93d1091a3e7556fccbd43bd

Merge "Make /linkerconfig/<apexname> directories accessible to others"

Add golden output of linkerconfig

./rundiff.sh runs linkerconfig on host and compares the output files
with golden output.

Bug: 140599044
Test: ./rundiffs.sh
Change-Id: I1b2892d70fc514cf70a28eb6e041ad0bfa232de1

Trim root path from output

When linkerconfig runs on host with --root argument, it prints root as
well like following:

$ linkerconfig --vndk R --root /tmp/test_root
...
namespace.foo.search_paths = /tmp/test_root/product/${LIB}
..

This change trims root from path variables and apex paths so the output
look more like real output.

Bug: n/a
Test: build/boot
      run linkerconfig on host and compare outputs

Change-Id: Icc91f297047f059684a42d825048e50e26873520

Make /linkerconfig/<apexname> directories accessible to others

When linkerconfig is exec'ed by init, umask is by default set to 0x0077.
As a result, even though we create the directories with mkdir(...,
0755), they are marked as inaccessible for group and others.

Fixing the issue by explicitly setting umask to 0x0022.

Bug: 144914078
Test: build a device with flattened APEX. Check the mode bits of the
directories of under /linkerconfig.

$ ls -al /linkerconfig
total 96
drwxr-xr-x  9 root root   200 2020-02-21 13:47 .
drwxr-xr-x 22 root root  4096 2020-02-21 13:54 ..
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.adbd
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.art
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.conscrypt
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.media.swcodec
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.os.statsd
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.runtime
drwx--xr-x  2 root root    60 2020-02-21 13:47 com.android.sdkext
-rw-r--r--  1 root root 91651 2020-02-21 13:47 ld.config.txt

Change-Id: Id2f7f8a363ba239943942ba6e3bc984cd8f630b0

Add libadbd_fs to list of stub dependencies.

Test: treehugger
Change-Id: Iebb108c212ace8470efef6c2838eaefdd4b4be70

[adbwifi] Add adbd apex exported libs to system namespace.

These libraries will be used by system_server
(libandroid_servers.so).

Bug: b/149181583
Bug: b/111434128

Test: boots, adbd is not crashing.
Test: cat /proc/`pidof system_server`/maps | grep libadb
Verify that only libadb_pairing_auth.so, libadb_pairing_connection.so,
and libadb_pairing_server.so are being used.
Test: cat /proc/`pidof adbd`/maps | grep libadb
Change-Id: I8b645c9aca2465d5231b3cb21ca85fc3ebfc9093

Define vndk_product namespace for product apps

Unbundled apps may use vndk-sp libs from the system section. However,
product apps must use the vndk-sp libs for the product vndk version
that can be different from the vendor vndk version. So we must define
a vndk namespace for product partition.

For this purpose, BuildVndkNamespace() function requires an
additional parameter to indicate which partition is using the
namespace.
Using this function, the system section has an additional namespace
called "vndk_product" which is used only for unbundled product apps.

Test: atest linkerconfig_modules_unittest
Bug: 149063221
Change-Id: I3658efdc75e5c9a0ac3b92b9afc68aff2d4cb300

Merge "Rename APEX Namespace" am: 52120b489b

Change-Id: I1ad4220320ce62274f553afa7e81b322e82e1f51

Merge "Rename APEX Namespace"

Merge "Fix full content test" am: 9fa8d06ecc

Change-Id: I41c27ed81cfb803954d773a07a4d0747bea4b33f

Merge "Fix full content test"

Fix full content test

Full content test was not working properly because the way of checking
if VNDK is enabled has been changed. This fix generates test context to
mock it available for VNDK.

Test: atest passed
Change-Id: I46064d1d5505dc5c6af438e69faae166dee9dc58

Rename APEX Namespace

Current APEX Namespace is named with APEX name itself, which also uses
.(dot) so linker configuration can keep the syntax safe. Update name of
namespace from APEX by replacing '_' with '.' so keep configuration safe
for future syntax update.

Bug: 148826508
Test: m -j passed
Test: boot succeeded from cuttlefish and walleye
Change-Id: I9185c909678e2e39d539622d51d8f378b819b466

Add com.android.os.statsd to legacy linkerconfig am: 2b465ce881

Change-Id: Ib2a29f358385d1b67a27f58e900dcd76a2efa20f

Add com.android.os.statsd to legacy linkerconfig

Similar to aosp/1237667, but for legacy stuff.

Bug: None
Test: Legacy stuff boots.
Change-Id: Iad17f5529ea6bb520c0ee5a4071a82ffd7c2cc27

Merge "Update linkerconfig for libs in statsd" am: 5504faa670

Change-Id: I969a8e8828d5e9273c863688c48d6d2ba9297d95

Merge "Update linkerconfig for libs in statsd"

Merge "Add com.android.os.statsd to linkerconfig" am: 91fccce607

Change-Id: Ia2aa4670b96557434b1c1c8675409e17515f00a3

Merge "Add com.android.os.statsd to linkerconfig"

Add com.android.os.statsd to linkerconfig

Adds com.android.os.statsd to System BuildSection. I believe this is to
get the namespace the proper visibility for jni.

Test: build/boots
Change-Id: Id516e79e12fde608888587ec15ab0f3358f4918d
Merged-In: Id516e79e12fde608888587ec15ab0f3358f4918d
(cherry picked from commit 91454166a50d86d2367c8ec5b4686087ecc13184)

Merge "Remove extra using statements" am: 28133def11

Change-Id: Iab0964273993949a3eb3c0c331eb58bc75e3c5e1

Merge "Remove extra using statements"

Update linkerconfig for libs in statsd

Bug: 148605160
Test: m -j
Change-Id: Idb6308e8f518536e97106c445b1945945040a4c3

Merge "Fix Camera bug" am: 5c811ebf1c

Change-Id: I78f49c50833251698c1fc5351e3a5ee66819e872

Merge "Fix Camera bug"

Remove extra using statements

android::base::Errorf and android::base::ErrnoErrorf are now #define's
instead of functions, so remove their using statements.

Test: build
Change-Id: I7333b916ee87668fb9fd15b9293de5da41c43409

Fix Camera bug

Bug: 149538115
Test: Flash and take a picture, verified no crash
Change-Id: Idd455f8803cf6f1b1f4c95d854a6077e33bae5e1

Allow libincident.so to be linked from apex am: 4e15bf53f0

Change-Id: Iea0a310c93367544a21a0964a7985647f888ef52

Allow libincident.so to be linked from apex

Bug: 145923087
Test: m -j
Change-Id: I787f15750e19c5028b334161c5a7d4fa13003c6f

Fix linkerconfig_backward_compatibility_test am: 2dd4e4d2cb

Change-Id: I978ed22941f49942163419544398f8925af3ddaa

Fix linkerconfig_backward_compatibility_test

The current GetValue() falls back to GetProperty(). This makes the test
show different result on different running environment. For consistent
test result, MockVariables() now overrides all product/vendor variables.

Bug: n/a
Test: atest --host linkerconfig_backward_compatibility_test
      atest linkerconfig_backward_compatibility_test

Change-Id: I4b7533e10058b0236a8aef4c54fbb58f3bec0ccc

Read vndk libraries txt files from vndk apex am: 7c2b73a71f

Change-Id: I24a5c1482e9a059025dac3aba1ea1e1cf485b7b2

Generate vendor stuff only if vndk is avilable am: 3f83cfb602

Change-Id: I90769d8e5ae9ea3116454aa3bf4d43d04021a877

Read vndk libraries txt files from vndk apex

Vndk apex contains some txt files which list libraries it contains.
- llndk.libraries.txt
- vndksp.libraries.txt
- vndkcore.libraries.txt
- vndkprivate.libraries.txt

These files can be read from vndk apex instead of /system/etc.

Bug: 145184886
Test: build / flash / boot
Change-Id: I2b242b71768af2a30de519ccd4d73b1a86e46382

Generate vendor stuff only if vndk is avilable

Linkerconfig runs even when there is no apexes available. When vndk apex
is not available, having vendor section or namespaces requiring vndk
libraries is meaningless.

This is necessary because we'd like to load some variables from vndk
apex, but relying on those variables when vndk apex is not available may
fail to generate linker configuration.

Bug: 145184886
Test: build / flash / boot
Change-Id: I7c77757254a63c8bc7470c4ae87f2db3779e6465

Merge "Add required modules for linkerconfig" am: bade76eb13

Change-Id: I9cad270d6fbfa1125f1524954f2b8f7765228499

Merge "Add required modules for linkerconfig"

Merge "2 more missing Result::ok() calls in system/linkerconfig" am: ffacdd9477

Change-Id: I0fbc15f0cb04554567ee042fb97a91e09a6faf6f

Merge "2 more missing Result::ok() calls in system/linkerconfig"

Merge changes I2ead6a77,I6bf2204a am: 4add3c0b71

Change-Id: I3b673660490dc97f6d080a9c0112a6b62a98c99c

Merge changes I2ead6a77,I6bf2204a

* changes:
  Remove ResolveVariables()
  Make ConfigWriter simpler

Add required modules for linkerconfig

linkerconfig relies on some txt files.
Adding those modules as required:
- llndk.libraries.txt
- vndkprivate.libraries.txt
- vndkcore.libraries.txt
- vndkcorevariant.libraries.txt
- vndksp.libraries.txt
- sanitizer.libraries.txt

Bug: N/A
Test: m installclean; m linkerconfig
      check if those txt files are installed together

Change-Id: I8ab98591026dad06b3140043c54b7134ca3dac33

2 more missing Result::ok() calls in system/linkerconfig

Test: cd system/linkerconfig && atest
Change-Id: I2c6f8eb4b81065b90512f88a9f248affecacf019

Merge "Use apex_name for the name of linker namespace" am: 2fa82ebd25

Change-Id: I92d3bdbf307e06d8439d957d779312498c12e20f

Merge "Use apex_name for the name of linker namespace"

Remove ResolveVariables()

ResolveVariables() takes long because it uses regex pattern search and
replace them with values. It can be avoided by resolving variables while
configurations are being built.

For easy transition, "Var()" utility function is introduced which should
be called after LoadVariables(). This will be enforced by moving Var()
into Context so that it only be called when Context is available (which
is, after LoadVariables() is called)

Bug: 149020353
Test: atest --test-mapping system/linkerconfig
      build / flash / boot
Change-Id: I2ead6a77dcc9ec14b4e083897a2818714438beb5

Make ConfigWriter simpler

- Remove varg WriteLine
- No prefix
- Add WriteVars for writing multiple lines using "="/ "+="

This refactoring is a prep for removing "ResolveVariables()" which is very slow.

Bug: 149020353
Test: atest --test-mapping system/linkerconfig
Change-Id: I6bf2204a9ac0794b0bdf8028999a3543821a26be

Last missing Result::ok() calls in system/linkerconfig am: 51be4fe748

Change-Id: I71de5d9c437049101bdf9c185242058776981c35

Last missing Result::ok() calls in system/linkerconfig

No functionality change.

Test: cd system/linkerconfig && atest
Change-Id: I5ffa14d1d935c8e01c694338400e366a19895ba6

Remove unused files am: 35df8b6140

Change-Id: I3d3de0df7afb119313feb7704d5e03a054b09510

Use apex_name for the name of linker namespace

Till now short names are used for apex namespaces but there are no
explicit rule for this. Instead of extracting namespace name from apex
name (com.android.foo => foo), we'd better use apex name as it is
because it is well-established identifier for apexes.

Bug: 148826508
Test: build/flash/boot
      atest

Change-Id: I4f77eb85652a999274844436ba14b5c05831cf5b

Remove unused files

conscrypt and neuralnetworks namespaces are now generated automatically.

Bug: N/A
Test: build
Change-Id: Ic13d3f4206ecfda1937adf2111977b279fba05ef

Auto-generate runtime namespace am: b818280515

Change-Id: I9ae5f026915c98aa1070560fd1923536c0a8d3b7

Auto-generate media namespace am: 834efc8c45

Change-Id: Idff81f99b6a7baf98882615653563e7cc69b09f2

Auto-generate apex namespaces: cronet,nn,conscrypt am: 9960aca469

Change-Id: I7e6ecab4463e0c50864ef186952d549b6a63b40f

Auto-generate runtime namespace

runtime apex provides bionic(libc/libm/libdl) libraries. But actually
they are supposed to be accessed via symlinks in /system/lib (i.e.
system namespace).

By amending soong not to emit those bionic libs as providing libs of
runtime apex, now runtime apex can be auto-generated by linkerconfig.

Bug: 144664390
Test: build / flash / boot
Change-Id: Idda42801a608e5fcb495bfccdd07ca6c0666a53d

Auto-generate media namespace

Now, media namespace is auto-generated.
This is done by adding /apex/<name>/lib as permitted path as well as
search path, because media apex needs a permitted path
/com.android.media/lib/extractors.

Bug: 144664390
Test: build / flash / boot
      atest --test-mapping system/linkerconfig
Change-Id: I9186b0768b3bce3ad19f3d14010725af0bbf5861