Commit 4540dae
authored
fix(templates): update robotics Jazzy templates per new setup script (#544)
* fix(templates): update robotics Jazzy templates per new setup script
- Remove Kisak Mesa and Kobuk Intel Graphics PPAs (no longer needed)
- Remove explicit Mesa packages (use Ubuntu defaults)
- Update RealSense repo URL to librealsense.realsenseai.com
- Pin librealsense2-dkms, librealsense2, and ros-jazzy-librealsense2
to exact versions in the packages list using _version syntax
- Replace first-boot systemd oneshot with direct DKMS install
- Add post-deployment APT version pins as upgrade protection
- Update metadata keywords (add dkms, remove mesa/ppa)
* fix(templates): correct RealSense package version pins
- Remove version pin from librealsense2-dkms (uses 1.3.x version scheme)
- Remove version pin from ros-jazzy-librealsense2 (only 2.57.7 available)
- Keep librealsense2 SDK pinned to 2.56.5-0~realsense.17055
- Simplify post-deployment APT pin to librealsense2 SDK only
* fix(templates): address Copilot review comments
- Add 'dkms' keyword to metadata in both raw and ISO templates
- Update OpenVINO pin comment to reference setup-robotics-jazzy.sh
instead of init_jazzy.sh
* fix(templates): address review comment on librealsense pin comments
Remove implementation detail comment about librealsense2-dkms version
scheme from the RealSense post-deployment pin section in both templates.
* fix(resolver,templates): fix oneAPI version conflict in robotics Jazzy builds
Fix the Go dependency resolver to handle version constraint conflicts when
packages from the same repository (same priority) need replacement. Also
update both robotics Jazzy templates (raw + ISO) to align with the
setup-robotics-jazzy.sh reference script.
Resolver changes:
- Track requested packages in resolvedDeps after dequeuing so later
transitive dependencies reuse the explicitly requested version
- Make constraint-satisfying replacements unconditional instead of
gating on comparePriorityBehavior(), which incorrectly rejected
valid replacements when both packages had the same repo priority
Template changes (raw + ISO):
- Add explicit versioned oneAPI packages before ros-jazzy-desktop to
lock intel-oneapi-runtime-compilers to 2025.3.3-30
- Change ROS_DOMAIN_ID from 45 to 42 (matches reference script)
- Expand oneAPI apt pin to cover runtime, compiler, and mkl patterns
- Fix RealSense apt pin to use librealsense2* wildcard
Tests:
- Add TestConstraintSatisfyingReplacementSamePriority to verify
unconditional replacement when constraint-satisfying candidate exists
- Add TestRequestedPackageVersionPreserved to verify explicitly
requested package versions are not overridden by transitive deps1 parent 4c59906 commit 4540dae
4 files changed
Lines changed: 184 additions & 125 deletions
File tree
- image-templates
- internal/ospackage/debutils
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| |||
53 | 53 | | |
54 | 54 | | |
55 | 55 | | |
56 | | - | |
| 56 | + | |
57 | 57 | | |
58 | 58 | | |
59 | 59 | | |
| |||
107 | 107 | | |
108 | 108 | | |
109 | 109 | | |
110 | | - | |
111 | | - | |
112 | | - | |
113 | | - | |
114 | | - | |
115 | | - | |
116 | | - | |
117 | | - | |
118 | | - | |
119 | | - | |
120 | | - | |
121 | | - | |
122 | 110 | | |
123 | 111 | | |
124 | 112 | | |
125 | 113 | | |
126 | 114 | | |
127 | 115 | | |
128 | | - | |
129 | | - | |
130 | | - | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
131 | 121 | | |
132 | 122 | | |
133 | 123 | | |
| |||
159 | 149 | | |
160 | 150 | | |
161 | 151 | | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
162 | 159 | | |
163 | 160 | | |
164 | 161 | | |
| |||
180 | 177 | | |
181 | 178 | | |
182 | 179 | | |
183 | | - | |
184 | | - | |
185 | | - | |
186 | | - | |
187 | | - | |
188 | | - | |
189 | | - | |
190 | | - | |
191 | | - | |
192 | | - | |
193 | | - | |
194 | | - | |
195 | | - | |
196 | | - | |
197 | | - | |
198 | | - | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
199 | 185 | | |
200 | 186 | | |
201 | 187 | | |
| |||
213 | 199 | | |
214 | 200 | | |
215 | 201 | | |
216 | | - | |
| 202 | + | |
217 | 203 | | |
218 | 204 | | |
219 | 205 | | |
| |||
222 | 208 | | |
223 | 209 | | |
224 | 210 | | |
225 | | - | |
226 | | - | |
227 | | - | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
228 | 214 | | |
229 | | - | |
230 | | - | |
| 215 | + | |
| 216 | + | |
231 | 217 | | |
232 | 218 | | |
233 | 219 | | |
| |||
236 | 222 | | |
237 | 223 | | |
238 | 224 | | |
239 | | - | |
240 | | - | |
241 | | - | |
242 | | - | |
243 | | - | |
244 | | - | |
245 | | - | |
246 | | - | |
247 | | - | |
248 | | - | |
249 | 225 | | |
250 | 226 | | |
251 | 227 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| |||
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
58 | | - | |
| 58 | + | |
59 | 59 | | |
60 | | - | |
61 | 60 | | |
62 | 61 | | |
63 | 62 | | |
| |||
133 | 132 | | |
134 | 133 | | |
135 | 134 | | |
136 | | - | |
137 | | - | |
138 | | - | |
139 | | - | |
140 | | - | |
141 | | - | |
142 | | - | |
143 | | - | |
144 | | - | |
145 | | - | |
146 | | - | |
147 | | - | |
148 | 135 | | |
149 | 136 | | |
150 | 137 | | |
151 | 138 | | |
152 | 139 | | |
153 | 140 | | |
154 | | - | |
155 | | - | |
156 | | - | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
157 | 146 | | |
158 | 147 | | |
159 | 148 | | |
| |||
185 | 174 | | |
186 | 175 | | |
187 | 176 | | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
188 | 184 | | |
189 | 185 | | |
190 | 186 | | |
| |||
206 | 202 | | |
207 | 203 | | |
208 | 204 | | |
209 | | - | |
210 | | - | |
211 | | - | |
212 | | - | |
213 | | - | |
214 | | - | |
215 | | - | |
216 | | - | |
217 | | - | |
218 | | - | |
219 | | - | |
220 | | - | |
221 | | - | |
222 | | - | |
223 | | - | |
224 | | - | |
| 205 | + | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
225 | 210 | | |
226 | 211 | | |
227 | 212 | | |
| |||
239 | 224 | | |
240 | 225 | | |
241 | 226 | | |
242 | | - | |
| 227 | + | |
243 | 228 | | |
244 | 229 | | |
245 | 230 | | |
| |||
248 | 233 | | |
249 | 234 | | |
250 | 235 | | |
251 | | - | |
252 | | - | |
253 | | - | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
254 | 239 | | |
255 | | - | |
256 | | - | |
| 240 | + | |
| 241 | + | |
257 | 242 | | |
258 | 243 | | |
259 | 244 | | |
| |||
262 | 247 | | |
263 | 248 | | |
264 | 249 | | |
265 | | - | |
266 | | - | |
267 | | - | |
268 | | - | |
269 | | - | |
270 | | - | |
271 | | - | |
272 | | - | |
273 | | - | |
274 | | - | |
275 | 250 | | |
276 | 251 | | |
277 | 252 | | |
| |||
0 commit comments