[sdk/kde-builder] kde_builder_lib: remove: Remove unneeded check for filtered_modules

Andrew Shark <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit dc4e77716e4c80429932d5d88cebf3762202e4ae by Andrew Shark.
Committed on 16/07/2026 at 22:05.
Pushed by ashark into branch 'master'.

remove: Remove unneeded check for filtered_modules

There is no possibility that there will be a duplicate
project in the list, because sort_modules_into_build_order()
iterated by graph keys, and obviously, dict cannot have
duplicate keys.

M  +0    -10   kde_builder_lib/application.py

https://invent.kde.org/sdk/kde-builder/-/commit/dc4e77716e4c80429932d5d88cebf3762202e4ae

diff --git a/kde_builder_lib/application.py b/kde_builder_lib/application.py
index ce95fff6..bc719503 100644
--- a/kde_builder_lib/application.py
+++ b/kde_builder_lib/application.py
@@ -363,16 +363,6 @@ class Application:
                     logger_app.debug(f"Project y[{module.name}] was removed due to being ignored.")
         modules = filtered_modules
 
-        filtered_modules: list[Module] = []
-        for module in modules:
-            if module in filtered_modules:
-                logger_app.debug("Skipping duplicate project " + module.name)
-            else:
-                logger_app.debug(f"Adding {module} to project list")
-                filtered_modules.append(module)
-
-        modules = filtered_modules
-
         for module in modules:
             module.set_resolved_repository()
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.