Re: Check for Fontforge with enable-double (1637) (issue 6484062)

2012-08-27 Thread John Mandereau
2012/8/27 : > On 2012/08/27 09:04:46, dak wrote: >> You are confusing grep -E with grep -e. -e just says that the next >> word is the regular expression to look for rather than an option, and it is >> required since the regular expression starts with dash itself. > > > Actually, one should just u

Re: Check for Fontforge with enable-double (1637) (issue 6484062)

2012-08-27 Thread dak
On 2012/08/27 09:04:46, dak wrote: On 2012/08/26 22:56:44, John Mandereau wrote: > 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

Re: Check for Fontforge with enable-double (1637) (issue 6484062)

2012-08-27 Thread dak
On 2012/08/26 22:56:44, John Mandereau wrote: 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

Re: Check for Fontforge with enable-double (1637) (issue 6484062)

2012-08-26 Thread john . mandereau
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. gr

Check for Fontforge with enable-double (1637) (issue 6484062)

2012-08-26 Thread graham
LGTM, I get the expected error from a stock ubuntu 12.04 (which does not include --enable-double). http://codereview.appspot.com/6484062/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel