comments on callrec
Timo Juhani Lindfors
timo.lindfors at iki.fi
Tue Apr 7 12:37:34 CEST 2009
Hi,
Tom <tom at stosb.com> writes:
> please make sure to send me bug reports/ideas/patches.
I did not test this yet but here are my comments based on reading the
source tarball:
* Thanks for using autotools. It helps a lot that simple autoreconf -i
&& ./configure && make && make install works and DESTDIR and
--prefix are respected.
* Code has comments, way to go! :-)
* #define GLADE_FILE "/usr/share/callrec/callrec.glade" is hardcoded
to source code and does not respect --prefix. This means I can not
install callrec as normal user to my home directory.
* There's a stray \r in the end of callrec.c
* Please consider making the state.patch with "diff -u" so that it has
context.
19,20c19,20
< value.0 0
< value.1 0
---
> value.0 195
> value.1 195
is not very readable since you don't know what control it is
changing.
* There are three state files but only one patch. Which statefile is
the patch for? If you have time it would be nice to have some
discussion on what each control does. Does this program record both
parties of the conversation in the same channel or are they recorded
as different channels?
* The only license to distribute gartoon redux icon theme 1.7 is GPL
version 2:
"This package is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation; version 2 dated June, 1991."
This might be a problem for your GPLv3 or later application. However,
note that version 1.8 gives you a license to distribute it under later
versions too:
"* Change license to GPL v2 or later."
best regards,
Timo Lindfors
More information about the community
mailing list