You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Rodent of Unusual Size <Ke...@Golux.Com> on 1998/02/22 05:17:33 UTC
Re: cvs commit: apache-1.3/src/modules/standard mod_status.c
jim@hyperreal.org wrote:
>
> Add SERVER_VERSION and SERVER_BUILT info to the mod_status Status report
>
> Changes with Apache 1.3b6
>
> + *) Have the mod_status Status page report server version and
> + built data [M.D. Parker <md...@netcom.com>] PR#1448
> +
Erm, didn't Alexei -1 this because of issues with dynamically-loaded
modules? Something that may have been specific to Win32 at the time,
but mod_so probably makes it applicable to the UNIX framework as well.
#ken P-)}
Ken Coar <http://Web.Golux.Com/coar/>
Apache Group member <http://www.apache.org/>
"Apache Server for Dummies" <http://WWW.Dummies.Com/
Re: cvs commit: apache-1.3/src/modules/standard mod_status.c
Posted by Martin Kraemer <Ma...@mch.sni.de>.
On Sun, Feb 22, 1998 at 01:07:50PM +0000, Paul Sutton wrote:
> Perhaps there should be an API function to return the
> core build and version information instead? I'm +1 on the idea of
> reporting this information accurately.
+1.
BTW: Nobody has complained about mod_info reporting the SERVER_VERSION
and SERVER_BUILT !? Was that because mod_info didn't work on Win** yet?
Martin
--
| S I E M E N S | <Ma...@mch.sni.de> | Siemens Nixdorf
| ------------- | Voice: +49-89-636-46021 | Informationssysteme AG
| N I X D O R F | FAX: +49-89-636-44994 | 81730 Munich, Germany
~~~~~~~~~~~~~~~~My opinions only, of course; pgp key available on request
Re: cvs commit: apache-1.3/src/modules/standard mod_status.c
Posted by Alexei Kosut <ak...@leland.Stanford.EDU>.
On Sat, 21 Feb 1998, Rodent of Unusual Size wrote:
> jim@hyperreal.org wrote:
> >
> > Add SERVER_VERSION and SERVER_BUILT info to the mod_status Status report
> >
> > Changes with Apache 1.3b6
> >
> > + *) Have the mod_status Status page report server version and
> > + built data [M.D. Parker <md...@netcom.com>] PR#1448
> > +
>
> Erm, didn't Alexei -1 this because of issues with dynamically-loaded
> modules? Something that may have been specific to Win32 at the time,
> but mod_so probably makes it applicable to the UNIX framework as well.
Yes, I believe I did.
-- Alexei Kosut <ak...@stanford.edu> <http://www.stanford.edu/~akosut/>
Stanford University, Class of 2001 * Apache <http://www.apache.org> *
Re: cvs commit: apache-1.3/src/modules/standard mod_status.c
Posted by Dean Gaudet <dg...@arctic.org>.
On Sun, 22 Feb 1998, Alexei Kosut wrote:
> Erm... no. Unless dlopen() is much more magical than I think it is, if I
> compile mod_status with "#define SERVER_VERSION Apache/1.3", and then load
> that module using dlopen() under Apache 2.7, the code inside mod_status
> will still say "Apache/1.3", regardless of what the actual server is.
>
> Note that SERVER_BUILT is a global variable (I don't know why it's all
> caps, it confuses me), and I never vetoed its inclusion. Only
> SERVER_VERSION.
Fine then, SERVER_BUILT is what I'm talking about.
Dean
Re: cvs commit: apache-1.3/src/modules/standard mod_status.c
Posted by Alexei Kosut <ak...@leland.Stanford.EDU>.
On Sun, 22 Feb 1998, Dean Gaudet wrote:
> On Sat, 21 Feb 1998, Rodent of Unusual Size wrote:
>
> > jim@hyperreal.org wrote:
> > >
> > > Add SERVER_VERSION and SERVER_BUILT info to the mod_status Status report
> > >
> > > Changes with Apache 1.3b6
> > >
> > > + *) Have the mod_status Status page report server version and
> > > + built data [M.D. Parker <md...@netcom.com>] PR#1448
> > > +
> >
> > Erm, didn't Alexei -1 this because of issues with dynamically-loaded
> > modules? Something that may have been specific to Win32 at the time,
> > but mod_so probably makes it applicable to the UNIX framework as well.
>
> No, unix dlopen() is way more sane than that. This works without any
> changes (assuming our httpd executable is linked with -rdynamic or the
> equivalent thereof).
Erm... no. Unless dlopen() is much more magical than I think it is, if I
compile mod_status with "#define SERVER_VERSION Apache/1.3", and then load
that module using dlopen() under Apache 2.7, the code inside mod_status
will still say "Apache/1.3", regardless of what the actual server is.
Note that SERVER_BUILT is a global variable (I don't know why it's all
caps, it confuses me), and I never vetoed its inclusion. Only
SERVER_VERSION.
-- Alexei Kosut <ak...@stanford.edu> <http://www.stanford.edu/~akosut/>
Stanford University, Class of 2001 * Apache <http://www.apache.org> *
Re: cvs commit: apache-1.3/src/modules/standard mod_status.c
Posted by Dean Gaudet <dg...@arctic.org>.
On Sat, 21 Feb 1998, Rodent of Unusual Size wrote:
> jim@hyperreal.org wrote:
> >
> > Add SERVER_VERSION and SERVER_BUILT info to the mod_status Status report
> >
> > Changes with Apache 1.3b6
> >
> > + *) Have the mod_status Status page report server version and
> > + built data [M.D. Parker <md...@netcom.com>] PR#1448
> > +
>
> Erm, didn't Alexei -1 this because of issues with dynamically-loaded
> modules? Something that may have been specific to Win32 at the time,
> but mod_so probably makes it applicable to the UNIX framework as well.
No, unix dlopen() is way more sane than that. This works without any
changes (assuming our httpd executable is linked with -rdynamic or the
equivalent thereof).
Dean
Re: cvs commit: apache-1.3/src/modules/standard mod_status.c
Posted by Paul Sutton <pa...@c2.net>.
On Sat, 21 Feb 1998, Rodent of Unusual Size wrote:
> jim@hyperreal.org wrote:
> >
> > Add SERVER_VERSION and SERVER_BUILT info to the mod_status Status report
> >
> > Changes with Apache 1.3b6
> >
> > + *) Have the mod_status Status page report server version and
> > + built data [M.D. Parker <md...@netcom.com>] PR#1448
> > +
>
> Erm, didn't Alexei -1 this because of issues with dynamically-loaded
> modules? Something that may have been specific to Win32 at the time,
> but mod_so probably makes it applicable to the UNIX framework as well.
Yep. I'm also -1. Perhaps there should be an API function to return the
core build and version information instead? I'm +1 on the idea of
reporting this information accurately.
//pcs