LGTM except one nitpick (see comment).

http://codereview.appspot.com/6484062/diff/1/configure.in
File configure.in (right):

http://codereview.appspot.com/6484062/diff/1/configure.in#newcode174
configure.in:174: if $FONTFORGE --version 2>&1|egrep -q -e '-L?D\.?$';
then
It seems that egrep vs. grep and egrep/grep -e flag are redundant, why
not just have "grep -q -e" instead?

http://codereview.appspot.com/6484062/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to