ITP: inetsim
Hi GengYu,
I think the package is fine now, I only have a few small beautification
suggestions:
* You still have lintian informational tags
"package-contains-empty-directory" which you might want to get rid of.
* debian/inetsim.postrm has a trailing space (end of rm command).
* debian/rules: It's conventional to have an empty line between the
targets to improve readability.
* debian/inetsim.postinst: The adduser command has `--quiet` twice (I
think this was my mistake actually?) so you can remove one.
* debian/inetsim.{prerm,postinst}: `exit 0` at the end is not required
(and pretty useless in combination with `set -e`) so you can remove
it.
Good luck finding a sponsor!
Regards
Lukas
Reply to: