Skip to content

Commit d1de2e8

Browse files
committed
extract_utils: simplify package / copy logic
Change-Id: Ic951626fad84fce8981db35f6932b6c27b3a8f81
1 parent 27c8331 commit d1de2e8

File tree

1 file changed

+14
-6
lines changed

1 file changed

+14
-6
lines changed

extract_utils/file.py

+14-6
Original file line numberDiff line numberDiff line change
@@ -528,18 +528,26 @@ def __add_file(self, file: File, section: Optional[str]):
528528

529529
if FileArgs.MAKE_COPY_RULE_ONLY in file.args:
530530
is_package = False
531-
else:
532-
is_package = self.__is_file_package(file)
533-
534-
if is_package or file.is_package:
535-
if is_package and file.is_package:
531+
elif self.__is_file_package(file):
532+
if file.is_package:
536533
color_print(
537534
f'{file.dst}: already a package, no need for -',
538535
color=Color.YELLOW,
539536
)
537+
538+
is_package = True
539+
else:
540+
is_package = file.is_package
541+
542+
if is_package:
540543
self.package_files.add(file)
541544

542-
if not is_package or FileArgs.MAKE_COPY_RULE in file.args:
545+
if FileArgs.MAKE_COPY_RULE in file.args:
546+
is_copy_rule = True
547+
else:
548+
is_copy_rule = not is_package
549+
550+
if is_copy_rule:
543551
self.copy_files.add(file)
544552

545553
self.all_files.add(file)

0 commit comments

Comments
 (0)