Skip to content

Commit

Permalink
Added Debian files packages files and minor updates to catch when PWM…
Browse files Browse the repository at this point in the history
… is zero. (#971)

* Initial Version of AStarBox

Initial version of AStarBox as a 3rd-party driver.

* New version of AStarBox.cpp

New version detects if there is a problem or not with the I2C connection.

Also, changed the names to be consistent with Rodolphe's files. (AStarBox vs CAStarBox), and changed astarbox to indi-astarbox.

* Update README

Update to improve format.

* Update README.md

Again change format.

* Another update

Formatting again

* And again

Another format change.

* Hopefully final update of README

I hope I have this right now!

* Readme

Getting name right in Readme

* Refactor of AStarBox.cpp

Called openAllPorts into connect so can use results to indicate whether connection successful  or not.

Moved getPortPWM for ports 5 and 6 into connect too - when in openAllPports m_bLinked was false at that point so no values collected.

Now called after m_bLlinked is true so that values are collected.

* Merged Rodolphe's updates and mine

Merged Rodolphe's updates and mine.

m_bPortsOpen is tested before opening all ports, but getting PWM5 and 6 is now in connect.

* Tidied up port numbers

Replaced 5 and 6 by their ENUMS instead for consistency.

* Changed logic when PWM percent is zero

Previously, setting PWM to zero turned the port off. However, this prevented the status from being read so that if changed elsewhere, it would not be picked up.

Now, the ports are left on though with power of zero (same as turned off from a physical perspective).

Also, tidied up the return codes from setPortPWMDutyCyclePercent since this returns an integer rather than a bool,

* First Pass Debian Package

First pass debian package. Copied from indi-asi-power.

* Updated rules file

Changed to refer to indi-astarbox, rather than indi-air-power.

* Updated changelog

Updated channgelog - changed time from +100 to +0100 to put it in the correct format.
  • Loading branch information
mcgillca authored Sep 11, 2024
1 parent 0886679 commit dafbbdf
Show file tree
Hide file tree
Showing 7 changed files with 66 additions and 0 deletions.
5 changes: 5 additions & 0 deletions debian/indi-astarbox/changelog
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
indi-astarbox (1.0) buster; urgency=medium

* all - Initial release of astarbox driver

-- Colin McGill <info@astarbox.co.uk> Sun, 8 Sep 2024 16:25:00 +0100
1 change: 1 addition & 0 deletions debian/indi-astarbox/compat
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
10
23 changes: 23 additions & 0 deletions debian/indi-astarbox/control
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
Source: indi-astarbox
Section: science
Priority: extra
Maintainer: Colin McGill <info@astarbox.co.uk>
Build-Depends: debhelper (>= 5), pkg-config, cdbs, cmake, libindi-dev
Standards-Version: 3.9.1

Package: indi-astarbox
Architecture: any
Depends: ${shlibs:Depends}, ${misc:Depends}
Description: INDI Driver for AStarBox
INDI Driver for AStarBox power control.
This driver is compatible with any INDI client but only works on an AStarBox.

Package: indi-astarbox-dbg
Priority: extra
Section: debug
Architecture: any
Depends: indi-astarbox (= ${binary:Version}), ${misc:Depends}
Description: INDI Driver for AStarBox
INDI Driver for AStarBox power control.
This driver is compatible with any INDI client but only works on an AStarBox.
This package contains the debug symbols for the this driver, useful to investigate problems in INDI.
27 changes: 27 additions & 0 deletions debian/indi-astarbox/copyright
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
This package was debianized by Colin McGill <info@astarbox.co.uk> on
Sun, 8 Sep 2024 16:25:00 +100

Copyright:

Copyright 2024 AStarBox Ltd <<info AT astarbox DOT co DOT uk>

License:

This library is free software; you can redistribute it and/or modify
it under the terms of the GNU Lesser General Public License as published
by the Free Software Foundation; either version 2.1 of the License, or
(at your option) any later version.

This library is distributed in the hope that it will be useful, but
WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public
License for more details.

You should have received a copy of the GNU Lesser General Public License
along with this library; if not, write to the Free Software Foundation,
Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA

See /usr/share/common-licenses/LGPL

The Debian packaging is (C) 2024, Colin McGill <info@astarbox.co.uk> and
is licensed under the LGPL License, see /usr/share/common-licenses/LGPL
2 changes: 2 additions & 0 deletions debian/indi-astarbox/indi-astarbox.install
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
usr/bin
usr/share/indi
7 changes: 7 additions & 0 deletions debian/indi-astarbox/rules
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
#!/usr/bin/make -f

include /usr/share/cdbs/1/rules/debhelper.mk
include /usr/share/cdbs/1/class/cmake.mk

DEB_SRCDIR=indi-astarbox
DEB_DH_SHLIBDEPS_ARGS=-u--ignore-missing-info
1 change: 1 addition & 0 deletions debian/indi-astarbox/source/format
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
3.0 (quilt)

0 comments on commit dafbbdf

Please sign in to comment.