Add deb builder for ArduPilot version #3
Conversation
khancyr
force-pushed
the
feature/debbuilder
branch
2 times, most recently
from
59aa0e0
to
af4438e
Compare
khancyr
force-pushed
the
feature/debbuilder
branch
from
af4438e
to
8a8de45
Compare
|
Nice! Should this go upstream? They already build debs. |
| @@ -0,0 +1,12 @@ | |||
| Package: ap-microxrceddsgen | |||
| Version: 0.0.1 | |||
There was a problem hiding this comment.
Ideally,this matches the version of the tool. Currently, it's not set, but official releases will have a version.
There was a problem hiding this comment.
yep, we can update this one, when you use the build script it update the Version to the right one's
| echo "Usage: $0 <version>" | ||
| exit 1 | ||
| fi | ||
|
|
There was a problem hiding this comment.
| set -euf -o pipefail | |
Catch errors earlier with this protection in shell.
|
|
||
| - uses: ./src/microxrceddsgen/.github/actions/install-apt-packages | ||
|
|
||
| - name: Get minimum supported version of CMake |
There was a problem hiding this comment.
Do we actually need cmake to build it?
There was a problem hiding this comment.
I just copied the current ci rules, I think it is used for the test as it download dds clients
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v3 |
There was a problem hiding this comment.
| - uses: actions/checkout@v3 | |
| - uses: actions/checkout@v4 |
v3 is EOL soon I think
|
@khancyr Got any time to finish this up? It's really great. |
|
A great next step would be to put this on an apt repo so we can have people |
add a simple way to create a deb file for the generator
also add a release build workflow. To trigger it, just create a github release with a tag and it will build the deb and put it as release artefact