Dear Mechtilde, Thanks for updating the package. When I did my first Java package I seem to recall being told by someone that the pkg-java team doesn't follow the convention of adding +ds or +dfsg to the version string on the grounds that virtually all Java source packages contain binaries or other content that needs to be removed, so it is assumed that all pkg-java packages have been repacked. I've had a quick look in the policy and the wiki and I can't find anything in writing to that effect, so I'm hoping someone else on this list can advise if that's still the case?
In any case, adding +ds to the package has upset uscan: ``` uscan --download-current-version uscan warn: uscan warning: In debian/watch no matching refs for version 1.0.35+ds in watch line https://git.code.sf.net/p/csvjdbc/code/ refs/tags/(?:v|csvjdbc-)([\d\.-]+) ``` I think you need to add `dversionmangle=s/\+ds$//,` to the opts in `debian/watch`. When updating to compat 12, I'd suggest using the new debhelper-compat=12 build dependency form [1] I tend to make a change, use dch to add a line to `debian/changelog` stage with git and then use debcommit so that changelog entries are aligned with git commits. I'd also do things like changing the version in `debian/compat` and the version of debhelper in `debian/control` in the same commit. This may just be personal taste though. One thing that worried me about this package (also true of the version I uploaded) is that the tests throw up an error: ``` [ERROR] Errors: [ERROR] org.relique.jdbc.csv.TestCsvDriver.testTimestampFormatGermany(org.relique.jdbc.csv.TestCsvDriver) [ERROR] Run 1: TestCsvDriver.testTimestampFormatGermany:3869 » NullPointer [ERROR] Run 2: TestCsvDriver.testTimestampFormatGermany:3869 » NullPointer [INFO] [INFO] [ERROR] Tests run: 828, Failures: 0, Errors: 1, Skipped: 1 [INFO] [ERROR] There are test failures. ``` This doesn't seem to stop the package building, but I was unable to get to the bottom of it (I'm not much of a Java programmer). As you probably know, the source for the package is maintained here at [2] so you should add your commits and tags to master, upstream and pristine-tar there before uploading. I hope that helps? Christopher [1] https://nthykier.wordpress.com/2019/01/04/debhelper-compat-12-is-now-released/ [2] https://salsa.debian.org/java-team/csvjdbc On Mon, 9 Sep 2019 at 20:11, Mechtilde <o...@mechtilde.de> wrote: > > Hello > > now I have successfully built my first java package. It is the update > for the package libcsvjdbc-java from version 1.0.34 to version 1.0.35. > > I uploaded it to Salsa at https://salsa.debian.org/mechtilde/csvjdbc and > to p.d.o at https://people.debian.org/~mechtilde/CsvJdbc/. > > I hope someone can review it before I do an upload. > > This is one of the dependeny for JVerein (ITP:#929477) > > Kind regards > > -- > Mechtilde Stehmann > ## Apache OpenOffice > ## Freie Office Suite für Linux, MacOSX, Windows > ## Debian Developer > ## PGP encryption welcome > ## F0E3 7F3D C87A 4998 2899 39E7 F287 7BBA 141A AD7F >