[PATCH] lwip translator: Remove unused reference to config.h

2018-05-02 Thread Joan Lledó
--- lwip/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/lwip/Makefile b/lwip/Makefile index c5f3c8cb..d0ddfd38 100644 --- a/lwip/Makefile +++ b/lwip/Makefile @@ -47,4 +47,3 @@ iioctl-MIGSFLAGS = -imacros $(srcdir)/mig-mutate.h # cpp doesn't automatically make dependencies for

Re: [PATCH] LwIP translator

2017-12-19 Thread Joan Lledó
Hi, 2017-12-19 19:21 GMT+01:00 Samuel Thibault : > Is it not actually valid for posix systems in general? (except the > cond_wait thing which I patched over) That's what the port/ folder contains: - lwipopts.h is for configuring the stack parameters - cc.h is for

Re: [PATCH] LwIP translator

2017-12-19 Thread Svante Signell
On Tue, 2017-12-19 at 19:21 +0100, Samuel Thibault wrote: > Joan Lledó, on mar. 19 déc. 2017 19:11:40 +0100, wrote: > > - The patches in debian/patches are not enough to make the server > > work, for instance, we're calling lwip_poll() which is not in 2.0.3 > > nor any patch in the package. I

Re: [PATCH] LwIP translator

2017-12-19 Thread Samuel Thibault
Joan Lledó, on mar. 19 déc. 2017 19:11:40 +0100, wrote: > 2017-12-18 20:03 GMT+01:00 Samuel Thibault : > > I am talking about the files you created in your repository, they are > > hold a GPL copyright notice. I guess at some point we will want to see > > them maintained

Re: [PATCH] LwIP translator

2017-12-19 Thread Joan Lledó
Hi Samuel, 2017-12-18 20:03 GMT+01:00 Samuel Thibault : > I am talking about the files you created in your repository, they are > hold a GPL copyright notice. I guess at some point we will want to see > them maintained within lwip, and thus BSD-licensed. > I've been

Re: [PATCH] LwIP translator

2017-12-18 Thread Joan Lledó
2017-12-18 17:28 GMT+01:00 Samuel Thibault : > BTW, your lwip code should probably be made BSD-licensed, just like the > rest of lwip. Mmm, I think I don't know what you mean, if I send patches to a BSD-licensed project, they are BSD too, right?

Re: [PATCH] LwIP translator

2017-12-18 Thread James Clarke
On 18 Dec 2017, at 17:09, Samuel Thibault wrote: > James Clarke, on lun. 18 déc. 2017 17:06:28 +, wrote: >> On 18 Dec 2017, at 16:28, Samuel Thibault wrote: >>> Joan Lledó, on lun. 18 déc. 2017 17:10:42 +0100, wrote: 2017-12-18 14:46

Re: [PATCH] LwIP translator

2017-12-18 Thread Samuel Thibault
James Clarke, on lun. 18 déc. 2017 17:06:28 +, wrote: > On 18 Dec 2017, at 16:28, Samuel Thibault wrote: > > Joan Lledó, on lun. 18 déc. 2017 17:10:42 +0100, wrote: > >> 2017-12-18 14:46 GMT+01:00 Samuel Thibault : > >>> We need to know what

Re: [PATCH] LwIP translator

2017-12-18 Thread James Clarke
On 18 Dec 2017, at 16:28, Samuel Thibault wrote: > Joan Lledó, on lun. 18 déc. 2017 17:10:42 +0100, wrote: >> 2017-12-18 14:46 GMT+01:00 Samuel Thibault : >>> We need to know what is not yet in upstream, what will >>> eventually get to upstream,

Re: [PATCH] LwIP translator

2017-12-18 Thread Samuel Thibault
Joan Lledó, on lun. 18 déc. 2017 17:10:42 +0100, wrote: > 2017-12-18 14:46 GMT+01:00 Samuel Thibault : > > We need to know what is not yet in upstream, what will > > eventually get to upstream, and what may not get to > > upstream. > > There're also some patches that are

