Re: RFS: stripclub - Online Comic Reader/Archiver
A few more minor problems:
* In interface.fl:163:
callback {system("xterm -e zless /usr/share/doc/stripclub/readme.txt.gz");}
You can't rely on xterm being available unless you depend upon it, and
even still, that's bad practice. Instead, you should use
/usr/bin/x-terminal-emulator.
* Code like stripclub:288:
snprintf(cmd, 1023, "rm -rf %s", GetCacheFileName("", false));
system(cmd);
is potentially dangerous, since what if %s is "some file /", without
the quotes? Yeah, from the code path, it looks like that wouldn't
happen under normal conditions, but still I'd avoid code like that
altogether.
* The CFLAGS and INSTALL_PROGRAM stuff in debian/rules is unused. You
should pass CFLAGS to the 'make' command, and just remove the
INSTALL_PROGRAM stuff since dh_strip does the right thing anyway.
Bah, I hate dh_make ...
* Finally, your debian/rules lacks the "binary-indep" target, which is a
policy violation. Quoting section 4.8:
Both the binary-arch and binary-indep targets must exist. If one of
them has nothing to do (which will always be the case if the source
generates only a single binary package, whether
architecture-dependent or not), it must still exist and must always
succeed.
So just add it and have it do nothing.
--
You win again, gravity!
Reply to: