Hi! Please, keep Lukas in CC, he's not on zypp-devel. Dne pátek 22 červen 2007 14:45 Duncan Mac-Vicar Prett napsal(a):
On Friday 22 June 2007 14:00:07 Jiri Srain wrote:
Hi! In order to allow workflows bound to patterns we need to extend the pattern definition on the media as well as the database and in-memory representation of a pattern. We need following tags:
I find the idea great, but the implementation propossal is really non-elegant (I don't have a better proposal)
"workflow_file" : string // file describing the workflow on the media "workflow_file_checksum" : the same as other checksums
what file is this? the workflow xml file?
Yes. The XML file describing the workflow the pattern brings.
"workflow_order" : string // to allow ordering, string for better extendibility
Yeah, I can see developers embeding YCP code in this string later. What is extendibility here then?
=Wkf =WkfCSum (if not specified in content file) =WkfOrd
What value does WkfOrd has? I still don't see YaST specific stuff being part of the pattern definition.
http://users.suse.cz/~locilka/PatternsAndAddOnsWithWorkflows/proposal-200 7- 06-04.txt
I have one question, I imagine the use case, you have a mail-server pattern, and if it installed you want to start the mail-server workflow.
How are you going to know either to start the postfix cfg workflow or the sendmail one, or [insert any supplied community module here]
The workflow must be defined the way that it will properly run which wizard is needed. It can check which dialogs to run depending on packages which are installed. In your case, if the pattern contains both sendmail and postfix, it should ask user in the first step. Obviously if we ship sendmail and postfix, user selects this pattern but installs eg. qmail, the pattern workflow cannot take care of it. Jiri -- Regards, Jiri Srain YaST Team Leader --------------------------------------------------------------------- SUSE LINUX, s.r.o. e-mail: jsrain@suse.cz Lihovarska 1060/12 tel: +420 284 028 959 190 00 Praha 9 fax: +420 284 028 951 Czech Republic http://www.suse.cz