Message ID | 1416953519-53534-1-git-send-email-mike.holmes@linaro.org |
---|---|
State | Accepted |
Commit | 20922d9d5bac04fd88231b2b21467f720c66071c |
Headers | show |
On Wed, Nov 26, 2014 at 12:11 AM, Mike Holmes <mike.holmes@linaro.org> wrote: > Signed-off-by: Mike Holmes <mike.holmes@linaro.org> Just a couple of nits, otherwise you can have my Reviewed-by > --- > > Add t to ar.bz2 > > DEPENDENCIES | 18 ++++++++++++++---- > 1 file changed, 14 insertions(+), 4 deletions(-) > > diff --git a/DEPENDENCIES b/DEPENDENCIES > index f70a1d4..bc39889 100644 > --- a/DEPENDENCIES > +++ b/DEPENDENCIES > @@ -78,23 +78,33 @@ Prerequisites for building the OpenDataPlane (ODP) API > > 4.0 Packages needed to build API tests > > - Cunit test framework > + Cunit test framework version 2.1-3 is required > Cunit prvodes a framework to run the API test suite that proves conformance to the > ODP API. The home page http://cunit.sourceforge.net/doc/introduction.html > > 4.1 Native Cunit install > > - # Debian/Ubuntu > + # Debian/Ubuntu check it is 2.1-3 > $ apt-get install libcunit1-dev > > -4.2 Cross compile of Cunit > +4.2 Built from src > + > + export CUNIT_VERSION=2.1-3 > + curl -sSOL http://sourceforge.net/projects/cunit/files/CUnit/${CUNIT_VERSION}/CUnit-${CUNIT_VERSION}.tar.bz2 > + tar -jxf *.bz2 I would suggest changing this to CUnit*.bz2. For example I have a dir with package source archives. If I use your command it may extract 100 archives instead of just one. > + cd CUnit* > + ./bootstrap > + make install > + #In Step 4.4 use --with-cunit-path=/home/<your name>/CUnitHome nit: although it's to be changed, you should avoid using spaces inside a path. Instead use /home/<your_name> or better /home/<user> > + > +4.3 Cross compile of Cunit > > $ git svn clone http://svn.code.sf.net/p/cunit/code/trunk cunit-code > $ cd cunit-code > $ ./bootstrap > $ ./configure --host=arm-linux-gnueabihf --prefix=/home/<user>/src/install-cunit > > -4.3 Using Cunit with ODP > +4.4 Using Cunit with ODP > $ Add the configuration option to the regular configuration options > ./configure --enable-cunit #if cunit is in the PATH > ./configure --with-cunit-path=DIR #only if you need a path to Cunit libs and headers > -- > 2.1.0 > > > _______________________________________________ > lng-odp mailing list > lng-odp@lists.linaro.org > http://lists.linaro.org/mailman/listinfo/lng-odp
On Wed, Nov 26, 2014 at 12:29 PM, Ciprian Barbu <ciprian.barbu@linaro.org> wrote: > On Wed, Nov 26, 2014 at 12:11 AM, Mike Holmes <mike.holmes@linaro.org> wrote: >> Signed-off-by: Mike Holmes <mike.holmes@linaro.org> > > Just a couple of nits, otherwise you can have my Reviewed-by > >> --- >> >> Add t to ar.bz2 >> >> DEPENDENCIES | 18 ++++++++++++++---- >> 1 file changed, 14 insertions(+), 4 deletions(-) >> >> diff --git a/DEPENDENCIES b/DEPENDENCIES >> index f70a1d4..bc39889 100644 >> --- a/DEPENDENCIES >> +++ b/DEPENDENCIES >> @@ -78,23 +78,33 @@ Prerequisites for building the OpenDataPlane (ODP) API >> >> 4.0 Packages needed to build API tests >> >> - Cunit test framework >> + Cunit test framework version 2.1-3 is required >> Cunit prvodes a framework to run the API test suite that proves conformance to the >> ODP API. The home page http://cunit.sourceforge.net/doc/introduction.html >> >> 4.1 Native Cunit install >> >> - # Debian/Ubuntu >> + # Debian/Ubuntu check it is 2.1-3 >> $ apt-get install libcunit1-dev >> >> -4.2 Cross compile of Cunit >> +4.2 Built from src >> + >> + export CUNIT_VERSION=2.1-3 >> + curl -sSOL http://sourceforge.net/projects/cunit/files/CUnit/${CUNIT_VERSION}/CUnit-${CUNIT_VERSION}.tar.bz2 >> + tar -jxf *.bz2 > > I would suggest changing this to CUnit*.bz2. For example I have a dir > with package source archives. If I use your command it may extract 100 > archives instead of just one. > >> + cd CUnit* >> + ./bootstrap >> + make install >> + #In Step 4.4 use --with-cunit-path=/home/<your name>/CUnitHome > > nit: although it's to be changed, you should avoid using spaces inside > a path. Instead use /home/<your_name> or better /home/<user> Or even better $HOME/CUnitHome > >> + >> +4.3 Cross compile of Cunit >> >> $ git svn clone http://svn.code.sf.net/p/cunit/code/trunk cunit-code >> $ cd cunit-code >> $ ./bootstrap >> $ ./configure --host=arm-linux-gnueabihf --prefix=/home/<user>/src/install-cunit >> >> -4.3 Using Cunit with ODP >> +4.4 Using Cunit with ODP >> $ Add the configuration option to the regular configuration options >> ./configure --enable-cunit #if cunit is in the PATH >> ./configure --with-cunit-path=DIR #only if you need a path to Cunit libs and headers >> -- >> 2.1.0 >> >> >> _______________________________________________ >> lng-odp mailing list >> lng-odp@lists.linaro.org >> http://lists.linaro.org/mailman/listinfo/lng-odp
diff --git a/DEPENDENCIES b/DEPENDENCIES index f70a1d4..bc39889 100644 --- a/DEPENDENCIES +++ b/DEPENDENCIES @@ -78,23 +78,33 @@ Prerequisites for building the OpenDataPlane (ODP) API 4.0 Packages needed to build API tests - Cunit test framework + Cunit test framework version 2.1-3 is required Cunit prvodes a framework to run the API test suite that proves conformance to the ODP API. The home page http://cunit.sourceforge.net/doc/introduction.html 4.1 Native Cunit install - # Debian/Ubuntu + # Debian/Ubuntu check it is 2.1-3 $ apt-get install libcunit1-dev -4.2 Cross compile of Cunit +4.2 Built from src + + export CUNIT_VERSION=2.1-3 + curl -sSOL http://sourceforge.net/projects/cunit/files/CUnit/${CUNIT_VERSION}/CUnit-${CUNIT_VERSION}.tar.bz2 + tar -jxf *.bz2 + cd CUnit* + ./bootstrap + make install + #In Step 4.4 use --with-cunit-path=/home/<your name>/CUnitHome + +4.3 Cross compile of Cunit $ git svn clone http://svn.code.sf.net/p/cunit/code/trunk cunit-code $ cd cunit-code $ ./bootstrap $ ./configure --host=arm-linux-gnueabihf --prefix=/home/<user>/src/install-cunit -4.3 Using Cunit with ODP +4.4 Using Cunit with ODP $ Add the configuration option to the regular configuration options ./configure --enable-cunit #if cunit is in the PATH ./configure --with-cunit-path=DIR #only if you need a path to Cunit libs and headers
Signed-off-by: Mike Holmes <mike.holmes@linaro.org> --- Add t to ar.bz2 DEPENDENCIES | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-)