Re: [ptxdist] [PATCH] tntnet: Add host package

["Bernhard Seßler" <bernhard.sessler@corscience.de>]

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