mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCH] tntnet: Add host package
@ 2014-02-05 12:42 Bernhard Sessler
  2014-02-11 11:02 ` Michael Olbrich
  2014-02-12  8:55 ` Michael Olbrich
  0 siblings, 2 replies; 8+ messages in thread
From: Bernhard Sessler @ 2014-02-05 12:42 UTC (permalink / raw)
  To: ptxdist

This host package builds the tntnet SDK, which is required in order
to generate the pre-compiled C++ modules.

Signed-off-by: Bernhard Sessler <bernhard.sessler@corscience.de>
---
 rules/host-tntnet.in   |  9 +++++++++
 rules/host-tntnet.make | 44 ++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 53 insertions(+)
 create mode 100644 rules/host-tntnet.in
 create mode 100644 rules/host-tntnet.make

diff --git a/rules/host-tntnet.in b/rules/host-tntnet.in
new file mode 100644
index 0000000..dafa4bf
--- /dev/null
+++ b/rules/host-tntnet.in
@@ -0,0 +1,9 @@
+## SECTION=hosttools_noprompt
+## SECTION=hosttools_platform
+
+config HOST_TNTNET
+	tristate
+	default ALLYES
+	select HOST_ZLIB
+	help
+	  web application server for c++
diff --git a/rules/host-tntnet.make b/rules/host-tntnet.make
new file mode 100644
index 0000000..53aee1a
--- /dev/null
+++ b/rules/host-tntnet.make
@@ -0,0 +1,44 @@
+# -*-makefile-*-
+#
+# Copyright (C) 2014 by Bernhard Seßler <bernhard.sessler@corscience.de>
+#
+# See CREDITS for details about who has contributed to this project.
+#
+# For further information about the PTXdist project and license conditions
+# see the README file.
+#
+
+#
+# We provide this package
+#
+HOST_PACKAGES-$(PTXCONF_HOST_TNTNET) += host-tntnet
+
+#
+# Paths and names
+#
+HOST_TNTNET_VERSION	:= 2.2.1
+HOST_TNTNET_MD5		:= febe799675c1b8b2f7259bad30cf6f23
+HOST_TNTNET		:= tntnet-$(HOST_TNTNET_VERSION)
+HOST_TNTNET_SUFFIX	:= tar.gz
+HOST_TNTNET_URL		:= http://www.tntnet.org/download/$(HOST_TNTNET).$(HOST_TNTNET_SUFFIX)
+HOST_TNTNET_SOURCE	:= $(SRCDIR)/$(HOST_TNTNET).$(HOST_TNTNET_SUFFIX)
+HOST_TNTNET_DIR		:= $(HOST_BUILDDIR)/$(HOST_TNTNET)
+
+# ----------------------------------------------------------------------------
+# Prepare
+# ----------------------------------------------------------------------------
+
+HOST_TNTNET_CONF_TOOL	:= autoconf
+HOST_TNTNET_CONF_OPT	:= \
+	$(HOST_AUTOCONF) \
+	--disable-dependency-tracking \
+	--disable-unittest \
+	--with-demos=no \
+	--with-epoll=no \
+	--with-sdk=yes \
+	--with-sendfile=no \
+	--with-server=no \
+	--with-ssl=no \
+	--with-stressjob=no
+
+# vim: syntax=make
-- 
1.8.3.2


-- 
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [PATCH] tntnet: Add host package
  2014-02-05 12:42 [ptxdist] [PATCH] tntnet: Add host package Bernhard Sessler
@ 2014-02-11 11:02 ` Michael Olbrich
  2014-02-11 11:23   ` Juergen Beisert
  2014-02-18  7:13   ` Bernhard Seßler
  2014-02-12  8:55 ` Michael Olbrich
  1 sibling, 2 replies; 8+ messages in thread
From: Michael Olbrich @ 2014-02-11 11:02 UTC (permalink / raw)
  To: ptxdist

On Wed, Feb 05, 2014 at 01:42:45PM +0100, Bernhard Sessler wrote:
> This host package builds the tntnet SDK, which is required in order
> to generate the pre-compiled C++ modules.

So this will be used to build an application that uses tntnet, right?

Btw this won't compile without cxxtools, so you need to add a host package
for that as well.

Michael

> Signed-off-by: Bernhard Sessler <bernhard.sessler@corscience.de>
> ---
>  rules/host-tntnet.in   |  9 +++++++++
>  rules/host-tntnet.make | 44 ++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 53 insertions(+)
>  create mode 100644 rules/host-tntnet.in
>  create mode 100644 rules/host-tntnet.make
> 
> diff --git a/rules/host-tntnet.in b/rules/host-tntnet.in
> new file mode 100644
> index 0000000..dafa4bf
> --- /dev/null
> +++ b/rules/host-tntnet.in
> @@ -0,0 +1,9 @@
> +## SECTION=hosttools_noprompt
> +## SECTION=hosttools_platform

no need to add hosttools_platform. That is only used for packages that are
also selected in the platformconfig

> +
> +config HOST_TNTNET
> +	tristate
> +	default ALLYES
> +	select HOST_ZLIB
> +	help
> +	  web application server for c++

You can drop the help here. It's not visible anyways.

> diff --git a/rules/host-tntnet.make b/rules/host-tntnet.make
> new file mode 100644
> index 0000000..53aee1a
> --- /dev/null
> +++ b/rules/host-tntnet.make
> @@ -0,0 +1,44 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2014 by Bernhard Seßler <bernhard.sessler@corscience.de>
> +#
> +# See CREDITS for details about who has contributed to this project.
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +HOST_PACKAGES-$(PTXCONF_HOST_TNTNET) += host-tntnet
> +
> +#
> +# Paths and names
> +#
> +HOST_TNTNET_VERSION	:= 2.2.1
> +HOST_TNTNET_MD5		:= febe799675c1b8b2f7259bad30cf6f23
> +HOST_TNTNET		:= tntnet-$(HOST_TNTNET_VERSION)
> +HOST_TNTNET_SUFFIX	:= tar.gz
> +HOST_TNTNET_URL		:= http://www.tntnet.org/download/$(HOST_TNTNET).$(HOST_TNTNET_SUFFIX)
> +HOST_TNTNET_SOURCE	:= $(SRCDIR)/$(HOST_TNTNET).$(HOST_TNTNET_SUFFIX)
> +HOST_TNTNET_DIR		:= $(HOST_BUILDDIR)/$(HOST_TNTNET)


This should not be necessary. Host packages define those from the
corresponding target package. You should only need:

HOST_TNTNET_DIR		:= $(HOST_BUILDDIR)/$(TNTNET)

Michael

> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +HOST_TNTNET_CONF_TOOL	:= autoconf
> +HOST_TNTNET_CONF_OPT	:= \
> +	$(HOST_AUTOCONF) \
> +	--disable-dependency-tracking \
> +	--disable-unittest \
> +	--with-demos=no \
> +	--with-epoll=no \
> +	--with-sdk=yes \
> +	--with-sendfile=no \
> +	--with-server=no \
> +	--with-ssl=no \
> +	--with-stressjob=no
> +
> +# vim: syntax=make
> -- 
> 1.8.3.2
> 
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [PATCH] tntnet: Add host package
  2014-02-11 11:02 ` Michael Olbrich
@ 2014-02-11 11:23   ` Juergen Beisert
  2014-02-11 11:32     ` Michael Olbrich
  2014-02-18  7:13   ` Bernhard Seßler
  1 sibling, 1 reply; 8+ messages in thread
From: Juergen Beisert @ 2014-02-11 11:23 UTC (permalink / raw)
  To: ptxdist; +Cc: Michael Olbrich

On Tuesday 11 February 2014 12:02:33 Michael Olbrich wrote:
> [...]
> > +
> > +config HOST_TNTNET
> > +	tristate
> > +	default ALLYES
> > +	select HOST_ZLIB
> > +	help
> > +	  web application server for c++
>
> You can drop the help here. It's not visible anyways.

Some people tend to add comments into their C-code which is also not visible 
after the compiler has done its job. But for the readers of the C-code these 
comments are still helpful.

jbe

-- 
Pengutronix e.K.                              | Juergen Beisert             |
Linux Solutions for Science and Industry      | http://www.pengutronix.de/  |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [PATCH] tntnet: Add host package
  2014-02-11 11:23   ` Juergen Beisert
@ 2014-02-11 11:32     ` Michael Olbrich
  2014-02-17  6:57       ` Bernhard Seßler
  0 siblings, 1 reply; 8+ messages in thread
From: Michael Olbrich @ 2014-02-11 11:32 UTC (permalink / raw)
  To: ptxdist

On Tue, Feb 11, 2014 at 12:23:21PM +0100, Juergen Beisert wrote:
> On Tuesday 11 February 2014 12:02:33 Michael Olbrich wrote:
> > [...]
> > > +
> > > +config HOST_TNTNET
> > > +	tristate
> > > +	default ALLYES
> > > +	select HOST_ZLIB
> > > +	help
> > > +	  web application server for c++
> >
> > You can drop the help here. It's not visible anyways.
> 
> Some people tend to add comments into their C-code which is also not visible 
> after the compiler has done its job. But for the readers of the C-code these 
> comments are still helpful.

The target package has the same comment, so it's just duplicated. And it's
not exactly correct. The host package is just for code generation.

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [PATCH] tntnet: Add host package
  2014-02-05 12:42 [ptxdist] [PATCH] tntnet: Add host package Bernhard Sessler
  2014-02-11 11:02 ` Michael Olbrich
@ 2014-02-12  8:55 ` Michael Olbrich
  2014-02-12  8:57   ` Michael Olbrich
  1 sibling, 1 reply; 8+ messages in thread
From: Michael Olbrich @ 2014-02-12  8:55 UTC (permalink / raw)
  To: ptxdist

On Wed, Feb 05, 2014 at 01:42:45PM +0100, Bernhard Sessler wrote:
> This host package builds the tntnet SDK, which is required in order
> to generate the pre-compiled C++ modules.
> 
> Signed-off-by: Bernhard Sessler <bernhard.sessler@corscience.de>

Thanks, applied.

Michael

> ---
>  rules/host-tntnet.in   |  9 +++++++++
>  rules/host-tntnet.make | 44 ++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 53 insertions(+)
>  create mode 100644 rules/host-tntnet.in
>  create mode 100644 rules/host-tntnet.make
> 
> diff --git a/rules/host-tntnet.in b/rules/host-tntnet.in
> new file mode 100644
> index 0000000..dafa4bf
> --- /dev/null
> +++ b/rules/host-tntnet.in
> @@ -0,0 +1,9 @@
> +## SECTION=hosttools_noprompt
> +## SECTION=hosttools_platform
> +
> +config HOST_TNTNET
> +	tristate
> +	default ALLYES
> +	select HOST_ZLIB
> +	help
> +	  web application server for c++
> diff --git a/rules/host-tntnet.make b/rules/host-tntnet.make
> new file mode 100644
> index 0000000..53aee1a
> --- /dev/null
> +++ b/rules/host-tntnet.make
> @@ -0,0 +1,44 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2014 by Bernhard Seßler <bernhard.sessler@corscience.de>
> +#
> +# See CREDITS for details about who has contributed to this project.
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +HOST_PACKAGES-$(PTXCONF_HOST_TNTNET) += host-tntnet
> +
> +#
> +# Paths and names
> +#
> +HOST_TNTNET_VERSION	:= 2.2.1
> +HOST_TNTNET_MD5		:= febe799675c1b8b2f7259bad30cf6f23
> +HOST_TNTNET		:= tntnet-$(HOST_TNTNET_VERSION)
> +HOST_TNTNET_SUFFIX	:= tar.gz
> +HOST_TNTNET_URL		:= http://www.tntnet.org/download/$(HOST_TNTNET).$(HOST_TNTNET_SUFFIX)
> +HOST_TNTNET_SOURCE	:= $(SRCDIR)/$(HOST_TNTNET).$(HOST_TNTNET_SUFFIX)
> +HOST_TNTNET_DIR		:= $(HOST_BUILDDIR)/$(HOST_TNTNET)
> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +HOST_TNTNET_CONF_TOOL	:= autoconf
> +HOST_TNTNET_CONF_OPT	:= \
> +	$(HOST_AUTOCONF) \
> +	--disable-dependency-tracking \
> +	--disable-unittest \
> +	--with-demos=no \
> +	--with-epoll=no \
> +	--with-sdk=yes \
> +	--with-sendfile=no \
> +	--with-server=no \
> +	--with-ssl=no \
> +	--with-stressjob=no
> +
> +# vim: syntax=make
> -- 
> 1.8.3.2
> 
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [PATCH] tntnet: Add host package
  2014-02-12  8:55 ` Michael Olbrich
@ 2014-02-12  8:57   ` Michael Olbrich
  0 siblings, 0 replies; 8+ messages in thread
From: Michael Olbrich @ 2014-02-12  8:57 UTC (permalink / raw)
  To: ptxdist

On Wed, Feb 12, 2014 at 09:55:36AM +0100, Michael Olbrich wrote:
> On Wed, Feb 05, 2014 at 01:42:45PM +0100, Bernhard Sessler wrote:
> > This host package builds the tntnet SDK, which is required in order
> > to generate the pre-compiled C++ modules.
> > 
> > Signed-off-by: Bernhard Sessler <bernhard.sessler@corscience.de>
> 
> Thanks, applied.

Sorry, not this one. I replied to the wrong thread...
See my other comments about this one.

Michael

> > ---
> >  rules/host-tntnet.in   |  9 +++++++++
> >  rules/host-tntnet.make | 44 ++++++++++++++++++++++++++++++++++++++++++++
> >  2 files changed, 53 insertions(+)
> >  create mode 100644 rules/host-tntnet.in
> >  create mode 100644 rules/host-tntnet.make
> > 
> > diff --git a/rules/host-tntnet.in b/rules/host-tntnet.in
> > new file mode 100644
> > index 0000000..dafa4bf
> > --- /dev/null
> > +++ b/rules/host-tntnet.in
> > @@ -0,0 +1,9 @@
> > +## SECTION=hosttools_noprompt
> > +## SECTION=hosttools_platform
> > +
> > +config HOST_TNTNET
> > +	tristate
> > +	default ALLYES
> > +	select HOST_ZLIB
> > +	help
> > +	  web application server for c++
> > diff --git a/rules/host-tntnet.make b/rules/host-tntnet.make
> > new file mode 100644
> > index 0000000..53aee1a
> > --- /dev/null
> > +++ b/rules/host-tntnet.make
> > @@ -0,0 +1,44 @@
> > +# -*-makefile-*-
> > +#
> > +# Copyright (C) 2014 by Bernhard Seßler <bernhard.sessler@corscience.de>
> > +#
> > +# See CREDITS for details about who has contributed to this project.
> > +#
> > +# For further information about the PTXdist project and license conditions
> > +# see the README file.
> > +#
> > +
> > +#
> > +# We provide this package
> > +#
> > +HOST_PACKAGES-$(PTXCONF_HOST_TNTNET) += host-tntnet
> > +
> > +#
> > +# Paths and names
> > +#
> > +HOST_TNTNET_VERSION	:= 2.2.1
> > +HOST_TNTNET_MD5		:= febe799675c1b8b2f7259bad30cf6f23
> > +HOST_TNTNET		:= tntnet-$(HOST_TNTNET_VERSION)
> > +HOST_TNTNET_SUFFIX	:= tar.gz
> > +HOST_TNTNET_URL		:= http://www.tntnet.org/download/$(HOST_TNTNET).$(HOST_TNTNET_SUFFIX)
> > +HOST_TNTNET_SOURCE	:= $(SRCDIR)/$(HOST_TNTNET).$(HOST_TNTNET_SUFFIX)
> > +HOST_TNTNET_DIR		:= $(HOST_BUILDDIR)/$(HOST_TNTNET)
> > +
> > +# ----------------------------------------------------------------------------
> > +# Prepare
> > +# ----------------------------------------------------------------------------
> > +
> > +HOST_TNTNET_CONF_TOOL	:= autoconf
> > +HOST_TNTNET_CONF_OPT	:= \
> > +	$(HOST_AUTOCONF) \
> > +	--disable-dependency-tracking \
> > +	--disable-unittest \
> > +	--with-demos=no \
> > +	--with-epoll=no \
> > +	--with-sdk=yes \
> > +	--with-sendfile=no \
> > +	--with-server=no \
> > +	--with-ssl=no \
> > +	--with-stressjob=no
> > +
> > +# vim: syntax=make
> > -- 
> > 1.8.3.2
> > 
> > 
> > -- 
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> 
> -- 
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [PATCH] tntnet: Add host package
  2014-02-11 11:32     ` Michael Olbrich
@ 2014-02-17  6:57       ` Bernhard Seßler
  0 siblings, 0 replies; 8+ messages in thread
From: Bernhard Seßler @ 2014-02-17  6:57 UTC (permalink / raw)
  To: ptxdist

Hi folks,

sorry, I've been on vacation last week and I was not thinking about IT related
stuff even once. ;)

On 11.02.2014 12:32, Michael Olbrich wrote:
> On Tue, Feb 11, 2014 at 12:23:21PM +0100, Juergen Beisert wrote:
>> On Tuesday 11 February 2014 12:02:33 Michael Olbrich wrote:
>> > [...]
>> > > +
>> > > +config HOST_TNTNET
>> > > +	tristate
>> > > +	default ALLYES
>> > > +	select HOST_ZLIB
>> > > +	help
>> > > +	  web application server for c++
>> >
>> > You can drop the help here. It's not visible anyways.
>> 
>> Some people tend to add comments into their C-code which is also not visible 
>> after the compiler has done its job. But for the readers of the C-code these 
>> comments are still helpful.
> 
> The target package has the same comment, so it's just duplicated. And it's
> not exactly correct. The host package is just for code generation.
> 
> Michael
> 

I tend to agree to Michael, I will drop the help text (as already stated it's
not visible anywhere and one might also look up the help text in the target
package). And I will add a dependency to a host cxxtools package (don't know why
I've not seen this myself).

Thanks for the hints.


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [PATCH] tntnet: Add host package
  2014-02-11 11:02 ` Michael Olbrich
  2014-02-11 11:23   ` Juergen Beisert
@ 2014-02-18  7:13   ` Bernhard Seßler
  1 sibling, 0 replies; 8+ messages in thread
From: Bernhard Seßler @ 2014-02-18  7:13 UTC (permalink / raw)
  To: ptxdist


On 11.02.2014 12:02, Michael Olbrich wrote:
> On Wed, Feb 05, 2014 at 01:42:45PM +0100, Bernhard Sessler wrote:
>> This host package builds the tntnet SDK, which is required in order
>> to generate the pre-compiled C++ modules.
> 
> So this will be used to build an application that uses tntnet, right?

Correct. Applications using Tntnet need to make use of its ecpp compiler, which
generates pure C++ code out of the ecpp files that come with each project.
That's somehow just like Qt's meta object compiler.

> 
> Btw this won't compile without cxxtools, so you need to add a host package
> for that as well.

Argh, seems that I had tntnet installed on my host machine when I was testing
this stuff. A new patchset is on the way.

> 
> Michael
> 

<snip>

Regards,
Bernhard


-- 
ptxdist mailing list
ptxdist@pengutronix.de

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

end of thread, other threads:[~2014-02-18  7:14 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-05 12:42 [ptxdist] [PATCH] tntnet: Add host package Bernhard Sessler
2014-02-11 11:02 ` Michael Olbrich
2014-02-11 11:23   ` Juergen Beisert
2014-02-11 11:32     ` Michael Olbrich
2014-02-17  6:57       ` Bernhard Seßler
2014-02-18  7:13   ` Bernhard Seßler
2014-02-12  8:55 ` Michael Olbrich
2014-02-12  8:57   ` Michael Olbrich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox