On Thu, 16 Nov 2000 [EMAIL PROTECTED] wrote:
[...]
> cvs import apr
>
> This allows us to keep the history for APR with the actual code, which is
> where it belongs. This also allows us keep APR with Apache, which should
> take care of some of the harrier issues that came up when splitting
Looks OK with that suplicate line removed.
+1
david
- Original Message -
From: "Jeff Trawick" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Thursday, November 16, 2000 6:25 PM
Subject: [PATCH] change apr_connect() to take apr_sockaddr_t, add
apr_getaddrinfo()
> This started off as D
> > > > This would then give
> > > > apr_sockaddr_t *sa;
> > > > apr_gethostbyname(&sa, "hostname", pool);
> > > > apr_connect(sock, sa);
> > >
> > > We need to add more arguments: address family for sure and possibly
> > > flags. Look at getaddrinfo(). And do we want to add an option
Do a "cvs tag" in apache before removal and after removal, I think.
You probably already know to check the perms and ownership after you
cp, since repositories are so sensitive to that. Might want to save
an "ls -lR" of the original apache apr dir somewhere for reference,
before you do anything.
This is scheduled to happen early next week, and we still need to decide
how to go about doing this. Here is my thought on the matter.
1) On locus:
cp -R /home/cvs/apache-2.0/src/lib/apr /home/cvs/apr
2) On remote machine
cd apache-2.0/src/lib/
cvs rm -R apr
cv
This is a reminder to everybody who is developing code in the Apache web
server that we are rolling an alpha release this weekend. It was supposed
to be tomorrow, but I think we are likely to delay for a day or two. The
reasons for the delay are relatively small.
1) Doesn't compile on Windows.
I'm buried deep in 1.3.x for the better part of the afternoon.
By tommorow I'll be playing in 2.0 once again, be happy to fix it :-)
Besides, I need to get i18n building on Win32.
Bill
> -Original Message-
> From: Jeff Trawick [mailto:[EMAIL PROTECTED]
> Sent: Thursday, November 16, 2000
[EMAIL PROTECTED] writes:
> > +
> > +apr_status_t apr_getaddrinfo(apr_sockaddr_t **sa,
> > + const char *hostname,
> > + apr_int32_t family,
> > + apr_port_t port,
> > + apr_pool_t *p);
> +
> +apr_status_t apr_getaddrinfo(apr_sockaddr_t **sa,
> + const char *hostname,
> + apr_int32_t family,
> + apr_port_t port,
> + apr_pool_t *p);
Can/should we make the hostname, fam
(I sent this to [EMAIL PROTECTED] the first time... not helpful...)
I just committed some changes to win32's networkio.h and sockets.c and
unix's inet_ntop.c to get Win32 up to speed with some Unix changes I
committed this morning.
However, there are at least two steps which I am punting on:
1)
This started off as David Reid's patch which he posted this a.m. I
renamed his apr_gethostbyname() to apr_getaddrinfo(), added fields
family and port to apr_getaddrinfo(), made some fixes, and changed the
remaining callers of apr_connect() to use the new parameter list.
Changes to the currently-c
"David Reid" <[EMAIL PROTECTED]> writes:
> > > This would then give
> > > apr_sockaddr_t *sa;
> > > apr_gethostbyname(&sa, "hostname", pool);
> > > apr_connect(sock, sa);
> >
> > We need to add more arguments: address family for sure and possibly
> > flags. Look at getaddrinfo(). And
On Thu, 16 Nov 2000, David Reid wrote:
> FreeBSD doesn't use this piece of code, but shouldn't it be a pointer...
No, because we pass it as a pointer to getpwnam_r. This value is an
actual structure that we fill out using memory from the buf. Then a
pointer is returned to this exact structure
"David Reid" <[EMAIL PROTECTED]> writes:
> OK, I've done some work on this but am getting an error now
> Could not connect: Can't assign requested address (49)
I have it working on Linux now...
Changes to your patch or to other code to get it to work:
1) The existing apr_set_port() doesn't ge
"David Reid" <[EMAIL PROTECTED]> writes:
> [OK, I'm getting better at this Reply To All stuff...]
I still haven't gotten the hang of it :( Maybe I need to study the
GNUS manual to figure out how to avoid editing the cc line to keep me
out of it.
> > > 1. Modify connect to take an apr_sockaddr_
[OK, I'm getting better at this Reply To All stuff...]
> > 1. Modify connect to take an apr_sockaddr_t instead of a hostname...
This
> > is a simple change so I'll make it sooner rather than later.
>
> yes...
OK, I've done some work on this but am getting an error now
Could not connect: Can't as
+1 from me.
- Original Message -
From: "Jeff Trawick" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Thursday, November 16, 2000 1:25 PM
Subject: [PATCH] add apr_create_socket(), some additional IPv6 support
> This adds apr_inet_ntop(), apr_inet_pton(), apr_create_socket(), and
> fixe
FreeBSD doesn't use this piece of code, but shouldn't it be a pointer...
Index: user/unix/homedir.c
===
RCS file: /home/cvs/apache-2.0/src/lib/apr/user/unix/homedir.c,v
retrieving revision 1.1
diff -u -r1.1 homedir.c
--- user/unix/hom
This adds apr_inet_ntop(), apr_inet_pton(), apr_create_socket(), and
fixes apr_get_ipaddr() to be thread-safe and to support IPv6.
apr_create_socket() supports the AF_UNSPEC idiom outlined in my last
post which means get AF_INET6 if you can but fall back to AF_INET
because I don't really care.
co
Here are some idioms I see in other IPv6-enabled code which I think
should be supported in APR. Part of this has a bearing on my comments
to you post below.
1) stream socket server
a. (normal) get me a darn socket, I don't care which kind
API: apr_create_socket(, AF_UNSPEC,
OK so Jeff has convinced me about not needing apr_in_addr_t's. So I'm
going to work on removing the apr_in_addr_t from the code all together...
There are a lot of questions that start with ??? so answers from anyone with
a point of view please!
1. Modify connect to take an apr_sockaddr_t instea
On Wed, 15 Nov 2000, William A. Rowe, Jr. wrote:
> > From: [EMAIL PROTECTED]
> > [mailto:[EMAIL PROTECTED]
> > Sent: Wednesday, November 15, 2000 1:23 PM
> > To: [EMAIL PROTECTED]
> > Subject: failure notice
> >
> > Hi. This is the qmail-send program at locus.apache.org.
> > I'm afraid I wasn't ab
22 matches
Mail list logo