Re: [PATCH] LwIP translator

2017-12-18 Thread Joan Lledó
2017-12-18 14:46 GMT+01:00 Samuel Thibault : > Ok, but I don't see that base in your repository, the first commit > already has a lot of source code which is not in 2.0.1. Oh, yes. That's because I first imported all lwip 2.0.1 code to my lwip-hurd repository and made

Re: [PATCH] LwIP translator

2017-12-18 Thread Samuel Thibault
Hello, Joan Lledó, on lun. 18 déc. 2017 12:58:35 +0100, wrote: > 2017-12-18 2:32 GMT+01:00 Samuel Thibault : > > Or are you actually based on another lwip? > > Yes, It's based in the version 2.0.1, as I started to work on this > back in February. Ok, but I don't see

Re: [PATCH] LwIP translator

2017-12-18 Thread Joan Lledó
Hello, 2017-12-18 2:32 GMT+01:00 Samuel Thibault : > Or are you actually based on another lwip? Yes, It's based in the version 2.0.1, as I started to work on this back in February. I plan to upgrade liblwip to the last version in a few months.

Re: [PATCH] LwIP translator

2017-12-17 Thread Samuel Thibault
Hello, I'm however having a hard time matching your g...@github.com:jlledom/liblwip-hurd.git repository with the upstream https://git.savannah.nongnu.org/git/lwip.git repository. Since the plan is to just pick up upstream as much as possible, we need to keep the delta small. Or are you actually

Re: [PATCH] LwIP translator

2017-12-17 Thread Samuel Thibault
Hello, At last, found the time to commit this :) Huge hanks! Samuel

Re: [PATCH] LwIP translator

2017-11-12 Thread Samuel Thibault
Joan Lledó, on lun. 13 nov. 2017 08:31:45 +0100, wrote: > After working on the perl tests fails, I can say they are due to bugs in > lwip, not in our server. I've already sent the bug reports. Heh, nice :D Samuel

[PATCH] LwIP translator

2017-11-12 Thread Joan Lledó
--- Makefile | 4 + config.make.in | 7 + configure.ac | 5 + lwip/Makefile| 52 +++ lwip/config.h| 5 + lwip/iioctl-ops.c| 412

[PATCH] LwIP translator

2017-11-12 Thread Joan Lledó
After working on the perl tests fails, I can say they are due to bugs in lwip, not in our server. I've already sent the bug reports. The attached patch includes all required changes in previous emails.

Re: [PATCH] LwIP translator

2017-10-29 Thread Samuel Thibault
Joan Lledó, on dim. 29 oct. 2017 11:46:20 +0100, wrote: > the perl testsuite indeed found some bugs. So I'll send the new patch > when I have them fixed. Heh :) Thanks! Samuel

Re: [PATCH] LwIP translator

2017-10-29 Thread Joan Lledó
2017-10-22 17:25 GMT+02:00 Samuel Thibault : >> >> Unfortunately, lwip doesn't provide a lwip_ppoll() function. > > Ah, I saw it in lwipv6, I thought it was generally available. > > That being said, it'd be better to make the code already use struct > timespec, converting

Re: [PATCH] LwIP translator

