[dev.icinga.com #2558] warn users to read Changelog after install #952
Comments
Updated by mfriedrich on 2012-04-26 12:37:50 +00:00 example on rhel
|
Updated by mfriedrich on 2012-04-26 12:43:12 +00:00
|
Updated by Tommi on 2012-04-26 19:50:03 +00:00 I would like to vote to rename or drop of the existing idomod.o, after 2 times i ran into trouble while i forgot to remove the broker entry from icinga.cfg or modules dir and therefore had to old idomod still activ. This resultes in very nice effects. When renaming/dropping a clear errormessage appears, if the user forgot to change his configs and we can help immediate |
Updated by mfriedrich on 2012-04-26 20:28:07 +00:00 rm'ing something (as source install) where i don't exactly where it was installed before, is dangerous. i had that in mind, but source is something for users and their attention imho - i don't want to be blamed when the rm does something wrong. and it will not work for packages - it will try to rm something in buildroot which is not there. i'd rather let the users paste their log output in case of problems. as well as the docs entry on upgrades. as well as already sent - a note to the mailinglists. last resort will be a core check when parsing the config and trying to load the neb module - if the name matches idomod.o spit a warning to syslog. |
Updated by mfriedrich on 2012-04-26 20:35:01 +00:00 base/nebmods.c
strcmp or strstr on the mod->filename, bailing out if "idomod.o" thoughts? |
Updated by mfriedrich on 2012-04-27 16:50:20 +00:00 a strstr is possible. given that idomod does not export its version, nor a function is there as well, we can't really use dlsym() call on the module in order to detect the version used - you could inject the logging somehow, but that's performance decreasement for nothing.
|
Updated by mfriedrich on 2012-04-27 16:56:39 +00:00
question remains if the loader should bail out entirely? |
This issue has been migrated from Redmine: https://dev.icinga.com/issues/2558
Created by mfriedrich on 2012-04-26 12:14:29 +00:00
Assignee: mfriedrich
Status: Resolved (closed on 2012-04-30 16:08:38 +00:00)
Target Version: 1.7
Last Update: 2012-04-30 16:08:38 +00:00 (in Redmine)
as now happened with the idomod change, the default make install routines should remark that as well.
Changesets
2012-04-26 12:38:17 +00:00 by mfriedrich fb2a14e
2012-04-26 12:41:23 +00:00 by mfriedrich 6f70d2b
2012-04-27 17:00:13 +00:00 by mfriedrich 453ed8e
Relations:
The text was updated successfully, but these errors were encountered: