diff mbox

OVS-ODP: better instructions to build ODP

Message ID 1420641637-23484-1-git-send-email-ciprian.barbu@linaro.org
State New
Headers show

Commit Message

Ciprian Barbu Jan. 7, 2015, 2:40 p.m. UTC
Signed-off-by: Ciprian Barbu <ciprian.barbu@linaro.org>
---
 INSTALL.ODP.md | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

Comments

Zoltan Kiss Jan. 7, 2015, 6:52 p.m. UTC | #1
Acked-by: Zoltan Kiss <zoltan.kiss@linaro.org>

On 07/01/15 14:40, Ciprian Barbu wrote:
> Signed-off-by: Ciprian Barbu <ciprian.barbu@linaro.org>
> ---
>   INSTALL.ODP.md | 8 +++-----
>   1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/INSTALL.ODP.md b/INSTALL.ODP.md
> index 23c0fbf..2cc543c 100644
> --- a/INSTALL.ODP.md
> +++ b/INSTALL.ODP.md
> @@ -26,21 +26,19 @@ Get the code:
>       ./bootstrap.sh
>
>   it is recommended to disable building shared library because then launching
> -ovs with sudo becomes a real pain
> -    ./configure --enable-debug --enable-shared=no
> +ovs with sudo becomes a real pain; you can install ODP anywhere you like
> +    ./configure --enable-debug --enable-shared=no --prefix=<odp_install>
>
>   to get debug symbols you can
>       ./configure --enable-debug CFLAGS="-g -O0" --enable-shared=no
>
>       make
> -
> -optionally:
>       make install
>
>   OVS:
>   =============
>       ./boot.sh
> -    ./configure --with-odp=<ODP_DIR>
> +    ./configure --with-odp=<odp_install>
>
>   to specify a different ODP platform you can use:
>       ./configure --with-odp=<ODP_DIR> --with-odp-platform=<platform>
>
Ciprian Barbu Jan. 8, 2015, 12:15 p.m. UTC | #2
Merged!

On Wed, Jan 7, 2015 at 8:52 PM, Zoltan Kiss <zoltan.kiss@linaro.org> wrote:
> Acked-by: Zoltan Kiss <zoltan.kiss@linaro.org>
>
>
> On 07/01/15 14:40, Ciprian Barbu wrote:
>>
>> Signed-off-by: Ciprian Barbu <ciprian.barbu@linaro.org>
>> ---
>>   INSTALL.ODP.md | 8 +++-----
>>   1 file changed, 3 insertions(+), 5 deletions(-)
>>
>> diff --git a/INSTALL.ODP.md b/INSTALL.ODP.md
>> index 23c0fbf..2cc543c 100644
>> --- a/INSTALL.ODP.md
>> +++ b/INSTALL.ODP.md
>> @@ -26,21 +26,19 @@ Get the code:
>>       ./bootstrap.sh
>>
>>   it is recommended to disable building shared library because then
>> launching
>> -ovs with sudo becomes a real pain
>> -    ./configure --enable-debug --enable-shared=no
>> +ovs with sudo becomes a real pain; you can install ODP anywhere you like
>> +    ./configure --enable-debug --enable-shared=no --prefix=<odp_install>
>>
>>   to get debug symbols you can
>>       ./configure --enable-debug CFLAGS="-g -O0" --enable-shared=no
>>
>>       make
>> -
>> -optionally:
>>       make install
>>
>>   OVS:
>>   =============
>>       ./boot.sh
>> -    ./configure --with-odp=<ODP_DIR>
>> +    ./configure --with-odp=<odp_install>
>>
>>   to specify a different ODP platform you can use:
>>       ./configure --with-odp=<ODP_DIR> --with-odp-platform=<platform>
>>
>
diff mbox

Patch

diff --git a/INSTALL.ODP.md b/INSTALL.ODP.md
index 23c0fbf..2cc543c 100644
--- a/INSTALL.ODP.md
+++ b/INSTALL.ODP.md
@@ -26,21 +26,19 @@  Get the code:
     ./bootstrap.sh
 
 it is recommended to disable building shared library because then launching
-ovs with sudo becomes a real pain
-    ./configure --enable-debug --enable-shared=no
+ovs with sudo becomes a real pain; you can install ODP anywhere you like
+    ./configure --enable-debug --enable-shared=no --prefix=<odp_install>
 
 to get debug symbols you can
     ./configure --enable-debug CFLAGS="-g -O0" --enable-shared=no
 
     make
-
-optionally:
     make install
 
 OVS:
 =============
     ./boot.sh
-    ./configure --with-odp=<ODP_DIR>
+    ./configure --with-odp=<odp_install>
 
 to specify a different ODP platform you can use:
     ./configure --with-odp=<ODP_DIR> --with-odp-platform=<platform>