2017-10-22 Thread Samuel Thibault
Hello, Joan Lledó, on mer. 18 oct. 2017 09:24:22 +0200, wrote: > >> +static error_t > >> +lwip_io_select_common (struct sock_user *user, > >> +mach_port_t reply, > >> +mach_msg_type_name_t reply_type, > >> +struct timeval *tv, int

Re: [PATCH] LwIP translator

2017-10-18 Thread Samuel Thibault
Joan Lledó, on mer. 18 oct. 2017 09:24:22 +0200, wrote: > > Last but not least, did you try to run the glibc testsuite while running > > this TCP/IP stack? It would probably find potential bugs. Also, the > > perl testsuite is probably a nice thing to run. > > I need help here. I've been

Re: [PATCH] LwIP translator

2017-10-18 Thread Joan Lledó
Hello Samuel, Thanks for taking your time reviewing my code. 2017-09-27 0:44 GMT+02:00 Samuel Thibault : >> +error_t >> +lwip_S_io_write (struct sock_user * user, >> + char *data, >> + size_t datalen, >> + off_t offset,

Re: [PATCH] LwIP translator

2017-09-26 Thread Samuel Thibault
Samuel Thibault, on mer. 27 sept. 2017 00:44:15 +0200, wrote: > Last but not least, did you try to run the glibc testsuite while running > this TCP/IP stack? It would probably find potential bugs. Also, the > perl testsuite is probably a nice thing to run. (I'm not saying I won't commit this

Re: [PATCH] LwIP translator

2017-09-26 Thread Samuel Thibault
Hello, Joan Lledó, on mer. 06 sept. 2017 10:09:36 +0200, wrote: > --- /dev/null > +++ b/lwip/iioctl-ops.c > +/* Get the interface from its name */ > +static struct netif * > +get_if (char *name) ... > + > + netif = netif_list; > + while (netif != 0) > +{ > + if (strcmp

[PATCH] LwIP translator

2017-09-06 Thread Joan Lledó
--- Makefile | 4 + config.make.in | 7 + configure.ac | 5 + lwip/Makefile| 51 +++ lwip/config.h| 5 + lwip/iioctl-ops.c| 418

[PATCH] LwIP translator

2017-09-06 Thread Joan Lledó
This patch includes some refactoring in the device modules to remove borrowed code from lwip.

[PATCH] LwIP translator

2017-08-30 Thread joanlluislledo
From: Joan Lledó --- Makefile | 4 + config.make.in | 7 + configure.ac | 5 + lwip/Makefile| 51 +++ lwip/config.h| 5 + lwip/iioctl-ops.c

[PATCH] LwIP translator

2017-08-30 Thread joanlluislledo
This patch incorporates Justus changes in the lwip Makefile

Re: [PATCH] LwIP translator

2017-08-25 Thread Joan Lledó
>> # Whether we found libgcrypt. >> HAVE_LIBGCRYPT = @HAVE_LIBGCRYPT@ >> >> +# Whether we found libgcrypt. > > Copy-paste error :) > Ops!, yes... you've got a sharp eye!

Re: [PATCH] LwIP translator

2017-08-24 Thread James Clarke
Hi Joan, On 24 Aug 2017, at 18:49, Joan Lledó wrote: > ... > diff --git a/config.make.in b/config.make.in > index dfbf0c1..f74220e 100644 > --- a/config.make.in > +++ b/config.make.in > @@ -99,6 +99,13 @@ HAVE_SUN_RPC = @HAVE_SUN_RPC@ > # Whether we found libgcrypt. >

[PATCH] LwIP translator

2017-08-24 Thread Joan Lledó
--- Makefile | 4 + config.make.in | 7 + configure.ac | 5 + lwip/Makefile| 51 +++ lwip/config.h| 5 + lwip/iioctl-ops.c| 418

[PATCH] LwIP translator

2017-08-24 Thread Joan Lledó
I made some changes to conditionally build the lwip translator depending on whether liblwip is installed.

[PATCH] LwIP translator

2017-08-23 Thread Joan Lledó
--- Makefile | 1 + lwip/Makefile| 51 +++ lwip/config.h| 5 + lwip/iioctl-ops.c| 418 lwip/io-ops.c| 560 ++ lwip/lwip-hurd.h

[PATCH] LwIP translator

2017-08-23 Thread Joan Lledó
Hello, This is the LwIP translator I wrote during my GSoC. The translator uses LwIP as a regular library and therefore the stack sources are not included in the patch. The lwip external libary may be downloaded from [1] or cloned from [2]. Regards, Joan -- [1]