| Submitter | Max Filippov |
|---|---|
| Date | Nov. 23, 2011, 9:40 p.m. |
| Message ID | <201111240140.08335.jcmvbkbc@gmail.com> |
| Download | mbox | patch |
| Permalink | /patch/127395/ |
| State | New |
| Headers | show |
Comments
On 23 November 2011 21:40, Max Filippov <jcmvbkbc@gmail.com> wrote: > --- a/configure > +++ b/configure > @@ -759,8 +759,6 @@ for opt do > ;; > --enable-opengl) opengl="yes" > ;; > - --*dir) > - ;; > --disable-rbd) rbd="no" > ;; > --enable-rbd) rbd="yes" Haha, nice catch. Incidentally I think that the proper fix for this should be to have a case statement that explicitly matches for the --thingydir options we accept, and drop the wildcard completely. -- PMM
Patch
diff --git a/configure b/configure index f033438..e5566c8 100755 --- a/configure +++ b/configure @@ -759,8 +759,6 @@ for opt do ;; --enable-opengl) opengl="yes" ;; - --*dir) - ;; --disable-rbd) rbd="no" ;; --enable-rbd) rbd="yes" @@ -783,6 +781,8 @@ for opt do ;; --disable-guest-agent) guest_agent="no" ;; + --*dir) + ;; *) echo "ERROR: unknown option $opt"; show_help="yes" ;; esac