Skip to content

Commit e9830ab

Browse files
Add some comments to python script
1 parent 50f82e1 commit e9830ab

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

.ci/get_install_build_packages.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,20 +13,24 @@
1313
system_dir = os.getenv("TUE_SYSTEM_DIR")
1414
packages = (f.path for f in os.scandir(os.path.join(system_dir, "src")) if f.is_dir())
1515

16-
install_build_pkgs = set()
17-
build_pkgs = set()
16+
install_build_pkgs = set() # All packages that need to be build and require all deps to be installed
17+
build_pkgs = set() # All packages that need to be build
1818

1919
for pkg_path in packages:
2020
pkg = os.path.split(pkg_path)[-1]
21+
22+
# Messages need to build and require there depencies
2123
for msg_type in ["action", "msg", "srv"]:
2224
if os.path.isdir(os.path.join(pkg_path, msg_type)):
2325
install_build_pkgs.add(pkg)
2426
break
2527

28+
# Pkg with python modules need to build to be added to PYTHONPATH
2629
if os.path.isfile(os.path.join(pkg_path, "setup.py")):
2730
catkin_package = catkin_pkg.package.parse_package(
2831
os.path.join(pkg_path, catkin_pkg.package.PACKAGE_MANIFEST_FILENAME)
2932
)
33+
# If cpp code in pkg, deps need to be installed
3034
for dep in catkin_package.build_depends:
3135
if "cpp" in dep.name: # Might be replaced by a better check to determine their is cpp code in this pkg
3236
install_build_pkgs.add(pkg)

0 commit comments

Comments
 (0)