[Yum-devel] Re: basearchonly plugin
Tim Lauridsen
tla at rasmil.dk
Tue Jul 31 14:25:07 UTC 2007
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
More information about the Yum-devel
mailing list