From: Stephen D. <sd...@gm...> - 2006-09-07 18:32:20
|
Looks like nsstats got moved to the contrib directory. Contrib is generally a bad idea because it becomes a dumping ground for random stuff. Two files in there, nsstats.tcl and nsconfig.tcl, need to be turned into modules. They are in fact modules, but kind of crappy ones because there's no README file, no Makefile to place them in the correct position, etc. Modules aren't second class citizens. Code in a module is not intended to imply it is lesser quality, or not supported, or whatever. The only difference is the *advantage* that the release schedule for a module is not tied to that of the server. (The name of nsconfig should probably be something less generic). Also, the list of mime types in contrib is confusing (it confused one new list member at least) and now redundant. All mime types are compiled in, and there's a new command [ns_info mimetypes] for introspection. Can this be removed? |
From: Vlad S. <vl...@cr...> - 2006-09-07 18:45:28
|
I agree overall, but i made them pages instead of tcl library and because page cannot reside under tcl/ they ended up in contrib. do we need special directory for index.adp and other pages, maybe pages/ in the CVS tree? Stephen Deasey wrote: > Looks like nsstats got moved to the contrib directory. Contrib is > generally a bad idea because it becomes a dumping ground for random > stuff. > > Two files in there, nsstats.tcl and nsconfig.tcl, need to be turned > into modules. They are in fact modules, but kind of crappy ones > because there's no README file, no Makefile to place them in the > correct position, etc. > > Modules aren't second class citizens. Code in a module is not intended > to imply it is lesser quality, or not supported, or whatever. The > only difference is the *advantage* that the release schedule for a > module is not tied to that of the server. > > (The name of nsconfig should probably be something less generic). > > Also, the list of mime types in contrib is confusing (it confused one > new list member at least) and now redundant. All mime types are > compiled in, and there's a new command [ns_info mimetypes] for > introspection. Can this be removed? > > ------------------------------------------------------------------------- > Using Tomcat but need to do more? Need to support web services, security? > Get stuff done quickly with pre-integrated technology to make your job easier > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > _______________________________________________ > naviserver-devel mailing list > nav...@li... > https://lists.sourceforge.net/lists/listinfo/naviserver-devel > -- Vlad Seryakov 571 262-8608 office vl...@cr... http://www.crystalballinc.com/vlad/ |
From: Stephen D. <sd...@gm...> - 2006-09-07 19:00:20
|
On 9/7/06, Vlad Seryakov <vl...@cr...> wrote: > I agree overall, but i made them pages instead of tcl library and > because page cannot reside under tcl/ they ended up in contrib. > do we need special directory for index.adp and other pages, maybe pages/ > in the CVS tree? There's only one page: index.adp. We can just put that in the root directory. nsstats.tcl is a module and should be in the modules/ directory of CVS. |
From: Vlad S. <vl...@cr...> - 2006-09-07 19:05:43
|
That was the reason i chnaged nsstats.tcl into page, not module, now you can put it anywhere under pages/ and point browser and use it instead of restarting server just or change something or enable/disable it. That was the main reason not to use it on existing production servers because it is a module Stephen Deasey wrote: > On 9/7/06, Vlad Seryakov <vl...@cr...> wrote: >> I agree overall, but i made them pages instead of tcl library and >> because page cannot reside under tcl/ they ended up in contrib. >> do we need special directory for index.adp and other pages, maybe pages/ >> in the CVS tree? > > > There's only one page: index.adp. We can just put that in the root directory. > > nsstats.tcl is a module and should be in the modules/ directory of CVS. > > ------------------------------------------------------------------------- > Using Tomcat but need to do more? Need to support web services, security? > Get stuff done quickly with pre-integrated technology to make your job easier > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > _______________________________________________ > naviserver-devel mailing list > nav...@li... > https://lists.sourceforge.net/lists/listinfo/naviserver-devel > -- Vlad Seryakov 571 262-8608 office vl...@cr... http://www.crystalballinc.com/vlad/ |
From: Stephen D. <sd...@gm...> - 2006-09-07 19:16:42
|
On 9/7/06, Vlad Seryakov <vl...@cr...> wrote: > That was the reason i chnaged nsstats.tcl into page, not module, now you > can put it anywhere under pages/ and point browser and use it instead of > restarting server just or change something or enable/disable it. > > That was the main reason not to use it on existing production servers > because it is a module OK. Great reason. It's still a module -- a module with one page. It should live in the same place as all out other modules, have a README, an example-config.tcl, etc... |
From: Vlad S. <vl...@cr...> - 2006-09-07 19:21:07
|
Ok, good point PS: On another note, if nobody objects, can i run all source files and remove trailing spaces from every line, it is like garbage anyway and i will not disturb CVS anymore with too many changes:-))) ??? Stephen Deasey wrote: > On 9/7/06, Vlad Seryakov <vl...@cr...> wrote: >> That was the reason i chnaged nsstats.tcl into page, not module, now you >> can put it anywhere under pages/ and point browser and use it instead of >> restarting server just or change something or enable/disable it. >> >> That was the main reason not to use it on existing production servers >> because it is a module > > > OK. Great reason. It's still a module -- a module with one page. It > should live in the same place as all out other modules, have a README, > an example-config.tcl, etc... > > ------------------------------------------------------------------------- > Using Tomcat but need to do more? Need to support web services, security? > Get stuff done quickly with pre-integrated technology to make your job easier > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > _______________________________________________ > naviserver-devel mailing list > nav...@li... > https://lists.sourceforge.net/lists/listinfo/naviserver-devel > -- Vlad Seryakov 571 262-8608 office vl...@cr... http://www.crystalballinc.com/vlad/ |
From: Stephen D. <sd...@gm...> - 2006-09-07 19:32:25
|
How about we do this after the next release, but before we start making changes for the next one? That way we minimise the noise while we're working in stuff. On 9/7/06, Vlad Seryakov <vl...@cr...> wrote: > Ok, good point > > > PS: On another note, if nobody objects, can i run all source files and > remove trailing spaces from every line, it is like garbage anyway and i > will not disturb CVS anymore with too many changes:-))) ??? > > Stephen Deasey wrote: > > On 9/7/06, Vlad Seryakov <vl...@cr...> wrote: > >> That was the reason i chnaged nsstats.tcl into page, not module, now you > >> can put it anywhere under pages/ and point browser and use it instead of > >> restarting server just or change something or enable/disable it. > >> > >> That was the main reason not to use it on existing production servers > >> because it is a module > > > > > > OK. Great reason. It's still a module -- a module with one page. It > > should live in the same place as all out other modules, have a README, > > an example-config.tcl, etc... > > > > ------------------------------------------------------------------------- > > Using Tomcat but need to do more? Need to support web services, security? > > Get stuff done quickly with pre-integrated technology to make your job easier > > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo > > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > > _______________________________________________ > > naviserver-devel mailing list > > nav...@li... > > https://lists.sourceforge.net/lists/listinfo/naviserver-devel > > > > -- > Vlad Seryakov > 571 262-8608 office > vl...@cr... > http://www.crystalballinc.com/vlad/ > > > ------------------------------------------------------------------------- > Using Tomcat but need to do more? Need to support web services, security? > Get stuff done quickly with pre-integrated technology to make your job easier > Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo > http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 > _______________________________________________ > naviserver-devel mailing list > nav...@li... > https://lists.sourceforge.net/lists/listinfo/naviserver-devel > |