Re: RFS: acgvision-agent [2nd try]
I uploaded version 5.1.4-2 of my package. It can be found on mentors using :
I tryed to fix the errors you showed me.
Here are the details :
close it through debian/changelog.
I did, the number is debbugs #578719
The file debian/conffiles should not be needed. Please remove it.
Ok, i removed the file.
The binary package should not Depend: java-virtual-machine. Please remove it.
The binary package should not Depend on java2-runtime only. Please use
something like default-jre-headless | java2-runtime-headless -
whatever is correct for your package. Please ask if you are not sure.
I remove java-virtual-machine and added default-jre-headless | java2-runtime-headless
BTW, why do you Build-Depend: openjdk-6-jdk instead of default-jdk? Is
Openjdk really needed for building?
I replaced openjdk by the default-jdk.
The long description has a typo (communcate), misses a full stop and
is very short. Please be more verbose.
I wrote a new (and longer) description and add "." for the full stop. Not sure about what is "the full stop".
The copyright file is empty and wouldn't be accepted by the FTP team.
I created one
Creating /var/log/acgvision in postinst shouldn't be needed but...
... postrm should remove /var/log/acgvision-agent only when purging.
But wait: is it /var/log/acgvision or /var/log/acgvision-agent? You
have the first variant in debian/dirs.
I rewrite a new way to handle log removing. The default log directory is acgvision. The acgvision-core package (not uploaded yet) logs its file there too.
prerm is virtually empty and should be removed.
I removed it
The #DEBHELPER# token is missing in all three scripts.
I add it too.
You call ant directly without refering to $JAVA_HOME in debian/rules.
That is not good. Please check other packages that Build-Depend
java-helper for examples of better packaging practices. I have been
told that jsch might be such an example. And don't forget to read the
tutorial for java-helper.
I add the $JAVA_HOME.
I also fixed some errors torsten showed me with lintian :
I add a lintian error : acgvision-agent source: missing-debian-source-format.
- Add a manpage for acgvision-agent (Maybe I should write one for each script). I am not sure this page is well formed it is the first manpage i write.
- Renamed all scripts without .sh or .ksh extension using mv in debian/rules (thanks torsten for this tip). A bug has been open on the upstream acgvision site about this for the next release.
- Changed the homepage of acgvision to the new page on launchpad.
- add -e option to all maintainers scripts.
- added the force-reload option to the init.d script
But i dont understand how to play with this.
Could anyone download and review my package please?
Thank you for your help.
Synergie Park Des Bonnettes
2, rue Willy Brandt
Ligne directe : +33 (0)3 21 15 36 36
Email : firstname.lastname@example.org
On Fri, Apr 23, 2010 at 2:48 PM, Rémi Debay <email@example.com>
Many thanks for reviewing my package.
I fixed most of things you told me. But I have a question about a warning you gave me on lintian :
I renamed the file to acgvision-agent. But now debuild tells it ignores the new file.
I think i should include the change in the original source code (and the change is not debian specific) and so increment the version number from 5.1.4 to 5.1.5
Could anyone confirm me it is the right way to proceed or is there a cleaner way to proceed?