Bug#658835: RFS: aspsms-t (try 4) [NEW] -- sms transport for your xmpp/jabber server
tag 658835 + moreinfo
thanks
Hi,
I have no intention of sponsoring this package, but as I did take a
brief look at it before:
| my $config_file = "/home/jabber/aspsms-t/etc/aspsms-web-notify.xml";
Not really useful for programs in /usr/bin.
| use lib "./";
Not really useful for programs in /usr/bin.
Code like
eval { unlink($passfile); }; if($@) { handle_error(); }
will not catch any errors, see
perl -E 'eval { unlink "/tmp/does-not-exist"; }; if ($@) { say "There was an error." }'
Code that is not or inconsistently indented is hard to read.
Regards,
Ansgar
Reply to: