diff options
| author | Manish Goregaokar <manishsmail@gmail.com> | 2015-03-06 16:25:43 +0530 |
|---|---|---|
| committer | Manish Goregaokar <manishsmail@gmail.com> | 2015-03-06 22:22:40 +0530 |
| commit | 7ed4660a51b048f8b166b298d1777544d3a0fd98 (patch) | |
| tree | 5c5fd8504ecfc7c02cda03bfe2dc583149d7dbfe | |
| parent | 61c6b199bcc23fe931f3fda2f968442256e07dab (diff) | |
| parent | 4349fa4756c4a7051bf83e5499f8b3876ad7c918 (diff) | |
| download | rust-7ed4660a51b048f8b166b298d1777544d3a0fd98.tar.gz rust-7ed4660a51b048f8b166b298d1777544d3a0fd98.zip | |
Rollup merge of #22474 - iKevinY:pandoc-version-check, r=brson
Executing `configure` seems to create the following error due to how the script [parses Pandoc's version](https://github.com/rust-lang/rust/blob/master/configure#L705): ```text ./configure: line 705: [: pandoc: integer expression expected ./configure: line 705: [: 1.12.4.2: integer expression expected ``` This issue seems to stem from a discrepancy between BSD and GNU versions of sed. This patch changes the sed command to use an extended regex, which works with both flavours of sed.
| -rwxr-xr-x | configure | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/configure b/configure index e5333b45525..af500a056f1 100755 --- a/configure +++ b/configure @@ -697,15 +697,17 @@ probe CFG_ADB adb if [ ! -z "$CFG_PANDOC" ] then - PV_MAJOR_MINOR=$(pandoc --version | grep '^pandoc\(.exe\)\? ' | - # extract the first 2 version fields, ignore everything else - sed 's/pandoc\(.exe\)\? \([0-9]*\)\.\([0-9]*\).*/\2 \3/') + PV_MAJOR_MINOR=$(pandoc --version | grep '^pandoc' | + # Extract "MAJOR MINOR" from Pandoc's version number + sed -E 's/pandoc(.exe)? ([0-9]+)\.([0-9]+).*/\2 \3/') MIN_PV_MAJOR="1" MIN_PV_MINOR="9" + # these patterns are shell globs, *not* regexps PV_MAJOR=${PV_MAJOR_MINOR% *} PV_MINOR=${PV_MAJOR_MINOR#* } + if [ "$PV_MAJOR" -lt "$MIN_PV_MAJOR" ] || [ "$PV_MINOR" -lt "$MIN_PV_MINOR" ] then step_msg "pandoc $PV_MAJOR.$PV_MINOR is too old. Need at least $MIN_PV_MAJOR.$MIN_PV_MINOR. Disabling" |
