Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 32bfd703 authored by Elliott Hughes's avatar Elliott Hughes
Browse files

Remove the hacks needed to support ash and mksh concurrently. 

We no longer have ash, and we'd rather not have unnecessary symbolic links
on the system.

Change-Id: Icfb1a51f1baaf1861c203f6ed93843b094deb65d
parent a7f254fe

core/base_rules.mk

+1 −1
Original line number Diff line number Diff line
@@ -75,7 +75,7 @@ endif 
# Only the tags mentioned in this test are expected to be set by module

# makefiles. Anything else is either a typo or a source of unexpected

# behaviors.

ifneq ($(filter-out debug eng tests optional samples shell_mksh,$(LOCAL_MODULE_TAGS)),)

ifneq ($(filter-out debug eng tests optional samples,$(LOCAL_MODULE_TAGS)),)

$(warning unusual tags $(LOCAL_MODULE_TAGS) on $(LOCAL_MODULE) at $(LOCAL_PATH))

endif

core/build-system.html

+2 −4
Original line number Diff line number Diff line
@@ -247,7 +247,6 @@ for these). 
        for <code>eng</code>.

        <ul>

        <li>Installs modules tagged with: <code>eng</code>, <code>debug</code>,

            <code>shell_</code>$(TARGET_SHELL),

            <code>user</code>, and/or <code>development</code>.

        <li>Installs non-APK modules that have no tags specified.

        <li>Installs APKs according to the product definition files, in
@@ -267,7 +266,7 @@ for these). 
        <p>

        This is the flavor intended to be the final release bits.

        <ul>

        <li>Installs modules tagged with <code>shell_</code>$(TARGET_SHELL) and <code>user</code>.

        <li>Installs modules tagged with <code>user</code>.

        <li>Installs non-APK modules that have no tags specified.

        <li>Installs APKs according to the product definition files; tags

            are ignored for APK modules.
@@ -674,8 +673,7 @@ up.</p> 
<h4>LOCAL_MODULE_TAGS</h4>

<p>Set <code>LOCAL_MODULE_TAGS</code> to any number of whitespace-separated

tags.  If the tag list is empty or contains <code>droid</code>, the module

will get installed as part of a <code>make droid</code>.  Modules with the tag

<code>shell_</code>$(TARGET_SHELL) will also be installed. Otherwise, it will

will get installed as part of a <code>make droid</code>.  Otherwise, it will

only get installed by running <code>make &lt;your-module&gt;</code>

or with the <code>make all</code> pseudotarget.</p>

core/config.mk

+0 −3
Original line number Diff line number Diff line
@@ -114,9 +114,6 @@ TARGET_ERROR_FLAGS := -Werror=return-type -Werror=non-virtual-dtor -Werror=addre 
# TODO: do symbol compression

TARGET_COMPRESS_MODULE_SYMBOLS := false



# Default shell is mksh. Other possible value is ash.

TARGET_SHELL := mksh



# ###############################################################

# Include sub-configuration files

# ###############################################################

core/main.mk

+0 −2
Original line number Diff line number Diff line
@@ -622,12 +622,10 @@ tests_MODULES := $(sort \ 


# TODO: Remove the 3 places in the tree that use ALL_DEFAULT_INSTALLED_MODULES

# and get rid of it from this list.

# TODO: The shell is chosen by magic.  Do we still need this?

modules_to_install := $(sort \

    $(ALL_DEFAULT_INSTALLED_MODULES) \

    $(product_FILES) \

    $(foreach tag,$(tags_to_install),$($(tag)_MODULES)) \

    $(call get-tagged-modules, shell_$(TARGET_SHELL)) \

    $(CUSTOM_MODULES) \

  )

target/product/embedded.mk

+2 −0
Original line number Diff line number Diff line
@@ -58,8 +58,10 @@ PRODUCT_PACKAGES += \ 
    linker \

    logcat \

    logwrapper \

    mkshrc \

    service \

    servicemanager \

    sh \

    surfaceflinger \

    toolbox