[Buildroot] netcat returns

[Buildroot] netcat returns

[iks hefem]

hi, could you include netcat to your great buildroot distro ?
A patch was submitted here :
http://busybox.net/lists/buildroot/2006-November/000440.html (with a small
problem : the last endif in the .mk file is not added properly due to a bad
"patched patch file".)

To answer the question : why not busybox ? Well , because is more powerful :
udp mode by example.

thanks !
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://busybox.net/lists/buildroot/attachments/20070117/3db76309/attachment.html 

[Buildroot] netcat returns

[Bernhard Fischer]

On Wed, Jan 17, 2007 at 12:27:12AM +0100, iks hefem wrote:
>hi, could you include netcat to your great buildroot distro ?
>A patch was submitted here :
>http://busybox.net/lists/buildroot/2006-November/000440.html (with a small
>problem : the last endif in the .mk file is not added properly due to a bad
>"patched patch file".)
>
>To answer the question : why not busybox ? Well , because is more powerful :
>udp mode by example.

Please make a complete svn diff --diff-cmd diff -x -rdup package/ >
~/buildroot.add-package-netcat.diff  and test it.

The patch you link to above does hardcode "tar xzf" which is not
acceptable. Fix it to do what all other packages do (NETCAT_ZCAT etc).
The CC_FOR_BUILD is nowadays already contained in the
TARGET_CONFIGURE_OPTS, so remove it.

Oh, and i, personally, would prefer to use the version that comes with
debian, since it seems to be more robust:
http://ftp.debian.org/debian/pool/main/n/netcat/

see
$ grep "debian/patch" package/*/*mk
on one possible way to support such packages that have debian patches
applied.

[Buildroot] netcat returns

[iks hefem]

> Please make a complete svn diff --diff-cmd diff -x -rdup package/ >
> ~/buildroot.add-package-netcat.diff  and test it.
>
> The patch you link to above

uh .. in fact I am a mere user of this patch and unfortunately I'm not
sure I have the sufficient knowledge to perform & maintain the full
implementation you propose. Maybe the original poster will notice ?

I understand this would be more work for you than to simply add this
patch and that you might not be too keen on doing  it.

So, I'll see if I can propose a full patch, either that way or
(ideally) busybox's nc to support UDP !

thanks for your answer.

[Buildroot] netcat returns

[iks hefem]

2007/1/17, Bernhard Fischer <rep.dot.nop@gmail.com>:
> Please make a complete svn diff --diff-cmd diff -x -rdup package/ >
> ~/buildroot.add-package-netcat.diff  and test it.
>
> The patch you link to above does hardcode "tar xzf" which is not
> acceptable. Fix it to do what all other packages do (NETCAT_ZCAT etc).
> The CC_FOR_BUILD is nowadays already contained in the
> TARGET_CONFIGURE_OPTS, so remove it.

OK, that was finally done (in fact, it wasn't that hard) & tested OK
with current SVN. I did as you said, put it in and optionnally hid it
if busybox option is enabled.

> Oh, and i, personally, would prefer to use the version that comes with
> debian

OK, I didn't went that far but I will see if I can submit such a patch.

thx,

   Ikse Hefem
-------------- next part --------------
A non-text attachment was scrubbed...
Name: buildroot.add-package-netcat.diff
Type: text/x-diff
Size: 3281 bytes
Desc: not available
Url : http://busybox.net/lists/buildroot/attachments/20070120/b04bb178/attachment.bin 

[Buildroot] netcat returns

[Bernhard Fischer]

On Sat, Jan 20, 2007 at 12:06:44AM +0100, iks hefem wrote:
>2007/1/17, Bernhard Fischer <rep.dot.nop@gmail.com>:
>>Please make a complete svn diff --diff-cmd diff -x -rdup package/ >
>>~/buildroot.add-package-netcat.diff  and test it.
>>
>>The patch you link to above does hardcode "tar xzf" which is not
>>acceptable. Fix it to do what all other packages do (NETCAT_ZCAT etc).
>>The CC_FOR_BUILD is nowadays already contained in the
>>TARGET_CONFIGURE_OPTS, so remove it.
>
>OK, that was finally done (in fact, it wasn't that hard) & tested OK
>with current SVN. I did as you said, put it in and optionnally hid it
>if busybox option is enabled.
>
>>Oh, and i, personally, would prefer to use the version that comes with
>>debian
>
>OK, I didn't went that far but I will see if I can submit such a patch.

Please do.
I have applied it meanwhile. Thanks!