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

Commit 07a79235 authored by Priyanka Advani (xWF)'s avatar Priyanka Advani (xWF) Committed by Gerrit Code Review
Browse files

Revert "Remove unassigned event log tags feature" 

Revert submission 3400623-remove_unassigned_event_log_tags

Reason for revert: Droidmonitor created revert due to b/383362497. Will be verifying through ABTD before submission.

Reverted changes: /q/submissionid:3400623-remove_unassigned_event_log_tags

Change-Id: I2f65b8327c1fc43f248685d8fe83bcb94f8e4bf4
parent 6ec58216

core/Makefile

+18 −3
Original line number Diff line number Diff line
@@ -970,12 +970,27 @@ systemimage: 


# -----------------------------------------------------------------



.PHONY: event-log-tags



# Produce an event logs tag file for everything we know about, in order

# to properly allocate numbers.  Then produce a file that's filtered

# for what's going to be installed.



all_event_log_tags_file := $(TARGET_OUT_COMMON_INTERMEDIATES)/all-event-log-tags.txt



event_log_tags_file := $(TARGET_OUT)/etc/event-log-tags



# Include tags from all packages that we know about

all_event_log_tags_src := \

    $(sort $(foreach m, $(ALL_MODULES), $(ALL_MODULES.$(m).EVENT_LOG_TAGS)))



$(all_event_log_tags_file): PRIVATE_SRC_FILES := $(all_event_log_tags_src)

$(all_event_log_tags_file): $(all_event_log_tags_src) $(MERGETAGS) build/make/tools/event_log_tags.py

	$(hide) mkdir -p $(dir $@)

	$(hide) $(MERGETAGS) -o $@ $(PRIVATE_SRC_FILES)



$(call declare-0p-target,$(all_event_log_tags_file))



# Include tags from all packages included in this product, plus all

# tags that are part of the system (ie, not in a vendor/ or device/

# directory).
@@ -987,13 +1002,13 @@ event_log_tags_src := \ 
      $(filter-out vendor/% device/% out/%,$(all_event_log_tags_src)))



$(event_log_tags_file): PRIVATE_SRC_FILES := $(event_log_tags_src)

$(event_log_tags_file): $(event_log_tags_src) $(MERGETAGS)

$(event_log_tags_file): PRIVATE_MERGED_FILE := $(all_event_log_tags_file)

$(event_log_tags_file): $(event_log_tags_src) $(all_event_log_tags_file) $(MERGETAGS) build/make/tools/event_log_tags.py

	$(hide) mkdir -p $(dir $@)

	$(hide) $(MERGETAGS) -o $@ $(PRIVATE_SRC_FILES)

	$(hide) $(MERGETAGS) -o $@ -m $(PRIVATE_MERGED_FILE) $(PRIVATE_SRC_FILES)



$(eval $(call declare-0p-target,$(event_log_tags_file)))



.PHONY: event-log-tags

event-log-tags: $(event_log_tags_file)



ALL_DEFAULT_INSTALLED_MODULES += $(event_log_tags_file)

core/config.mk

+1 −1
Original line number Diff line number Diff line
@@ -731,7 +731,7 @@ PROGUARD_HOME := external/proguard 
PROGUARD := $(PROGUARD_HOME)/bin/proguard.sh

PROGUARD_DEPS := $(PROGUARD) $(PROGUARD_HOME)/lib/proguard.jar

JAVATAGS := build/make/tools/java-event-log-tags.py

MERGETAGS := $(HOST_OUT_EXECUTABLES)/merge-event-log-tags

MERGETAGS := build/make/tools/merge-event-log-tags.py

APPEND2SIMG := $(HOST_OUT_EXECUTABLES)/append2simg

VERITY_SIGNER := $(HOST_OUT_EXECUTABLES)/verity_signer

BUILD_VERITY_METADATA := $(HOST_OUT_EXECUTABLES)/build_verity_metadata

core/definitions.mk

+1 −1
Original line number Diff line number Diff line
@@ -1555,7 +1555,7 @@ endef 
define transform-logtags-to-java

@mkdir -p $(dir $@)

@echo "logtags: $@ <= $<"

$(hide) $(JAVATAGS) -o $@ $<

$(hide) $(JAVATAGS) -o $@ $< $(PRIVATE_MERGED_TAG)

endef

core/java.mk

+1 −0
Original line number Diff line number Diff line
@@ -140,6 +140,7 @@ ifneq ($(strip $(logtags_sources)),) 
logtags_java_sources := $(patsubst %.logtags,%.java,$(addprefix $(intermediates.COMMON)/logtags/, $(logtags_sources)))

logtags_sources := $(addprefix $(LOCAL_PATH)/, $(logtags_sources))



$(logtags_java_sources): PRIVATE_MERGED_TAG := $(TARGET_OUT_COMMON_INTERMEDIATES)/all-event-log-tags.txt

$(logtags_java_sources): $(intermediates.COMMON)/logtags/%.java: $(LOCAL_PATH)/%.logtags $(TARGET_OUT_COMMON_INTERMEDIATES)/all-event-log-tags.txt $(JAVATAGS) build/make/tools/event_log_tags.py

	$(transform-logtags-to-java)

tools/event_log_tags.py

+20 −17
Original line number Diff line number Diff line
@@ -14,21 +14,21 @@ 


"""A module for reading and parsing event-log-tags files."""



import dataclasses

import re

import sys

from typing import Optional



@dataclasses.dataclass

class Tag:

  tagnum: int

  tagname: str

  description: Optional[str]

  filename: str

  linenum: int

class Tag(object):

  __slots__ = ["tagnum", "tagname", "description", "filename", "linenum"]



  def __init__(self, tagnum, tagname, description, filename, linenum):

    self.tagnum = tagnum

    self.tagname = tagname

    self.description = description

    self.filename = filename

    self.linenum = linenum





class TagFile:

class TagFile(object):

  """Read an input event-log-tags file."""

  def AddError(self, msg, linenum=None):

    if linenum is None:
@@ -76,10 +76,13 @@ class TagFile: 
          self.options[parts[1]] = parts[2:]

          continue



        if parts[0] == "?":

          tag = None

        else:

          try:

            tag = int(parts[0])

          except ValueError:

          self.AddError("\"%s\" isn't an integer tag" % (parts[0],))

            self.AddError("\"%s\" isn't an integer tag or '?'" % (parts[0],))

            continue



        tagname = parts[1]
@@ -125,8 +128,8 @@ def WriteOutput(output_file, data): 
      out = sys.stdout

      output_file = "<stdout>"

    else:

      out = open(output_file, "w")

    out.write(data)

      out = open(output_file, "wb")

    out.write(str.encode(data))

    out.close()

  except (IOError, OSError) as e:

    print("failed to write %s: %s" % (output_file, e), file=sys.stderr)