On Wed, Sep 26, 2012 at 11:26:54PM -0400, Francois Mikus wrote: > Well, we should have the test suite use some names using the hyphen > character. That should tell us if we are ok. > > And manual testing for: notifications, graphite export, livestatus logs > and livestatus states, webUI, bp_rules, triggers > > Then that should cover a lot of ground. :-) > > If anyone wants to take a shot at it, that would be good.
Hmm, searching for the usage on the master branch yields: $ rgrep -l illegal_char . ./.hg/data/shinken/basemodule.py.i ./shinken/modules/graphite_broker.py ./shinken/modules/graphite_ui.py ./shinken/basemodule.py And basemodule.py contains only: ./shinken/basemodule.py: self.illegal_char = compile(r'[^\w-]') So I don't think the patch is touching the shinken core. Thanks Hermann -- Netzwerkadministration/Zentrale Dienste, Interdiziplinaeres Zentrum fuer wissenschaftliches Rechnen der Universitaet Heidelberg IWR; INF 368; 69120 Heidelberg; Tel: (06221)54-8236 Fax: -5224 Email: hermann.la...@iwr.uni-heidelberg.de ------------------------------------------------------------------------------ Everyone hates slow websites. So do we. Make your web apps faster with AppDynamics Download AppDynamics Lite for free today: http://ad.doubleclick.net/clk;258768047;13503038;j? http://info.appdynamics.com/FreeJavaPerformanceDownload.html _______________________________________________ Shinken-devel mailing list Shinken-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/shinken-devel
