On Fri, Aug 01, 2008 at 10:33:14PM +0100, Matthew Johnson wrote: > On Mon Jul 28 20:12, Iustin Pop wrote: > > Hello, > > > > I'm trying to package protobuf (http://code.google.com/p/protobuf/). > > The source package will generate C++, Python and Java bindings. Since > > I'm not knowledgeable with Java, and even less with Java packaging, > > could someone please look over the Java bits? > > Hi, sorry I've not had a chance to look over this until now. I've had a > look at the Java bits and I have a couple of comments. Thanks for taking the time. > > - upstream uses maven to build, but since this is not really needed > > and maven resources are not available in Debian we just use a custom > > ant script > > This works, although I've recently added javahelper to the archive which > has jh_build for exactly these situations. Ah, I see. I'll keep that in mind for future versions. > > - openjdk-6 and the sun-jdk 5 and 6 are the only jdks that can build > > the package, due the fact that the java.util.regex in classpath is > > missing some functions (already implemented upstream, I think, but > > not yet in sid's gcj); since openjdk works, this still allows the > > package to be in main, right? > > Java policy is that you should pick one JDK to build with and not allow > other alternatives so that you get repeatable results. Allowing > alternatives to run with is fine. So you need to just depend on openjdk > and change the ant wrapper to just use that. Ok, fixed. This makes sense indeed. > Also (speaking of dependencies), library packages shouldn't depend on > JDKs, only applications which use those libraries should, so you can > remove the dependencies unless you also provide a program with it. Thanks, fixed. > > - I'm not really sure about the naming of the (actual) jars in > > /usr/share/java... right now I have protobuf-java-2.0.0beta.jar: is > > the -java part needed? A few other jars in there have such a part > > I would say not. Good, changed. > The rest of the Java stuff looks reasonable, although as a general point > you should remove the commented out dh_ commands. I assume you'll have > checked that it's pbuilder and lintian clean. Yep. Well, I still have on lintian 'info' message (no-symbols-control-file, but from what I read on the debian wiki, it's better to only start providing this file once the library is stable, and protobuf is not yet there) for each of the two shared libraries, but nothing else. I've uploaded a new version to mentors (http://mentors.debian.net/debian/pool/main/p/protobuf/protobuf_2.0.0~b-2.dsc) with these and a few other small pending changes. Resuming the search for a sponsor... :) Thanks a lot for your feedback! iustin
Attachment:
signature.asc
Description: Digital signature