https://bugzilla.novell.com/show_bug.cgi?id=366867 Jiří­ Suchomel changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO Info Provider| |anicka@novell.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User anicka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c2 Anna Bernathova changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |ASSIGNED Info Provider|anicka@novell.com | --- Comment #2 from Anna Bernathova 2008-03-05 06:47:10 MST --- In fact, the module is even more buggy: not only it completely messes up a file with more comments in a node, it also tends to drop values for nodes with one comment. Looking at the code written only with speed in mind, it will be a nasty work to fix it. But if you would like to continue using perl-XML-Bare, feel free to reassign to me, I hope I will fix it one day :-) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 Anna Bernathova changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User jsuchome@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c5 Jiří­ Suchomel changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|WONTFIX | --- Comment #5 from Jiří­ Suchomel 2008-03-06 01:45:17 MST --- That's bad :-( But of course, if the package is not fixable, rather drop it. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c7 Lukas Ocilka changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |anicka@novell.com --- Comment #7 from Lukas Ocilka 2008-03-06 01:53:31 MST --- No, I don't but shouldn't we rather ask Anicka :)? BTW it perfectly fits the needs of Community Repositories (which ag_anyxml was designed for), I haven't seen any problems the so far. On the other hand, it seems it cannot parse "any" XML ;) of course. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User anicka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c9 Anna Bernathova changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |ASSIGNED Info Provider|anicka@novell.com | --- Comment #9 from Anna Bernathova 2008-03-06 02:40:12 MST --- As for modules ready and easy to use from perl, there are basically three other possibilities: XML::MyXML which is very simple but written in perl (and so probably too slow) and XML::Simple with either XML::SAX or XML::Parser which is very easy to use (in fact, except of function names, usage is quite the same as for XML::Bare). I do not know what do you really need - something very fast, something easy to use or both? As for easiness, XML::Simple is a good choice. But I even do not know whether are XML::Parser or XML::SAX that much slower than XML::Bare - I think they should not be, they are using expat or libxml2, but you should know better. I think that using XML::Simple is a pretty good option, because it uses parsers that just work :-) If you really need a XML parser fast like a hell, I actually happen to know one. Unfortunately, it is written in C and has no perl bindings yet, so we would have to write them. (Which might not be easy, but at least possible, unlike fixing XML::Bare.) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User anicka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c11 --- Comment #11 from Anna Bernathova 2008-03-06 03:21:43 MST --- Fine. Anyway, please let me know if yast will not need XML::Bare anymore, so we could drop it. BTW, what is wrong with using XML::Simple for kiwi's config? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c13 --- Comment #13 from Lukas Ocilka 2008-03-06 03:39:23 MST --- BTW: the main reason why we use XML::Bare is that this package doesn't have huge RPM dependencies as XML::Simple has. We have/had had to use the ag_anyxml in installation (inst-sys). We really want to have the inst-sys as little as possible (You probably know the discussion: Not everybody has 512 MB RAM). -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c15 --- Comment #15 from Lukas Ocilka 2008-03-06 04:37:32 MST --- Yes, there are two issues: * We really need a unified ANY-XML agent for several YaST modules The current one often doesn't work and doesn't deserve name any_xml :) * We really need simple (and fast?) agent for reading and parsing the community repositories XML. This agent mustn't have huge dependencies the less the better. Solution(s): For community repositories, the solution is this - Either write simple one-task XML parser. - Or rename the any_xml to something else (but we don't want broken code in our distro). For the any-xml agent - Use another generic XML parser. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User jsuchome@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c17 --- Comment #17 from Jiří­ Suchomel 2008-03-19 09:24:11 MST --- So, Lukas, please: - rename current ag_anyxml to something different, e.g. ag_barexml and use this new one for reading the list of repositories during installation - let the ag_anyxml code use XML::Simple library. For start, just replace the lines my $xml = new XML::Bare ('text' => join ('', @lines)); my $root = $xml->parse(); undef $xml; with my $xml = new XML::Simple (); my $root = $xml->XMLin ($file); undef $xml; As I'm currently the only potential user of such agent, I could take care of its maintenance. So, reassign the bug to me after you adapt the repository reading code to the new agent path. I know it could be done the other way (leave ag_anyxml as it is and create new ag_simplexml with XML::Simple), but my proposal leaves the name "any" for the agent that should really be able to take care of any XML file. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User jsuchome@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c19 Jiří­ Suchomel changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |mvidner@novell.com --- Comment #19 from Jiří­ Suchomel 2008-03-25 01:39:43 MST --- Maybe the new ag_xml could be part of yast2-xml package instead of yast2... Martin, what do you think? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User jsuchome@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c21 --- Comment #21 from Jiří­ Suchomel 2008-03-25 06:57:44 MST --- (In reply to comment #20 from Martin Vidner)

yast2.rpm requires yast2-xml.rpm, nice try ;-) What are we trying to achieve here? How big are the respective packages and their dependencies?

Yes, I have realized it just minutes ago... The required package is perl-XML-Simple. It requires perl-XML-Parser, which is required by perl-XML-Bare as well. perl-XML-Simple.rpm has 67K, perl-XML-Bare.rpm has 23K. Basically, we need xml agent able to read /etc/security/pam_mount.conf.xml. The one based on Perl-XML-Bare doesn't handle it the one based on perl-XML-Simple does. yast2-users needs such agent, and in future possibly other modules (yast2-product-creator). For installation time, the Bare-based agent is sufficient. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User jsuchome@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c22 Jiří­ Suchomel changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jsrain@novell.com --- Comment #22 from Jiří­ Suchomel 2008-03-26 04:22:27 MST --- So, here's some proposal: 1. include both agents in yast2.rpm: perl-XML-Parser and other dependencies are probably already installed at the system anyway (for example it is required by suse_register), so we can afford additional requirement of (only) perl-XML-Simple 2. leave only ag_barexml in inst-sys: Bare.pm from perl-XML-Bare is already there, but perl-XML-Parser is not, so we will save some space with this. (If we agree on this solution, we must notify Steffen to replace ag_anyxml with ag_barexml) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User jsuchome@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c24 Jiří­ Suchomel changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|jsuchome@novell.com |locilka@novell.com Status|NEEDINFO |NEW Info Provider|snwint@novell.com | --- Comment #24 from Jiří­ Suchomel 2008-03-27 03:39:31 MST --- Well, I submitted yast2-2.16.40 with new ag_anyxml (based on perl-XML-Simple), because the new agent is needed anyway. This solves the original problem met in comment 0. What to use during installation is a task for Lukas (and/or Steffen) to decide. Just a note: current solution will not work, as inst-sys contains old ag_anyxml but the yast code in installation already uses ag_barexml. So, please find some solution (either put ag_barexml to inst-sys or drop the usage of Bare at all) before next alpha release. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User snwint@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c26 Steffen Winterfeldt changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #26 from Steffen Winterfeldt 2008-03-27 04:56:34 MST --- I'm including basically all of yast2 anyway. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=366867 User snwint@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=366867#c28 --- Comment #28 from Steffen Winterfeldt 2008-03-27 06:02:26 MST --- # du /work/CDs/all/full-i386/suse/i586/yast2.rpm 477 /work/CDs/all/full-i386/suse/i586/yast2.rpm With that size, I don't even start thinking about size optimizations... -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.