Adel Gadllah wrote:
Tim Lauridsen wrote:

Sorry, i forgot to answer.

There is some issue.
* It is using if cmd[0] == 'install', it is a TYPE_CORE plugin so it is running all yum api application (yumex, pirut, pup, yum-utils) But most of them don't give you any command line options to read, and you plugin will Traceback if cmd=[].
it seems you looked at the inital version and not the updated one I created later (I attached it to this mail again)
it does:
if not cmd:
       return
to avoid this traceback so its safe inside pup,pirut, yumex etc
* some people do a 'yum install yum-*' and install all plugins, without needing them, will this plugin be safe in this situation, or will people get strange result (could be avoided if the plugin action only was run by a command line option or is disabled by default)
no it shouldn't cause any problems the only apps where it would cause strange behaviors are whitelisted in default config file (ex: firefox because of flash)
* a man page for plugin and conf file would be usefull.

conf file is included in the later versions.
If the issues are solved, i have no objection to include it in yum-utils.

ok, I attached the current version please have a look at it.

Fine i will add it to yum-utils.

Tim
_______________________________________________
Yum-devel mailing list
[email protected]
https://lists.dulug.duke.edu/mailman/listinfo/yum-devel

Reply via email to