[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

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: