enable parallel builds for debian#715
enable parallel builds for debian#715christianrauch wants to merge 1 commit intoros-infrastructure:masterfrom
Conversation
|
The Debian and RPM jobs run on buildfarm workers with a per-package approach to parallelism, wherein each package is allocated a single core. That said, this is a buildfarm implementation detail and it would be nice if it didn't "leak" into Bloom like this. The RPM builds are parallel by default and a buildfarm flag limits them to a single core. If there is a similar approach for the Debian builds so that we can maintain the desired behavior on the buildfarm, we can consider this change. |
|
The better way here would be to bump the compat version (Debian is at 13 nowadays), as proposed in #643.
By exporting |
Elegance defined. |
3d9024d to
bd9607f
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #715 +/- ##
=========================================
Coverage ? 54.58%
=========================================
Files ? 52
Lines ? 6363
Branches ? 1128
=========================================
Hits ? 3473
Misses ? 2551
Partials ? 339 ☔ View full report in Codecov by Sentry. |
|
This didn't actually work for me. I still had to specify |
|
@russkel can you share a log? Preferably with all package version like sbuild will get it. The feature is used in all over Debian so it would be surprising if it fails for you. |
|
I worry that our stuff has deviated from the upstream bloom repo and may have caused it (https://github.com/Greenroom-Robotics/bloom)
The package versions should be the latest or quite modern on both our noble and jammy images. A log is not so easy to get at this second. |
bd9607f to
33aef15
Compare
By default, depending on the CMake generator, build jobs do not run in parallel. Fix this by passing the
--parallelflag which then allows to run parallel jobs by passing options toDEB_BUILD_OPTIONS, e.g.DEB_BUILD_OPTIONS="parallel=8" fakeroot debian/rules binary.