On Wednesday 03 February 2016 12:05 AM, Ximin Luo wrote:
iv. Upload new packages twextpy and pg8000
I've just uploaded these. I fixed a few minor things before uploading, please check git
and review them. In general, I run "lintian -i -I --pedantic --color auto
xxx.changes" to catch things like that, and it's a good habit to get into.
I had created two entries in debian/changelog because I had already
tagged the first release version and did not want to delete the tags. I
have now, however, deleted the tags and recreated them. Like you said, I
think it is best to create tags, once the packages are uploaded.
Also, I always run lintian, but don't use the --pedantic option. Will
start using it hereafter.
As part of the fix to (k) earlier, I pushed another patch to a side branch of
calendarserver: [1]. You haven't applied it to debian/sid yet, but I think it may be
necessary. To test, you should try to add/remove events with non-ascii unicode characters
in them such as "ßßß". If it fails, then try the patch and if it works please
add it to git as well.
X
[1]
https://anonscm.debian.org/cgit/calendarserver/calendarserver.git/diff/debian/patches/unicode-fixes.patch?h=debian/_wip_sid&id=4476a73ff4df39baa297606e00e66241f371178c
I will see how I could test unicode characters. Also, I do not know the
impact of the change. Should I get it reviewed by upstream?
Thanks,
Rahul.