Skip to content

Commit a29b7e0

Browse files
committed
Installation folders for main packages and submodules should be properly isolated and not install all into
the target directory of the main packagee, here learning/bolero. This fixes the setup for learning/bolero when enabling building with separated prefixes in Autoproj. Otherwise a cleanup of package learning/bolero will include the submodules if isolation is not given.
1 parent 88171c1 commit a29b7e0

File tree

2 files changed

+39
-16
lines changed

2 files changed

+39
-16
lines changed

libs.autobuild

Lines changed: 38 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,34 +1,56 @@
1-
def learning_package(srcDir)
1+
def learning_package(baseDir, subDir = "")
2+
baseDir.chomp!("/")
3+
4+
if subDir.empty?
5+
srcDir = baseDir
6+
else
7+
srcDir = File.join(baseDir, subDir)
8+
prefixDir = baseDir + "-submodule" + "/" + subDir
9+
end
10+
211
if Gem::Version.new(Autobuild::VERSION) <= Gem::Version.new('1.7.2')
3-
learning_path="learning/bolero/"
412
cmake_package srcDir do |pkg|
513
yield block if block_given?
6-
pkg.srcdir = learning_path
14+
pkg.srcdir = baseDir
15+
pkg.prefix = prefixDir if prefixDir
16+
17+
if File.exist?(File.join(pkg.srcdir, "setup.py"))
18+
bin, version, sitelib = Rock.activate_python_path(pkg)
19+
pkg.define 'PYTHON_EXECUTABLE', bin if bin
20+
pkg.define 'BINDINGS_PYTHON', bin
21+
end
22+
723
pkg.post_import do
824
pkg.srcdir = srcDir
925
Autoproj.manifest.load_package_manifest(pkg.name)
1026
end
1127
end
1228
else
1329
cmake_package srcDir do |pkg|
14-
pkg.importdir = "learning/bolero/"
30+
pkg.importdir = baseDir
31+
pkg.prefix = prefixDir if prefixDir
32+
if File.exist?(File.join(pkg.srcdir, "setup.py"))
33+
bin, version, sitelib = Rock.activate_python_path(pkg)
34+
pkg.define 'PYTHON_EXECUTABLE', bin if bin
35+
pkg.define 'BINDINGS_PYTHON', bin
36+
end
1537
end
1638
end
1739
end
1840

1941
in_flavor 'master','next','stable' do
2042
cmake_package("tools/catch")
21-
learning_package("learning/bolero/src/representation/promp")
22-
learning_package("learning/bolero/bolero/wrapper")
23-
learning_package("learning/bolero/include")
24-
learning_package("learning/bolero/src/bl_loader")
25-
learning_package("learning/bolero/src/controller")
26-
learning_package("learning/bolero/src/optimizer/cmaes_optimizer")
27-
learning_package("learning/bolero/src/optimizer/pso_optimizer")
28-
learning_package("learning/bolero/src/environment/cec13_test_functions")
29-
learning_package("learning/bolero/src/environment/function_approximation")
30-
learning_package("learning/bolero/src/environment/mars_environment")
31-
learning_package("learning/bolero/src/environment/mountain_car")
32-
learning_package("learning/bolero/src/representation/dmp")
43+
learning_package("learning/bolero", "src/representation/promp")
44+
learning_package("learning/bolero", "bolero/wrapper")
45+
learning_package("learning/bolero", "include")
46+
learning_package("learning/bolero", "src/bl_loader")
47+
learning_package("learning/bolero", "src/controller")
48+
learning_package("learning/bolero", "src/optimizer/cmaes_optimizer")
49+
learning_package("learning/bolero", "src/optimizer/pso_optimizer")
50+
learning_package("learning/bolero", "src/environment/cec13_test_functions")
51+
learning_package("learning/bolero", "src/environment/function_approximation")
52+
learning_package("learning/bolero", "src/environment/mars_environment")
53+
learning_package("learning/bolero", "src/environment/mountain_car")
54+
learning_package("learning/bolero", "src/representation/dmp")
3355
learning_package("learning/bolero")
3456
end

source.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ imports:
44
# for basic setup and osdeps definitions
55
- github: rock-core/package_set
66
# includes helping libraries
7+
- github: rock-core/package_set
78
- github: rock-simulation/package_set
89

910
version_control:

0 commit comments

Comments
 (0)