All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] New recipe for Bullet Physics library
@ 2010-10-26  3:28 dbatzle
  2010-10-26  6:38 ` Frans Meulenbroeks
  0 siblings, 1 reply; 3+ messages in thread
From: dbatzle @ 2010-10-26  3:28 UTC (permalink / raw)
  To: openembedded-devel

Signed-off-by: dbatzle <dbatzle@dcbcyber.com>
---
 recipes/bullet/bullet.bb |   18 ++++++++++++++++++
 1 files changed, 18 insertions(+), 0 deletions(-)
 create mode 100644 recipes/bullet/bullet.bb

diff --git a/recipes/bullet/bullet.bb b/recipes/bullet/bullet.bb
new file mode 100644
index 0000000..d28734f
--- /dev/null
+++ b/recipes/bullet/bullet.bb
@@ -0,0 +1,18 @@
+DESCRIPTION = "Bullet Physics Library"
+LICENSE = "Zlib"
+
+PR = "r1"
+SRCREV="2218"
+PV="2.77+svnr${SRCPV}"
+
+#SRC_URI = "file:///home/David/gamestuff/bullet-2.75"
+SRC_URI = "svn://bullet.googlecode.com/svn;module=trunk;proto=http"
+
+#If you want to build the demos you will have to have xlib and glut installed.
+#Then set BUILD_DEMOS, BUILD_EXTRAS, and USE_GLUT to ON
+
+EXTRA_OECMAKE="-DBUILD_DEMOS:BOOL=OFF -DBUILD_EXTRAS:BOOL=OFF -DUSE_GLUT:BOOL=OFF -DBUILD_SHARED_LIBS:BOOL=ON"
+
+inherit cmake
+
+S = "${WORKDIR}/trunk"
-- 
1.6.0.6




^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] New recipe for Bullet Physics library
  2010-10-26  3:28 [PATCH] New recipe for Bullet Physics library dbatzle
@ 2010-10-26  6:38 ` Frans Meulenbroeks
  2010-10-26 13:35   ` Paul Menzel
  0 siblings, 1 reply; 3+ messages in thread
From: Frans Meulenbroeks @ 2010-10-26  6:38 UTC (permalink / raw)
  To: openembedded-devel

2010/10/26 dbatzle <dbatzle@dcbcyber.com>:
> Signed-off-by: dbatzle <dbatzle@dcbcyber.com>
> ---
>  recipes/bullet/bullet.bb |   18 ++++++++++++++++++
>  1 files changed, 18 insertions(+), 0 deletions(-)
>  create mode 100644 recipes/bullet/bullet.bb
>
> diff --git a/recipes/bullet/bullet.bb b/recipes/bullet/bullet.bb
> new file mode 100644
> index 0000000..d28734f
> --- /dev/null
> +++ b/recipes/bullet/bullet.bb
> @@ -0,0 +1,18 @@
> +DESCRIPTION = "Bullet Physics Library"
> +LICENSE = "Zlib"
> +
> +PR = "r1"

A recipe should start at r0.
Can you also add HOMEPAGE (and maybe some more of the header fields)

> +SRCREV="2218"
> +PV="2.77+svnr${SRCPV}"
> +
> +#SRC_URI = "file:///home/David/gamestuff/bullet-2.75"

suggest to remove the line above

> +SRC_URI = "svn://bullet.googlecode.com/svn;module=trunk;proto=http"
> +
> +#If you want to build the demos you will have to have xlib and glut installed.
> +#Then set BUILD_DEMOS, BUILD_EXTRAS, and USE_GLUT to ON
> +
> +EXTRA_OECMAKE="-DBUILD_DEMOS:BOOL=OFF -DBUILD_EXTRAS:BOOL=OFF -DUSE_GLUT:BOOL=OFF -DBUILD_SHARED_LIBS:BOOL=ON"
> +
> +inherit cmake
> +
> +S = "${WORKDIR}/trunk"
> --
> 1.6.0.6
>

Best regards, Frans



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] New recipe for Bullet Physics library
  2010-10-26  6:38 ` Frans Meulenbroeks
@ 2010-10-26 13:35   ` Paul Menzel
  0 siblings, 0 replies; 3+ messages in thread
From: Paul Menzel @ 2010-10-26 13:35 UTC (permalink / raw)
  To: openembedded-devel

[-- Attachment #1: Type: text/plain, Size: 1935 bytes --]

Am Dienstag, den 26.10.2010, 08:38 +0200 schrieb Frans Meulenbroeks:
> 2010/10/26 dbatzle <dbatzle@dcbcyber.com>:

The commit summary should start with the recipe name as suggested by the
commit policy [1].

> > Signed-off-by: dbatzle <dbatzle@dcbcyber.com>

I guess a real name would be preferred. But if you want to stay
anonymous that is alright too I guess.

> > ---
> >  recipes/bullet/bullet.bb |   18 ++++++++++++++++++
> >  1 files changed, 18 insertions(+), 0 deletions(-)
> >  create mode 100644 recipes/bullet/bullet.bb
> >
> > diff --git a/recipes/bullet/bullet.bb b/recipes/bullet/bullet.bb
> > new file mode 100644
> > index 0000000..d28734f
> > --- /dev/null
> > +++ b/recipes/bullet/bullet.bb
> > @@ -0,0 +1,18 @@
> > +DESCRIPTION = "Bullet Physics Library"
> > +LICENSE = "Zlib"
> > +
> > +PR = "r1"
> 
> A recipe should start at r0.
> Can you also add HOMEPAGE (and maybe some more of the header fields)
> 
> > +SRCREV="2218"
> > +PV="2.77+svnr${SRCPV}"
> > +
> > +#SRC_URI = "file:///home/David/gamestuff/bullet-2.75"
> 
> suggest to remove the line above
> 
> > +SRC_URI = "svn://bullet.googlecode.com/svn;module=trunk;proto=http"
> > +
> > +#If you want to build the demos you will have to have xlib and glut installed.
> > +#Then set BUILD_DEMOS, BUILD_EXTRAS, and USE_GLUT to ON
> > +
> > +EXTRA_OECMAKE="-DBUILD_DEMOS:BOOL=OFF -DBUILD_EXTRAS:BOOL=OFF -DUSE_GLUT:BOOL=OFF -DBUILD_SHARED_LIBS:BOOL=ON"
> > +
> > +inherit cmake
> > +
> > +S = "${WORKDIR}/trunk"
> > --
> > 1.6.0.6

I agree with Frans’ suggestions. You can also run `oe-stylize.py` [2] if
you have not done so already.

Please do not forget to update the patch queue [3].


Thanks,

Paul


[1] http://wiki.openembedded.org/index.php/Commit_Policy
[2] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/contrib/oe-stylize.py
[3] http://wiki.openembedded.org/index.php/Patchwork

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-10-26 13:36 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-10-26  3:28 [PATCH] New recipe for Bullet Physics library dbatzle
2010-10-26  6:38 ` Frans Meulenbroeks
2010-10-26 13:35   ` Paul Menzel

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.