Bug#1073027: RFS: parlatype/4.2-1 -- Minimal audio player for manual speech transcription
Hi Phil,
thank you for your review! I saw it quite late, because I thought I was
automatically subscribed to this bug.
As for your comment at "1. Build", I intentionally didn't use these
substitution variables because in
/usr/share/doc/gobject-introspection/README.Debian.gz at "3. Runtime
dependencies" it says:
* Add ${gir:Depends} to -dev packages if they contain GIR XML
* Add ${gir:Provides} to -dev packages if they contain GIR XML
And there is no GIR XML in this -dev package.
Regards
Gabor
OpenPGP_0xC892C2F1310F600B.asc
Description: OpenPGP public key
OpenPGP_signature.asc
Description: OpenPGP digital signature
Bug#1073027: RFS: parlatype/4.2-1 -- Minimal audio player for manual speech transcription
Package: sponsorship-requests Severity: normal Dear mentors, I am looking for a sponsor for my package "parlatype": * Package name : parlatype Version : 4.2-1 Upstream contact : that's actually me * URL : www.parlatype.xyz * License : GPL-3+ * Vcs : https://salsa.debian.org/gkarsay/parlatype Section : sound The source builds the following binary packages: parlatype - Minimal audio player for manual speech transcription parlatype-common - Minimal audio player for manual speech transcription (arch-independent files) libparlatype7 - Library for Parlatype - runtime version libparlatype-dev - Library for Parlatype - development version libparlatype-doc - Documentation files for the Parlatype library gir1.2-parlatype-5.0 - Library for Parlatype - gir bindings To access further information about this package, please visit the following URL: https://mentors.debian.net/package/parlatype/ Alternatively, you can download the package with 'dget' using this command: dget -x https://mentors.debian.net/debian/pool/main/p/parlatype/parlatype_4.2-1.dsc Changes since the last upload: parlatype (4.2-1) unstable; urgency=medium . * New upstream version 4.2 * Update watch file * Refresh patch #02 * Add patch for autopkgtest * Remove glade option and copyright of its assets * Update copyright for added/removed files * Update homepage to www.parlatype.xyz * Change app ID to xyz.parlatype.Parlatype (install) * Raise GTK dependency from 3 to 4 * Update gir package to 5.0 * Update lib package to libparlatype7 * Move arch-independent files into package parlatype-common * Enable cross-compiling GObject introspection data * Bump Standards-Version to 4.7.0 (no changes needed) * Lintian override for gir (false positive) Other info: * Although I intended to fix cross compiling, the Salsa CI cross compile test didn't succeed. * According to mentors' QA information there's an unused override. I had that lintian warning locally and as described in the override I think it's a false positive. Regards, Gabor Karsay
reprotest fails in dh_auto_test. What to do?
Dear Mentors, my package parlatype is failing reprotest in dh_auto_test. In detail a test in appstream-util is failing: it checks the release date in shipped appstream data and if the time is varied, that release date is in the future, resulting in a failed test, failed build and failed reprotest. Log output is: --- command --- /usr/bin/appstream-util validate-relax --nonet com.github.gkarsay.parlatype.appdata.xml --- stdout --- com.github.gkarsay.parlatype.appdata.xml: FAILED: • attribute-invalid : timestamp is in the future --- stderr --- Validation of files failed --- This is not uploaded yet, only in Debian Salsa: https://salsa.debian.org/gabork-guest/parlatype What would be the best solution to this problem:: 1) Disabling appstream-util test entirely. 2) Running reprotest with DEB_BUILD_OPTIONS=nocheck There is a commit for Salsa CI pipeline removing this option because it might hide reproducibility problems: https://salsa.debian.org/salsa-ci-team/pipeline/commit/4942def0eac1808aeead80cd87e8d4c6560a96f1 There is a similar discussion in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=786644 "reproducible builds should vary whether nocheck is added to DEB_BUILD_OPTIONS" This package doesn't change the binary during the test. The first build could be run with the test, the second build could be run without and there should be no difference in the binary. I understand I could change this in Salsa CI, but not for the tests run on the archive. I failed to find out which options are used for the repro tests run on the archive and I guess they can't be changed on a per-package basis? 3) Is there a possibility to continue reprotest even if dh_auto_test failed, making it non-fatal? Those are the solutions I could think of, what would you advise? Regards, Gabor
Bug#867727: RFS: parlatype/1.5.2-1 [ITP]
Ross Gammon schrieb am 2017-09-24 um 21:03: The only thing I spotted, was the lack of a docbase file to help people fire up the docs locally: https://www.debian.org/doc/manuals/maint-guide/dother.en.html#doc-base Done. Added the doc-base file and put a symlink from /usr/share/gtk-doc/html/parlatype to /usr/share/doc/libparlatype-doc/html. I looked how other packages are doing this and I found both install locations: e.g. gtk installs docs in /usr/share/doc and symlinks to /usr/share/gtk-doc but clutter for example does it the other way round, installing in /usr/share/gtk-doc and symlinking to /usr/share/doc as I did. Would you like to maintain this within the Debian Multimedia Team? If so: https://wiki.debian.org/DebianMultimedia/Join and then follow the stuff in the Develop Packaging link there (to create the git repository etc). We can always do this before the next upload if you prefer. Just let me know. I'm not sure. I guess it would be better but at the moment I don't have much time to learn a new workflow. Besides I probably wouldn't do much work for other packages in the Multimedia Team. I'd prefer not to join at the moment if that is possible. Some minor things we can work on for next time: 1. Lintian complains about there being no upstream changelog. As you are upstream, maybe you can fix this? It is a nice service to our users to see what changed in each version without going online. The Debian changelog only contains changes in the packaging. Yes, I can release an upstream changelog next time. There are already release notes for every release. 2. It would be nice to add some autopkgtests. For parlatype, that might just be to run the binary command without a file to check that it installs & doesn't crash. For the library & gir library you can compile a simple app against the libraries to check they compile & run OK. If you search on Debian Codesearch you will find examples. I plan that for the next minor release (which might or might not be this year). Regards, Gabor
Bug#867727: Request for sponsoring Parlatype -- audio player for transcription
Juhani Numminen schrieb am 2017-09-15 um 15:24: debian/parlatype.manpages is only needed when upstream build system doesn't install the man pages, please remove it. done debian/watch can be updated to version 4 (changing the first line is enough). done Additionally removed build-dependency on libfile-fcntllock-perl and added copyright for translations. For some reason after upload mentors claims there is no debhelper compatibility level set and a watchfile is missing although everything is set. Best regards, Gabor
Bug#867727: Request for sponsoring Parlatype -- audio player for transcription
Juhani Numminen schrieb am 2017-09-10 um 17:27: Ross Gammon kirjoitti 10.09.2017 klo 13:15: I would be happy to take a look at this. Unfortunately, I am quite busy at the moment so I can't promise much this week. I have cc'd you because you did not say whether you are subscribed to the Multimedia list. I have also copied in the RFS bug so that other potential sponsors see this and take over if they are looking for something to do :-) Regards, Ross I can review the package and offer my comments in the meantime. Disclaimer: I am not Debian Developer or Debian Maintainer. Thank you Ross and Juhani! Your package is not lintian-clean, please investigate (lintian -EviI --pedantic) and fix what you can. W: parlatype source: newer-standards-version 4.0.1 (current is 4.0.0) I guess no need to override this. I: parlatype source: testsuite-autopkgtest-missing There are no tests and it's only informational, should I override this? P: parlatype source: debian-watch-may-check-gpg-signature fixed debian/control: I think Section: sound would be more suitable than Section: gnome. done You can drop some version requirements, please run 'cme check dpkg'. done, intltool didn't need versioning Latest Policy version is 4.1.0. done, changed watch file (checking for signature) Didn't test or change for "4.15: Packages should build reproducibly" Do you expect me to test reproducibility? Please use https for the homepage address. done The Debian changelog for new packages is usually very simple. This should be enough: parlatype (1.5.2-1) unstable; urgency=medium * Initial release (Closes: #868886) -- Gabor Karsay Thu, 10 Aug 2017 17:43:18 +0200 done, removed old changelog entries. They were either never released, just for private purposes, or Ubuntu PPA releases. debian/copyright should use the established short license names, i.e. GPL-3+ for "version 3 or later" and CC-BY-SA-4, not CCBYSA-4. They are listed in the format specification. done For the CC licenses, I think it is not enough to just refer to a web page, as the terms must be readable offline. done, included full CC licenses See if any http URL can be changed to https in this file, too. done In debian/rules, based on my testing, I can tell these two lines are not needed: DPKG_EXPORT_BUILDFLAGS = 1 include /usr/share/dpkg/buildflags.mk done, removed I have still a question: There's a build-dependency for libfile-fcntllock-perl only to avoid a warning in the build log "dpkg-gencontrol: warning: File::FcntlLock not available; using flock which is not NFS-safe". But it builds also without, can it stay for a cleaner build log or should I remove it? Thank you Gabor
Bug#867727: RFS: parlatype/1.5.1-1 [ITP] follow-up
Follow-up after uploading new version 1.5.2-1. To access further information about this package, please visit the following URL: https://mentors.debian.net/package/parlatype Alternatively, one can download the package with dget using this command: dget -x https://mentors.debian.net/debian/pool/main/p/parlatype/parlatype_1.5.2-1.dsc Changes since last upload to mentors: * New upstream bugfix release * Bump standards version to 4.0.1 (no changes) * Switch compat level 9 to 10 * Bump debhelper to 10 in Build-Depends * Remove dh-autoreconf from Build-Depends * Remove --with autoreconf in rules * Update install path for help files * Update install path for AppStream data * Update icon names (install and copyright) * Install DBus service * Install symbolic icon * Override lintian's hardening-no-fortify-functions: false positive * Add Build-Depends appstream-util and desktop-file-utils for checks * Copyright: change URL format to https Anyone willing to sponsor or review? Regards Gabor Karsay
Bug#867727: RFS: parlatype/1.5.1-1 [ITP]
I have re-uploaded the package to mentors, closing ITP: #868886, published my packaging history, added the Vcs fields to the control file and added a patch for Plasma desktops.
Bug#867727: RFS: parlatype/1.5.1-1 [ITP]
Package: sponsorship-requests Severity: wishlist Dear mentors, I am looking for a sponsor for my package "parlatype" * Package name: parlatype Version : 1.5.1-1 Upstream Author : Gabor Karsay <gabor.kar...@gmx.at> * URL : http://gkarsay.github.io/parlatype/ * License : GPL-3 Section : gnome It builds those binary packages: gir1.2-parlatype-1.0 - Library for Parlatype - gir bindings libparlatype-dev - Library for Parlatype - development version libparlatype-doc - Documentation files for the Parlatype library libparlatype1 - Library for Parlatype - runtime version parlatype - Minimal audio player for manual speech transcription parlatype-libreoffice-helpers - Set of macros for LibreOffice To access further information about this package, please visit the following URL: https://mentors.debian.net/package/parlatype Alternatively, one can download the package with dget using this command: dget -x https://mentors.debian.net/debian/pool/main/p/parlatype/parlatype_1.5.1-1.dsc More information about parlatype can be obtained from http://gkarsay.github.io/parlatype/. I am also the upstream author. The first public release was in January 2016, the program is in a good state, is already translated in a few languages and fills a gap for an easy to use transcription utility (manual transcription). I'm not aware of another program in Debian for GNOME, targeting transcriptions and also offering a waveform. I was packaging it already for a time for Ubuntu. The package was built with pbuilder in a sid chroot. There is one lintian warning that I don't know what to do with: hardening-no-fortify-functions. I think the compiler flags are set right, but I don't know if I should override it. There's also one thing unclear to me about the procedure: Is this sponsorship request enough or should I file a WNPP bug, too? Thank you Gabor Karsay
