Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Add debian package files #927

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

silentjet
Copy link

No description provided.

python3-setuptools (>= 0.6.29),
python3-numpy,
python3-pytest,
X-Python-Version: all
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not familiar with this tag set, but this one doesn't look right....

libxml2-dev, libxslt1-dev, zlib1g-dev,
python3-setuptools (>= 0.6.29),
python3-numpy,
python3-pytest,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure we have a hard dep for this.

Multi-Arch: same
Depends: ${python3:Depends}, ${shlibs:Depends}, ${misc:Depends}, python3-future, python3-lxml
Provides: ${python3:Provides}
Description: Pymavlink is a Python implementation of the MAVLink protocol
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It really is three distinct things.

The most commonly used mavlink bindings generator. Which also generates the Python mavlink bindings.

A set of utilities that make working with the Python bindings easier, most notably an object to handle opening data streams.

The command-line utilities that you've mentioned.

I don't think you really need to fix this in your stuff here - it's no different to many other places....

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh well, I just have to specify something here, thus stating what I can at my best :D
I would appreciate a more accurate Summary and Longer description.

@@ -0,0 +1,5 @@
pymavlink (2.4.41) UNRELEASED; urgency=medium
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this the changelog for the package or the packaged software?

@silentjet
Copy link
Author

Hi @peterbarker
Thanks for your review. Was busy for quite some time. I'll update it and will upload a clean version. This one is quite made-on-knees, thus dirty (and that's why WIP is here).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants