Hi Joost, On Sun, Aug 09, 2026 at 07:40:55PM +0200, Joost van Baal-Ilić wrote: > Hi Roland, > > I'm happy you found the time to work on this :) > > Le Sun, Aug 09, 2026 at 07:26:39PM +0200, Roland Hieber a écrit : > > On Fri, May 22, 2026 at 03:56:43PM +0200, Roland Hieber wrote: > > > I've pushed it to my Salsa namespace for now for the initial review: > > > https://salsa.debian.org/rohieb/mutt-ical > > > > > > I also noticed that I still have to assemble the debian/copyright, which > > > was not > > > really needed for our internal use case… I can do that next week. > > > > > > If all the packaging is ready, the repo could also be transferred to the > > > python-packaging namespace. > > > > Alright, it's been a bit more than a week, but I consider this ready now. > > I've > > fixed some things that Joost mentioned to me personally, also added a man > > page > > (based on the project's REAMDE) and fixed a lot of other lintian > > complaints. (It > > still warns about 'hyphen-in-upstream-part-of-debian-changelog-version', > > since > > the 2.0.2 release tag is missing in the upstream repo, I've resorted to a > > 'git > > describe'-style version number.) > > > > My current state is at > > <https://salsa.debian.org/rohieb/mutt-ical/-/commits/debian%2Flatest>, and > > I've > > uploaded the resulting package to mentors.debian.org for sponsorship review: > > <https://mentors.debian.net/package/mutt-ical/> > > I had a quick look at your work:
Thanks! > @ https://mentors.debian.net/package/mutt-ical/ : > > – Package has lintian warnings: mutt-ical source > W missing-debian-watch-file-standard > [debian/watch] I have the feeling that this is a false-positive. I converted the file to Version 5 format (because my lintian told me so ;)), and the warning does not happen on my machine with lintian v2.138.0. The full warning text mentions: "The first non-comment line of debian/watch should be a version= or Version: declaration.", which is definitely the case here… 🤔 > W recommended-field > (in section for source) Priority [debian/control:1] Same for this one. This warning was removed in lintian v2.128.0 according to <https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1111513>. > P hyphen-in-upstream-part-of-debian-changelog-version > 2.0.2-20240916-g7c795ee1fd74 [debian/changelog:1] That indeed also happens on my machine, but I'm not sure how to handle it and keep 'gbp buildpackage' happy. Upstream forgot to push the Git tag for version 2.0.2 (I've opened an upstream issue about that), so I set the version in d/changelog to something that can be parsed by 'git rev-parse' (using the 'git describe' format, which must include at least two dashes). The 2.0.1 release is not buildable by pybuild due to a syntax error in pyproject.toml which was fixed in 2.0.2. However, when re-read my own bug report [2], strictly speaking there is no 2.0.2 release (on GitHub) yet, even if there claims to be one on PyPI [3]. [2]: https://github.com/asciipip/mutt-ical/issues/5 [3]: https://pypi.org/project/mutt-ical/ When thinking about it again, the best way to solve this is by packaging the latest release on GitHub, 2.0.1, instead, and pull in the changes between 2.0.1 and 2.0.2 as a Debian patch series, and name it '2.0.1-1` or even `2.0.1+really2.0.2-1` in debian/changelog. Does that sound reasonable? > It has, in d/control: > > Vcs-Git: https://salsa.debian.org/rohieb/mutt-ical.git > > Maybe that could better be https://salsa.debian.org/debian/ (so that > cooperation would be easier) or https://salsa.debian.org/python-team/packages > (after an ACK from the Python team, and after consulting the Team's policy > document at https://www.debian.org/doc/packaging-manuals/python-policy/ ; I > believe having [email protected] as Maintainer would require > that.) I'm all for it, but I'm not sure who can move it to another namespace. I certainly don't have the access rights for it. > (Also: https://salsa.debian.org/rohieb/mutt-ical/-/pipelines : "This project > is > not currently set up to run pipelines." Hrm, but upstream doesn't seem to > ship > any test infrastructure, so I guess that would be non-trivial...) Hmm yes, there are no tests, but we could at least include the deb build infrastructure. I've done so for now by setting up pipelines via <https://salsa.debian.org/rohieb/mutt-ical/-/blob/debian/latest/debian/salsa-ci.yml>. Cheers, - Roland > Anyway: Once your package is lintian-clean I'd be happy to upload it. > > HTH! > > Bye, > > Joost -- Roland Hieber, Pengutronix e.K. | [email protected] | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
