Is there a reason this can't go to Debian first, and then flow down to Ubuntu? I will be happy to sponsor to Debian.
I have reviewed this package; here are some items which should be addressed:
1. The current version of debhelper is 12, while this package has 10. Could that be updated?
2. Standards-version is at 4.1.2, could that be updated to 4.3.0?
3. Can this package only build on the four architectures you have specified in the PPA? Ubuntu builds for two additional architectures and Debian builds for many more; if it can only build for those four, you should replace "any" in debian/control with a space-separated list of architectures.
4. After doing a copyright review, I found that some of the files only have a copyright notice for the author, and not the standard LGPL header, which should be fixed upstream. Additionally, in one file, another author is credited: "Additional code for pwmSetClock by Chris Hall <email address hidden>" (wiringPi/wiringPi.c), and in wiringPi/wiringShift.h the copyright dates go back to 2009.
You aren't required to fix all of them (in fact, all of them are info or lower, so you don't have to), but addressing them would be beneficial and make a higher-quality package.
Unsubscribing sponsors for now, let me know if you have any questions.
Hi Dave, thanks for your work on this!
Is there a reason this can't go to Debian first, and then flow down to Ubuntu? I will be happy to sponsor to Debian.
I have reviewed this package; here are some items which should be addressed: wiringPi. c), and in wiringPi/ wiringShift. h the copyright dates go back to 2009.
1. The current version of debhelper is 12, while this package has 10. Could that be updated?
2. Standards-version is at 4.1.2, could that be updated to 4.3.0?
3. Can this package only build on the four architectures you have specified in the PPA? Ubuntu builds for two additional architectures and Debian builds for many more; if it can only build for those four, you should replace "any" in debian/control with a space-separated list of architectures.
4. After doing a copyright review, I found that some of the files only have a copyright notice for the author, and not the standard LGPL header, which should be fixed upstream. Additionally, in one file, another author is credited: "Additional code for pwmSetClock by Chris Hall <email address hidden>" (wiringPi/
Here's what Lintian returns: short-descripti on libwiringpi2-dev libwiringpi2 does-not- install- examples examples/ trailing- whitespace debian/changelog (line 27) uses-old- debhelper- compat- version 10 date-standards- version 4.1.2 (released 2017-11-30) (current is 4.3.0) autopkgtest- missing metadata- file-is- missing watch-uses- insecure- uri http:// git.drogon. net/wiringpi. git watch-contains- dh_make- template (line 1) watch-does- not-check- gpg-signature error-in- binary usr/lib/ x86_64- linux-gnu/ libwiringPi. so.2.50 certianly certainly no-bindnow usr/lib/ x86_64- linux-gnu/ libwiringPi. so.2.50 no-bindnow usr/lib/ x86_64- linux-gnu/ libwiringPiDev. so.2.50 control- file usr/lib/ x86_64- linux-gnu/ libwiringPi. so.2.50 control- file usr/lib/ x86_64- linux-gnu/ libwiringPiDev. so.2.50 in-library- section libs usr/bin/gpio description- is-probably- too-short
I: wiringpi source: duplicate-
P: wiringpi source: package-
P: wiringpi source: file-contains-
P: wiringpi source: package-
I: wiringpi source: out-of-
I: wiringpi source: testsuite-
X: wiringpi source: upstream-
I: wiringpi source: debian-
I: wiringpi source: debian-
X: wiringpi source: debian-
I: libwiringpi2: spelling-
I: libwiringpi2: hardening-
I: libwiringpi2: hardening-
I: libwiringpi2: no-symbols-
I: libwiringpi2: no-symbols-
X: wiringpi: application-
I: wiringpi: extended-
You can find detailed information about each tag here: https:/ /lintian. debian. org/tags/
You aren't required to fix all of them (in fact, all of them are info or lower, so you don't have to), but addressing them would be beneficial and make a higher-quality package.
Unsubscribing sponsors for now, let me know if you have any questions.