June 2011 - YaST Development - openSUSE Mailing Lists

[yast-devel] Translation rounds for SP2?
by Ladislav Slezak 08 Jul '11

08 Jul '11

Hi, will be there any translation round for SP2? If yes, when? Can we change the translatable strings to SP2? I have noticed many typo fixes in trunk, can we backport them to SP2? -- Best Regards Ladislav Slezák Yast Developer ------------------------------------------------------------------------ SUSE LINUX, s.r.o. e-mail: lslezak(a)suse.cz Lihovarská 1060/12 tel: +420 284 028 960 190 00 Prague 9 fax: +420 284 028 951 Czech Republic http://www.suse.cz/ -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

3 2

[yast-devel] GsoC Weekly Report - "Separating libYUI"
by N.B.Prashanth 05 Jul '11

05 Jul '11

Hi, I would like to summarize my progress so far. I am working on the libYUI project as mentioned above. 1. libYUI and all three UI plugins (Qt, ncurses and GTK) have been ported to target systems (Fedora and Ubuntu). 2. The git repository at gitorious.org/libyui/libyui has been removed. This will be replaced with the official YUI svn repo once the conversion is complete. The old repo can be found at gitorious.org/libyui/gsoc2011 and will contain files related to my project. 3. Builds now make use of OBS. The download repository can be found at : http://download.opensuse.org/repositories/home:/nbprashanth/ . Currently build targets are Ubuntu (10.10 & 11.04), Fedora (14 & 15) and Debian 6. 4. GTK plugin no longer has libzypp as a dependency and can hence be ported. 5. Due to CMake issues on Ubuntu 11.04 with the GTK CMake module (Bug report : https://bugs.launchpad.net/ubuntu/+source/cmake/+bug/751940 ), builds using CMake have failed. Moreover, the default install location for gdk-pixbuf on F14 and F15 are different which caused the GTK plugin build to fail on F14 as well. 6. I have converted the GTK plugin to use autotools as opposed to CMake. This has made it compile successfully in Ubuntu 11.04 using pkg-config and I hope, it will compile of F14 as well. I will be updating the GTK package on OBS this week. 7. Documentation for libyui is now online and can be found at :http://doc.opensuse.org/projects/libyui/HEAD/ . This is an initial version and improvements may be expected in near future. The absolute path issue is now fixed. 8. Examples, update for YUI and full GsoC reports are also updated on my blog which can be found at : http://nbprashanth.wordpress.com/ . My last weeks GsoC report (GsoC Report - 3) can be found at : http://nbprashanth.wordpress.com/2011/06/27/gsoc-report-%E2%80%93-3/ 9. Patches for minor bugs such as "long long warnings" have been made. Not integrated upstream yet. 10. If anyone happens to test any of the YUI packages at OBS and finds a bug, please send me an email or leave a comment on the blog. -- Regards, N.B.Prashanth -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

4 3

[yast-devel] Re: [yast-commit] r64648 - /trunk/fcoe-client/src/FcoeClient.ycp
by Ladislav Slezak 30 Jun '11

30 Jun '11

Dne 30.6.2011 13:27, gs(a)svn2.opensuse.org napsal(a): [...] > + fcoe_config = FcoeClient::GetFcoeConfig(); > + summary = Summary::AddLine( summary, sformat( "DEBUG: %1", fcoe_config["DEBUG"]:"" ) ); > + summary = Summary::AddLine( summary, sformat( "USE SYSLOG: %1", fcoe_config["USE_SYSLOG"]:"" ) ); Just some notes: The translation marks are missing. I guess these values mean the direct config options, I think there should be rather some human readable texts instead. > + summary = Summary::AddLine( summary, _("Starting of services") ); > + summary = Summary::AddLine( summary, sformat( "fcoe: %1", (service_start["fcoe"]:false)?"true":"false" ) ); > + summary = Summary::AddLine( summary, sformat( "lldpad: %1", (service_start["lldpad"]:false)?"true":"false" ) ); The same here: no translation. Although "fcoe" might seem to be untranslatable translation allows to change the order which is essential in RTL (right-to-left) languages (like arabic). But again, here I'd rather see a service description text instead of the abbreviations (lldpad? WTF? Do I need it?). Do not use "true" "false" but rather _("Enabled") _("Disabled") to be consistent with other proposals. And the summary is printed in proportion font (in GUI) so the spaces do not guarantee any alignment in the output, just one space is enough IMO. -- Best Regards Ladislav Slezák Yast Developer ------------------------------------------------------------------------ SUSE LINUX, s.r.o. e-mail: lslezak(a)suse.cz Lihovarská 1060/12 tel: +420 284 028 960 190 00 Prague 9 fax: +420 284 028 951 Czech Republic http://www.suse.cz/ -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] GsoC Weekly Report - "Separating libYUI"
by N B Prashanth 30 Jun '11

30 Jun '11

Hi, I would like to summarize my progress so far. I am working on the libYUI project as mentioned above. 1. libYUI and all three UI plugins (Qt, ncurses and GTK) have been ported to target systems (Fedora and Ubuntu). 2. The git repository at gitorious.org/libyui/libyui has been removed. This will be replaced with the official YUI svn repo once the conversion is complete. The old repo can be found at gitorious.org/libyui/gsoc2011 and will contain files related to my project. 3. Builds now make use of OBS. The download repository can be found at : http://download.opensuse.org/repositories/home:/nbprashanth/ . Currently build targets are Ubuntu (10.10 & 11.04), Fedora (14 & 15) and Debian 6. 4. GTK plugin no longer has libzypp as a dependency and can hence be ported. 5. Due to CMake issues on Ubuntu 11.04 with the GTK CMake module (Bug report : https://bugs.launchpad.net/ubuntu/+source/cmake/+bug/751940 ), builds using CMake have failed. Moreover, the default install location for gdk-pixbuf on F14 and F15 are different which caused the GTK plugin build to fail on F14 as well. 6. I have converted the GTK plugin to use autotools as opposed to CMake. This has made it compile successfully in Ubuntu 11.04 using pkg-config and I hope, it will compile of F14 as well. I will be updating the GTK package on OBS this week. 7. Documentation for libyui is now online and can be found at :http://doc.opensuse.org/projects/libyui/HEAD/ . This is an initial version and improvements may be expected in near future. The absolute path issue is now fixed. 8. Examples, update for YUI and full GsoC reports are also updated on my blog which can be found at : http://nbprashanth.wordpress.com/ . My last weeks GsoC report (GsoC Report - 3) can be found at : http://nbprashanth.wordpress.com/2011/06/27/gsoc-report-%E2%80%93-3/ 9. Patches for minor bugs such as "long long warnings" have been made. Not integrated upstream yet. 10. If anyone happens to test any of the YUI packages at OBS and finds a bug, please send me an email or leave a comment on the blog. -- Regards, N.B.Prashanth -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

2 3

[yast-devel] Review code, please
by Martin Vidner 28 Jun '11

28 Jun '11

Hi, so far only a few people have been commenting on others' commits, and I'd like to ask the others to join the party. We need you! - One obvious reason is sharing the workload (but so far the volume is manageable). - Reading other people's code makes you a better programmer. - Commenting on other people's code improves the code, it prevents ugly code or outright bugs from getting in. - You think it is not your problem? So did I think before inheriting a certain ****ing piece of juicy ****. - Reading *good* code makes me feel better about our work, the project, the product. Or is everyone already reading everything and finding nothing to comment on? -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

4 5

[yast-devel] Deleting items from a selectionWidget
by Manu Gupta 28 Jun '11

28 Jun '11

Hi All I am trying to delete a single item from a selection widget, but I am unable to find a solution to it directly atleast. -- Regards Manu Gupta -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

2 1

[yast-devel] Changing the Attitude to Coding YaST
by Lukas Ocilka 27 Jun '11

27 Jun '11

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Yasties, Developers and Friends, According to the feedback I've got recently and what I've been feeling for a longer time, there is a chance to make YaST development cleaner, simpler and more enjoyable just with little tricks learned from the other projects we've been working on recently. Here you can see a list of unsorted ideas that came into my mind thinking about the other project and reusing experience with them in the YaST development: Code Reusability - - Usually the code is written for one purpose, but maybe we should start thinking before the development: Isn't the code reusable from anywhere else? Can the code be more generic? Think about CWM. - - Additionally, a lot of the code is in "include"d files or even in "client"s which makes the code almost impossible to be tested. That's also why we get syntax and/or runtime errors. - - Fix|Easy: Develop code in reusable libraries and write simple and understandable tests. Testing - - Connected to the previous point. Most of the current code is not tested because it actually cannot be tested. - - On the other hand, sometimes we think that writing testsuites is for dummies. But untested code causes bugs that need to be fixed later and thus is doesn't save the time at all. - - Fix|Medium: Write tests, make sure they are called during build. Coding Standards - - The code should be not only understandable to ourselves but also to the others. Using the same coding standards make it easier to understand the code structure, parameters, expected return value and so long. - - Examples: * Package::IsValid(package) -> returns boolean (maybe only Valid()) * Package::Install(package) -> do you need to read any documentation? * Using hashes with parameters instead of twelve parameters, eight of them optional * Indentation * No more "include"s if not necessary * Useful comments * Useful documentation of all the public functions * ... other ideas? ... - - Fix|Medium: Define the standards and use them (let's learn from other projects) Cooperation - - When the code has tests and it's mostly in compiled libraries, it's quite easy for more developers to work on the same project in the same time. The only thing they need to define is the API and using the same standards they can understand the API almost instantly. - - Better cooperation is usually useful when someone needs help from the others: too many features and bugs and to few time. - - Fix|Depends: ... Code Reviews - - In the SLMS team, we've found out that all the commits should be reviewed by someone else. This helps to identify potential issues in time. All the commits are public anyway. - - Fix|Easy to Medium: Either someone is always assigned to review everyone's code (takes too much time) or developers usually check code of someone else only. Shared Task List - - There are often even smaller tasks that need to be done by anyone in the team but they don't have their priority high enough (so called easy-hacks) - and that's why we forget them quite easily. - - Fix|Easy to Medium: Use some of the task managers, e.g., http://www.producteev.com/ anyone can pick a task and try to implement / fix it. I believe that you have other ideas how to improve the development - how to make the development and maintenance easier for you and/or the others. Or you maybe have an opinion on the ideas I just presented. Share them with us, please! :) Thanks in advance and Bye Lukas - -- Lukas Ocilka, Appliances Department, SUSE LINUX s.r.o. MD: Jeff Hawn, Jennifer Guild, Alena Hendrichova -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/ iD8DBQFN73YzVSqMdRCqTiwRAuC2AJsFP391iBGZwiJtbFNeN3FdwfZqRACfQqZZ xvmYF1Ar/q64o1iFMhVImlI= =R8U6 -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

13 31

[yast-devel] Testing Classical YaST in Hudson
by Lukas Ocilka 27 Jun '11

27 Jun '11

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hi, Many of us have already found out how running tests in Hudson can find potential (or real) issue just a few moments after a commit that breaks the code is submitted. We use Hudson for most of our projects now but we still don't run any tests for the Classical YaST: * What do you think of running the YaST tests in Hudson? * How could we do that? * Are there any additional tests we could run, for instance YCP syntax check for clients (?). * What about checking whether all functions are actually in use? Thanks in advance for your opinions and ideas. Lukas -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.15 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/ iEYEARECAAYFAk4EjEkACgkQVSqMdRCqTizhGACgkUuu7QAsJFQaqF5spFr9YmLI 02gAmwX+bP9dhmly3CuXZf89o9egmkb/ =B0Mk -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

3 2

[yast-devel] Classical YaST in Appliances: Choosing the UI
by Lukas Ocilka 24 Jun '11

24 Jun '11

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hi, I've just created a new JeOS appliance in SUSE Studio for testing purpose (YaST Backup bug). JeOS is known to have as few packages as possible. After selecting YaST Backup for installation, I've rather also checked whether I'll be actually able to run it, so searched for all YaST packages, especially the UI ones. And which one was selected? *yast2-gtk* OK, yast2.rpm requires yast2_ui and yast2-gtk, yast2-ncurses and yast2-qt all provide that symbol. Isn't there anything we could do about /better/ [1] selection [2]? Ideas? Opinions? Thx && Bye Lukas #1 'better' might be pretty subjective, I know. But this seems to be just alphabetically sorted list of solutions (for the first sight) #2 for example, KDE is installed -> Qt, GNOME/LXDE is installed -> GTK, neither of them (?) -> ncurses - -- Lukas Ocilka, Appliances Department, SUSE LINUX s.r.o. MD: Jeff Hawn, Jennifer Guild, Alena Hendrichova -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/ iD8DBQFOAvTtVSqMdRCqTiwRApcVAJ0aI3C01fxcJJSBIVUXduIqbLaFuQCfeIvy EIcQnPEenv7n64RHekoQiho= =GoF7 -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

3 4

[yast-devel] A few questions about libYUI
by Manu Gupta 23 Jun '11

23 Jun '11

Hi I am trying a few things in libYUI to ask of 1. How to set a value in a ComboBox? I have tried with both editable and non-editable comboBoxes I have a YComboBox * cc = YUI::widgetFactory(dialog,"SELECT",true); cc->addItem("XYZ"); cc->addItem("Blah"); cc->setValue("Manu"); In this case, the value Manu is not set and highlighted Now lets take this case YComboBox * cc = YUI::widgetFactory(dialog,"SELECT",true); cc->addItem("XYZ"); cc->addItem("Blah"); cc->setValue("Blah"); Same thing blah is not highlighted case 3 YComboBox * cc = YUI::widgetFactory(dialog,"SELECT",false); cc->addItem("XYZ"); cc->addItem("Blah"); cc->setValue("Blah"); I get a YUI::Exception So if anyone can cross check this case and let me know if it is a bug or not? Second Question is related to event handling If a selection in a comboBox changes, how would I able to recognise it Currently I follow this approach YDialog * dialog = YUI::widgetFactory->createDialog(); YComboBox * cc = YUI::widgetFactory(dialog,"SELECT",false); YWidgetEvent * event = (YWidgetEvent*)dialog->waitForEvent(); while(1){ if(event->reason()==SelectionChanged){ cout<<"XYZ"; } }; this does not seem to work ie the dialog is not recognising changes in my combobox, how do I solve them? -- Regards Manu Gupta -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

3 3

[yast-devel] YaST Group Desktop files do not comply with FreeDesktop standards
by Lukas Ocilka 22 Jun '11

22 Jun '11

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hi, I'm currently cleaning my bugzilla up and I've found this bug: Bug #470482 - YaST2 group .desktop files do not comply to FreeDesktop standard [1] which results in too much noise in .xsession-errors See https://bugzilla.novell.com/show_bug.cgi?id=470482 According to comments, YaST Group Desktop files cause many .xsession-errors because: * They contain Type="Application" * They (of course) do not contain any Exec=/some/binary There are two proposed solutions: * Add fake Exec=... line * Move the desktop files out of /usr/share/applications I don't like any of the proposed solutions much, so I'm asking *you* whether we have another option or what do you think of the proposed ones? Thanks in advance Lukas #1 http://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-late… - -- Lukas Ocilka, Appliances Department, SUSE LINUX s.r.o. MD: Jeff Hawn, Jennifer Guild, Alena Hendrichova -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/ iD8DBQFN/whuVSqMdRCqTiwRAoSAAKCH1b+V4oOWchroTjEvOfMkfiZadwCfXJI+ aUTjg/RkTb3Qnb4ORX/lnd0= =TXCn -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

4 6

[yast-devel] Re: [yast-commit] r64465 - Y2ALLGLOBAL
by Martin Vidner 21 Jun '11

21 Jun '11

On Fri, Jun 17, 2011 at 03:28:05PM -0000, lslezak(a)svn2.opensuse.org wrote: > Author: lslezak > Date: Fri Jun 17 17:28:04 2011 > New Revision: 64465 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64465&view=rev > Log: > move Wagon::FindMountPoint() function to separate include file > > so it can be tested, added a testsuite for it TESTS, WOW! BTW it is a secret little feature of yast2-core that defining the environment variable Y2ALLGLOBAL makes all functions in a module global. It is enabled during the tests, so wagon_helpers.ycp just because of the tests is unnecessary. > > Added: > trunk/wagon/src/include/wagon_helpers.ycp > trunk/wagon/testsuite/tests/FindMountPoint.err > trunk/wagon/testsuite/tests/FindMountPoint.out > trunk/wagon/testsuite/tests/FindMountPoint.ycp > Modified: > trunk/wagon/src/modules/Makefile.am > trunk/wagon/src/modules/Wagon.ycp -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

2 1

[yast-devel] Re: [yast-commit] r64487 - in /branches/SuSE-Code-11-SP2-Branch/dns-server: VERSION package/yast2-dns-server.changes src/dialog-main.ycp
by Martin Vidner 21 Jun '11

21 Jun '11

On Mon, Jun 20, 2011 at 01:47:39PM -0000, locilka(a)svn2.opensuse.org wrote: > Author: locilka > Date: Mon Jun 20 15:47:37 2011 > New Revision: 64487 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64487&view=rev > Log: > - Automatically changing entered forwarder to a localhost IP if > the server would forward to itself (non-local IP) (BNC #676676). > - 2.17.16 > Modified: branches/SuSE-Code-11-SP2-Branch/dns-server/src/dialog-main.ycp > URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/dns-s… > ============================================================================== > --- branches/SuSE-Code-11-SP2-Branch/dns-server/src/dialog-main.ycp (original) > +++ branches/SuSE-Code-11-SP2-Branch/dns-server/src/dialog-main.ycp Mon Jun 20 15:47:37 2011 > @@ -428,6 +428,66 @@ > } > > /** > + * Returns list of IPs currently used by the system. > + * > + * @param boolean whether local addresses should be returned as well (the default is false) > + */ > +list <string> CurrentlyUsedIPs (boolean including_local) { > + string cmd = "ip addr show | grep 'inet\$6\$\\?' | sed 's/^[ \\t]\\+inet\$6\$\\?[ \\t]\\+\$[^\\/]\\+\$\\/.*$/\\2/'"; BTW the -E/--extended-regexp flag makes it easier: grep -E 'inet6?' | ... And what I learned today, sed has -r/--regexp-extended (don't you love consistent tools?) > + map cmd_ret = (map) SCR::Execute (.target.bash_output, cmd); > + > + if (cmd_ret == nil || cmd_ret["exit"]:-1 != 0) { > + y2error ("Cannot get list of used IPs: %1", cmd_ret); > + return nil; > + } > + > + list <string> used_ips = splitstring (cmd_ret["stdout"]:"", "\n"); > + used_ips = filter (string used_ip, used_ips, ``(used_ip != nil && used_ip != "")); list <string> used_ips = String::NewlineItems(cmd_ret["stdout"]:""); Everyone raise your mouse cursor if you've ever reimplemented this function ;-) > + > + // Filtering out all local IPs > + if (including_local != true) { > + used_ips = filter (string used_ip, used_ips, ``( > + !regexpmatch(used_ip, "127\.0\.0\..*") && used_ip != "::1" > + )); > + } > + > + return used_ips; > +} -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

2 1

[yast-devel] Re: [yast-commit] r64445 - in /trunk/wagon: package/yast2-wagon.changes src/Makefile.am src/clients/wagon_registration_check.ycp src/config/online_migration-SLE.xml src/config/online_migration-openSUSE.xml
by Martin Vidner 20 Jun '11

20 Jun '11

On Fri, Jun 17, 2011 at 11:49:46AM -0000, lslezak(a)svn2.opensuse.org wrote: > Author: lslezak > Date: Fri Jun 17 13:49:46 2011 > New Revision: 64445 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64445&view=rev > Log: > added a new client for checking registration status before starting migration (fate#309594) > > Added: trunk/wagon/src/clients/wagon_registration_check.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/wagon/src/clients/wagon_registra… > ============================================================================== > --- trunk/wagon/src/clients/wagon_registration_check.ycp (added) > +++ trunk/wagon/src/clients/wagon_registration_check.ycp Fri Jun 17 13:49:46 2011 > + define boolean OutdatedStatus(map<string,any> status) Please document the format of status. In a previous commit, I found a reference to https://wiki.innerweb.novell.com/index.php/Registration#Add_Registration_St… which is not readable publicly. Additionally, it only documents the XML format and who remembers how XML::Simple presents it in a map? It would be unmaintainable to repeat the description everywhere but I think at least each file should have reference. > + { > + // missing data, we cannot tell if it's outdated, suppose not > + if (!haskey(status, "timestamp") || status["timestamp"]:0 <= 0) > + { > + return false; > + } > + > + boolean ret = status["timestamp"]:time() + max_age < time(); > + y2milestone("Registration status is outdated: %1", ret); > + > + return ret; > + } ... especially if it is then used all over the place: > + define list<map<string,any> > UnknownProducts(map<string,any> status) > + define string RegistrationSummary(map<string,any> status) > + define void SetContent(map<string,any> status) > + define symbol RegistrationSummaryDialog(map<string,any> status) > + define boolean Interactive(map<string,any> status) -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

1 0

[yast-devel] Re: [yast-commit] r64378 - in /trunk/sshd: VERSION package/yast2-sshd.changes src/Sshd.ycp src/dialogs.ycp src/wizards.ycp
by Martin Vidner 20 Jun '11

20 Jun '11

On Wed, Jun 15, 2011 at 08:47:50AM -0000, locilka(a)svn2.opensuse.org wrote: > Author: locilka > Date: Wed Jun 15 10:47:50 2011 > New Revision: 64378 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64378&view=rev > Log: > Added possibility to manage the sshd service (#556535) > > > Modified: > trunk/sshd/VERSION > trunk/sshd/package/yast2-sshd.changes > trunk/sshd/src/Sshd.ycp > trunk/sshd/src/dialogs.ycp > trunk/sshd/src/wizards.ycp > > Modified: trunk/sshd/VERSION > URL: http://svn.opensuse.org/viewcvs/yast/trunk/sshd/VERSION?rev=64378&r1=64377&… > ============================================================================== > --- trunk/sshd/VERSION (original) > +++ trunk/sshd/VERSION Wed Jun 15 10:47:50 2011 > @@ -1 +1 @@ > -2.18.0 > +2.21.0 > > Modified: trunk/sshd/package/yast2-sshd.changes > URL: http://svn.opensuse.org/viewcvs/yast/trunk/sshd/package/yast2-sshd.changes?… > ============================================================================== > --- trunk/sshd/package/yast2-sshd.changes (original) > +++ trunk/sshd/package/yast2-sshd.changes Wed Jun 15 10:47:50 2011 > @@ -1,4 +1,10 @@ > ------------------------------------------------------------------- > +Wed Jun 15 10:45:10 CEST 2011 - locilka(a)suse.cz > + > +- Added possibility to manage the sshd service (#556535). s/#556535/bnc#556535/ > +- 2.21.0 > + > +------------------------------------------------------------------- > Wed Jan 13 18:56:03 CET 2010 - kmachalkova(a)suse.cz > > - Adjusted .desktop file(s) to wrap /sbin/yast2/ calls in xdg-su > > Modified: trunk/sshd/src/Sshd.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/sshd/src/Sshd.ycp?rev=64378&r1=6… > ============================================================================== > --- trunk/sshd/src/Sshd.ycp (original) > +++ trunk/sshd/src/Sshd.ycp Wed Jun 15 10:47:50 2011 > @@ -30,7 +30,7 @@ > /** > * Sleep time between Read or Write steps > */ > -integer sl = 200; > +integer sl = 150; WTF? > > /** > * Returns whether the configuration has been modified. > @@ -46,6 +46,40 @@ > modified = true; > } > > +string service_name = "sshd"; > + > +boolean start_service = nil; > + > +global boolean GetStartService () { > + return start_service; > +} > + > +global void SetStartService (boolean new_state) { > + if (new_state == nil) { > + y2error ("Cannot set 'StartService' to %1", new_state); > + return; > + } else if (start_service == new_state) { > + y2warning ("'StartService' unchanged"); > + return; > + } > + > + start_service = new_state; > + SetModified(); > +} > + > +/** > + * Describes whether the daemon is running > + */ > +boolean Running () { > + return (Service::Status(service_name) == 0); > +} > + > +boolean ReadStartService () { > + start_service = Running(); > + > + return (start_service != nil); > +} > + > /** > * map of SSHD settings > */ > @@ -129,26 +163,6 @@ > } > > /** > - * Describes whether the daemon is running > - */ > -boolean sshd_is_running = false; > - > -/** > - * Reads current sshd status > - */ > -boolean ReadSSHDService () { > - if (Service::Status("sshd") == 0) { > - sshd_is_running = true; > - } else { > - sshd_is_running = false; > - } > - > - y2milestone((sshd_is_running ? "SSH is running":"SSH is not running")); > - > - return true; > -} > - > -/** > * Exports the current configuration. > * > * @return map of a current configuration > @@ -161,7 +175,7 @@ > global map Export () { > return $[ > "config" : SETTINGS, > - "status" : sshd_is_running, > + "status" : Running(), > ]; > } > > @@ -170,29 +184,13 @@ > */ > global boolean Import (map import_map) { > SETTINGS = (map <string, list<string> >) import_map["config"]:$[]; > - sshd_is_running = (import_map["status"]:false == true); > + SetStartService (import_map["status"]:false == true); > SetModified(); > y2milestone ("Configuration has been imported"); > return true; > } > > /** > - * Restarts the sshd when the daemon was running when starting the configuration > - */ > -boolean WriteSSHDService () { > - boolean all_ok = true; > - > - if (sshd_is_running) { > - y2milestone("Restarting sshd daemon"); > - all_ok = Service::Restart("sshd"); > - } else { > - y2milestone("Sshd is not running - leaving..."); > - } > - > - return all_ok; > -} > - > -/** > Returns a confirmation popup dialog whether user wants to really abort. > */ > global boolean Abort() { > @@ -225,7 +223,7 @@ > /* SSHD read dialog caption */ > string caption = _("Initializing the SSHD Configuration"); > > - integer steps = 3; > + integer steps = 4; aaah, so "integer sl = 600 / steps" above, right? 8-] This Progress stuff is so... er... useful? > > Progress::New( caption, " ", steps, [ > /* Progress stage 1 */ > @@ -233,15 +231,18 @@ > /* Progress stage 2 */ > _("Read the current SSHD state"), > /* Progress stage 3 */ > - _("Read firewall settings") > + _("Read firewall settings"), > + /* Progress stage 4 */ > + _("Read service status"), > ], [ > /* Progress step 1 */ > _("Reading the current SSHD configuration..."), > /* Progress step 2 */ > - _("Reading the current SSHD state..."), > + _("Reading the current SSHD state"), > /* Progress step 3 */ > _("Reading firewall settings..."), > - /* Progress finished */ > + /* Progress step 4 */ > + _("Reading service status..."), > Message::Finished() > ], > "" > @@ -251,14 +252,12 @@ > > if (PollAbort()) return false; > Progress::NextStage(); > - /* Error message */ > if(!ReadSSHDSettings()) Report::Error(Message::CannotReadCurrentSettings()); > sleep(sl); > > if (PollAbort()) return false; > - Progress::NextStep(); > - /* Error message */ > - if(!ReadSSHDService()) Report::Error(_("Cannot read the current SSHD state.")); > + Progress::NextStage(); > + if (!ReadStartService()) Report::Error(Message::CannotReadCurrentSettings()); > sleep(sl); > > if (PollAbort()) return false; > @@ -273,10 +272,61 @@ > Progress::NextStage (); > sleep(sl); > > + if (PollAbort()) return false; > + Progress::NextStage (); > + start_service = Service::Enabled (service_name); > + sleep(sl); > + > modified = false; > return true; > } > > +boolean AdjustSSHDService () { > + boolean enable_and_start = GetStartService(); > + boolean enabled = Service::Enabled (service_name); > + boolean running = (Service::Status (service_name) == 0); > + > + if (enable_and_start == nil) { > + y2error ("Configuration error: Cannot start/stop service %1", service_name); > + return false; > + } > + > + // Service enable/disable > + if (enable_and_start == enabled) { > + y2milestone ("Service '%1' is already in the desired state", service_name); > + } else { > + if (enable_and_start && !Service::Enable (service_name)) { > + y2error ("Cannot enable service %1", service_name); > + return false; > + } else if (!enable_and_start && !Service::Disable (service_name)) { > + y2error ("Cannot disable service %1", service_name); > + return false; > + } > + } > + > + // Start / restart service > + if (enable_and_start) { > + if (Running()) { > + if (!Service::Restart(service_name)) { > + y2error ("Cannot restart service %1", service_name); > + return false; > + } > + } else { > + if (!Service::Start (service_name)) { > + y2error ("Cannot start service %1", service_name); > + return false; > + } > + } > + // Stop a running service > + } else if (Running()) { > + if (!Service::Stop (service_name)) { > + return false; > + } > + } > + > + return true; > +} > + > /** > * Write all SSHD settings > * @return true on success > @@ -286,22 +336,22 @@ > /* SSHD read dialog caption */ > string caption = _("Saving the SSHD Configuration"); > > - integer steps = 3; > - > + integer steps = 4; > + > Progress::New(caption, " ", steps, [ > /* Progress stage 1 */ > _("Write the SSHD settings"), > /* Progress stage 2 */ > - _("Adjust the SSHD service"), > + _("Write firewall settings"), > /* Progress stage 3 */ > - _("Write firewall settings") > + _("Adjust SSHD service"), > ], [ > /* Progress step 1 */ > _("Writing the SSHD settings..."), > /* Progress step 2 */ > - _("Adjusting the SSHD service..."), > - /* Progress step 3 */ > _("Writing firewall settings..."), > + /* Progress step 3 */ > + _("Adjusting SSHD service..."), > Message::Finished() > ], > "" > @@ -317,16 +367,15 @@ > > if(PollAbort()) return false; > Progress::NextStage (); > + boolean progress_state = Progress::set (false); > /* Error message */ > - if(!WriteSSHDService()) Report::Error (Message::CannotAdjustService("sshd")); > + if(!SuSEFirewall::Write()) Report::Error(_("Cannot write firewall settings.")); > + Progress::set (progress_state); > sleep(sl); > > if(PollAbort()) return false; > Progress::NextStage (); > - boolean progress_state = Progress::set (false); > - /* Error message */ > - if(!SuSEFirewall::Write()) Report::Error(_("Cannot write firewall settings.")); > - Progress::set (progress_state); > + AdjustSSHDService(); > sleep(sl); > > Progress::NextStage (); > > Modified: trunk/sshd/src/dialogs.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/sshd/src/dialogs.ycp?rev=64378&r… > ============================================================================== > --- trunk/sshd/src/dialogs.ycp (original) > +++ trunk/sshd/src/dialogs.ycp Wed Jun 15 10:47:50 2011 > @@ -28,6 +28,15 @@ > mbox_y = 0; > } > > +term ServiceConfigurationDialogContent () { > + return `VBox ( > + "start_stop", > + `VSpacing(1), > + "auto_start_up", > + `VStretch() > + ); > +} > + > term ServerConfigurationDialogContent () { > return `VBox ( > `Left(`Label(_("SSHD TCP Ports"))), > @@ -45,7 +54,7 @@ > /* a push button */ > `PushButton(`id("delete_port"), _("&Delete")) > )), > - `VSpacing(1), > + `VSpacing (1), > `Frame ( > /* a dialog frame caption */ > _("Server Features"), > > Modified: trunk/sshd/src/wizards.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/sshd/src/wizards.ycp?rev=64378&r… > ============================================================================== > --- trunk/sshd/src/wizards.ycp (original) > +++ trunk/sshd/src/wizards.ycp Wed Jun 15 10:47:50 2011 > @@ -14,10 +14,20 @@ > import "CWM"; > import "CWMTab"; > import "CWMFirewallInterfaces"; > +import "CWMServiceStart"; > +import "Sshd"; > > include "sshd/complex.ycp"; > include "sshd/dialogs.ycp"; > > +void SaveAndRestart () { > + Wizard::CreateDialog (); > + Wizard::RestoreHelp (HELPS["write"]:""); > + Sshd::Write (); > + sleep (1000); > + UI::CloseDialog (); > +} > + > /** > * Main workflow of the sshd configuration > * @return sequence result > @@ -52,9 +62,49 @@ > "services" : [ "service:sshd" ], > "display_details" : true, > ]), > + "start_stop" : CWMServiceStart::CreateStartStopWidget ($[ > + "service_id" : "sshd", > + // label - service status, informative text > + "service_running_label" : _("SSH server is running"), > + // label - service status, informative text > + "service_not_running_label" : _("SSH server is not running"), > + // push button (SSH service handling) > + "start_now_button" : _("&Start SSH Server Now"), > + // push button (SSH service handling) > + "stop_now_button" : _("S&top SSH Server Now"), > + "save_now_action" : SaveAndRestart, > + // push button (SSH service handling) > + "save_now_button" : _("Save Settings and Reload SSH Server &Now"), > + "help" : sformat (CWMServiceStart::StartStopHelpTemplate (true), > + // part of help text, used to describe pusbuttons (matching SSH service handling but without "&") > + _("Start SSH Server Now"), > + // part of help text, used to describe pusbuttons (matching SSH service handling but without "&") > + _("Stop SSH Server Now"), > + // part of help text, used to describe pusbuttons (matching SSH service handling but without "&") > + _("Save Settings and Reload SSH Server Now")), > + ]), > + "auto_start_up" : CWMServiceStart::CreateAutoStartWidget ($[ > + "get_service_auto_start" : Sshd::GetStartService, > + "set_service_auto_start" : Sshd::SetStartService, > + // radio button (starting SSH service - option 1) > + "start_auto_button" : _("Now and When &Booting"), > + // radio button (starting SSH service - option 2) > + "start_manual_button" : _("&Manually"), > + "help" : sformat (CWMServiceStart::AutoStartHelpTemplate (), > + // part of help text, used to describe radiobuttons (matching starting SSH service but without "&") > + _("Now and When Booting"), > + // part of help text, used to describe radiobuttons (matching starting SSH service but without "&") > + _("Manually") > + ), > + ]), Hmm, when I saw 'import "CWMServiceStart"' above, I thought "good, that will save us so much repetitive code" :-/ Overall... God, so much code for a stupid totally standard sysvinit service?! I want a simple declaration to replace this *entire* patch: service :name => _("SSH Server"), :sysvinit => "sshd" We really should make better libraries. > ]; > > map <string, any> tabs = $[ > + "service_configuration" : $[ > + "header" : _("&Start-Up"), > + "widget_names" : ["start_stop", "auto_start_up"], > + "contents" : ServiceConfigurationDialogContent(), > + ], > "server_configuration" : $[ > "header" : _("&General"), > "widget_names" : ["sc", "fw"], > @@ -74,7 +124,7 @@ > > map <string, any> wd = $[ > "tab" : CWMTab::CreateWidget ($[ > - "tab_order" : ["server_configuration", "login_settings", "proto_and_cipher"], > + "tab_order" : ["service_configuration", "server_configuration", "login_settings", "proto_and_cipher"], > "tabs" : tabs, > "widget_descr" : widgets, > "initial_tab" : "server_configuration", > > -- > To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org > For additional commands, e-mail: yast-commit+help(a)opensuse.org > -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

2 1

[yast-devel] Re: [yast-commit] r64422 - in /trunk/yast2/library/network: src/SuSEFirewallExpertRules.ycp testsuite/tests/SuSEFirewallExpertRules.out testsuite/tests/SuSEFirewallExpertRules.ycp
by Martin Vidner 17 Jun '11

17 Jun '11

On Thu, Jun 16, 2011 at 01:09:58PM -0000, locilka(a)svn2.opensuse.org wrote: > Author: locilka > Date: Thu Jun 16 15:09:58 2011 > New Revision: 64422 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64422&view=rev > Log: > A better fix for my previous commit (stricter/correct validation) Wow! That is a sign that having tests does help. > Modified: > trunk/yast2/library/network/src/SuSEFirewallExpertRules.ycp > trunk/yast2/library/network/testsuite/tests/SuSEFirewallExpertRules.out > trunk/yast2/library/network/testsuite/tests/SuSEFirewallExpertRules.ycp > > Modified: trunk/yast2/library/network/src/SuSEFirewallExpertRules.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/yast2/library/network/src/SuSEFi… > ============================================================================== > --- trunk/yast2/library/network/src/SuSEFirewallExpertRules.ycp (original) > +++ trunk/yast2/library/network/src/SuSEFirewallExpertRules.ycp Thu Jun 16 15:09:58 2011 > @@ -83,7 +83,7 @@ > // A.B.C.D/1 - A.B.C.D/32 (IP with a numeric netmask) > else if (regexpmatch(network, "^" + type_ip4 + "/[01234567890]+$")) { > string part_ip = regexpsub (network, "^(" + type_ip4 + ")/[01234567890]+$", "\\1"); > - string part_bits = regexpsub (network, "^" + type_ip4 + "/([01234567890])+$", "\\1"); > + string part_bits = regexpsub (network, "^" + type_ip4 + "/([01234567890]+)$", "\\1"); BTW there is still 0 twice. Next time use [0-9] which is safe (unlike [a-z]). > > return (IP::Check4(part_ip) && ValidNetmaskBits(tointeger(part_bits))); > } > > Modified: trunk/yast2/library/network/testsuite/tests/SuSEFirewallExpertRules.out > URL: http://svn.opensuse.org/viewcvs/yast/trunk/yast2/library/network/testsuite/… > ============================================================================== > --- trunk/yast2/library/network/testsuite/tests/SuSEFirewallExpertRules.out (original) > +++ trunk/yast2/library/network/testsuite/tests/SuSEFirewallExpertRules.out Thu Jun 16 15:09:58 2011 > @@ -6,8 +6,11 @@ > Return true > Return true > Return true > +Return true > Dump All these should be *invalid* (false): > Return false > Return false > Return false > +Return false > +Return false > Dump == Done == > > Modified: trunk/yast2/library/network/testsuite/tests/SuSEFirewallExpertRules.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/yast2/library/network/testsuite/… > ============================================================================== > --- trunk/yast2/library/network/testsuite/tests/SuSEFirewallExpertRules.ycp (original) > +++ trunk/yast2/library/network/testsuite/tests/SuSEFirewallExpertRules.ycp Thu Jun 16 15:09:58 2011 > @@ -9,11 +9,12 @@ > > list <string> valid_network_definitions = [ > "192.168.0.1", "192.168.0.0/24", "192.168.0.1/32", "192.168.0.0/255.255.0.0", > - "192.168.0.0/255.255.224.0", "0/0", > + "192.168.0.0/255.255.224.0", "0/0", "192.168.0.1/16", > ]; > > list <string> invalid_network_definitions = [ > "192.168.0.355", "192.168.0.1/0", "192.168.0.0/255.255.333.0", > + "192.168.0.1/888", "192.168.0.1/33", > ]; > > DUMP("All these should be *valid* (true):"); > > -- > To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org > For additional commands, e-mail: yast-commit+help(a)opensuse.org > -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

1 0

[yast-devel] Re: [yast-commit] r64419 - in /trunk/yast2: ./ library/network/src/ library/network/testsuite/tests/ package/
by Lukas Ocilka 16 Jun '11

16 Jun '11

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Dne 16.6.2011 14:58, locilka(a)svn2.opensuse.org napsal(a): > Author: locilka > Date: Thu Jun 16 14:58:44 2011 > New Revision: 64419 > > Modified: trunk/yast2/library/network/src/SuSEFirewallExpertRules.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/yast2/library/network/src/SuSEFi… > ============================================================================== > --- trunk/yast2/library/network/src/SuSEFirewallExpertRules.ycp (original) > +++ trunk/yast2/library/network/src/SuSEFirewallExpertRules.ycp Thu Jun 16 14:58:44 2011 > @@ -83,7 +83,7 @@ > // A.B.C.D/1 - A.B.C.D/32 (IP with a numeric netmask) > else if (regexpmatch(network, "^" + type_ip4 + "/[01234567890]+$")) { > string part_ip = regexpsub (network, "^(" + type_ip4 + ")/[01234567890]+$", "\\1"); > - string part_bits = regexpsub (network, "^" + type_ip4 + "(/[01234567890])+$", "\\1"); > + string part_bits = regexpsub (network, "^" + type_ip4 + "/([01234567890])+$", "\\1"); Ouch :) regexpsub (network, "^" + type_ip4 + "/([01234567890])+$", "\\1") should be regexpsub (network, "^" + type_ip4 + "/([01234567890]+)$", "\\1"); although it passes the tests - -- Lukas Ocilka, Appliances Department, SUSE LINUX s.r.o. MD: Jeff Hawn, Jennifer Guild, Alena Hendrichova -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/ iD8DBQFN+f8PVSqMdRCqTiwRAibZAJoDu33xLMb93n/Zw2cqOqrOhrS64wCfXZu8 nttdeTSNJzM+FfIVYfFMU3E= =iuE+ -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] Re: [yast-commit] r64348 - in /trunk/ca-management: ./ VERSION package/yast2-ca-management.changes src/CaMgm.ycp src/ca_mgm_proposal.ycp src/ca_select_proposal.ycp src/new_cert_read_write.ycp
by Martin Vidner 16 Jun '11

16 Jun '11

On Tue, Jun 14, 2011 at 12:53:18PM -0000, jreidinger(a)svn2.opensuse.org wrote: > Author: jreidinger > Date: Tue Jun 14 14:53:17 2011 > New Revision: 64348 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64348&view=rev > Log: > merge SP1 features to ca management > > Modified: > trunk/ca-management/ (props changed) > trunk/ca-management/VERSION > trunk/ca-management/package/yast2-ca-management.changes > trunk/ca-management/src/CaMgm.ycp > trunk/ca-management/src/ca_mgm_proposal.ycp > trunk/ca-management/src/ca_select_proposal.ycp > trunk/ca-management/src/new_cert_read_write.ycp > > Modified: trunk/ca-management/VERSION > URL: http://svn.opensuse.org/viewcvs/yast/trunk/ca-management/VERSION?rev=64348&… > ============================================================================== > --- trunk/ca-management/VERSION (original) > +++ trunk/ca-management/VERSION Tue Jun 14 14:53:17 2011 > @@ -1 +1 @@ > -2.21.0 > +2.21.1 > > Modified: trunk/ca-management/package/yast2-ca-management.changes > URL: http://svn.opensuse.org/viewcvs/yast/trunk/ca-management/package/yast2-ca-m… > ============================================================================== > --- trunk/ca-management/package/yast2-ca-management.changes (original) > +++ trunk/ca-management/package/yast2-ca-management.changes Tue Jun 14 14:53:17 2011 > @@ -1,4 +1,12 @@ > ------------------------------------------------------------------- > +Tue Jun 14 13:02:34 CEST 2011 - jreidinger(a)suse.cz > + > +- version 2.21.1 > +- country selection combobox is editable to allow choose of choosing > + countries not listed there (FATE#310179) > +- allow to change altarnative name in CA proposal (FATE#308423) alternative > + > +------------------------------------------------------------------- > Fri Apr 29 11:32:34 CEST 2011 - mc(a)suse.de > > - version 2.21.0 > > Modified: trunk/ca-management/src/CaMgm.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/ca-management/src/CaMgm.ycp?rev=… > ============================================================================== > --- trunk/ca-management/src/CaMgm.ycp (original) > +++ trunk/ca-management/src/CaMgm.ycp Tue Jun 14 14:53:17 2011 > @@ -136,6 +136,7 @@ > global symbol prop_selection = `def; // `none, `disk > > global list<map> prop_subject_alt_name_list = []; > +global boolean prop_adv_cri_issuer_alt_name = false; bbrvtns! i lv bbrvtns! f ths s prt f xstng cnvntn (*sgh*) , plz dd cmmnt, lk: // property: advanced criminal issuer altimeter name > > global boolean adv_subject_alt_name_show_email = false; > > > Modified: trunk/ca-management/src/ca_mgm_proposal.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/ca-management/src/ca_mgm_proposa… > ============================================================================== > --- trunk/ca-management/src/ca_mgm_proposal.ycp (original) > +++ trunk/ca-management/src/ca_mgm_proposal.ycp Tue Jun 14 14:53:17 2011 > @@ -54,18 +54,20 @@ > > map<string,string> host_ips = getHostIPs (); > string hostname_bak = ""; > - CaMgm::prop_subject_alt_name_list = []; > + if (size(CaMgm::prop_subject_alt_name_list) <= 0){ > + CaMgm::prop_subject_alt_name_list = []; > > - foreach(string key, string value, host_ips, { > - if(value == "DNS") > - { > - hostname_bak = key; > - } > - map elem = $[]; > - elem["kind"] = value; > - elem["name"] = key; > - CaMgm::prop_subject_alt_name_list = add (CaMgm::prop_subject_alt_name_list, elem); > - }); > + foreach(string key, string value, host_ips, { > + if(value == "DNS") > + { > + hostname_bak = key; > + } > + map elem = $[]; > + elem["kind"] = value; > + elem["name"] = key; > + CaMgm::prop_subject_alt_name_list = add (CaMgm::prop_subject_alt_name_list, elem); > + }); > + } Thanks for fixing the indentation. Could you please make such fixes in separate commits, it makes reviewing easier. > > map retmap = (map) SCR::Execute (.target.bash_output, > "/bin/hostname --long", > > Modified: trunk/ca-management/src/ca_select_proposal.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/ca-management/src/ca_select_prop… > ============================================================================== > --- trunk/ca-management/src/ca_select_proposal.ycp (original) > +++ trunk/ca-management/src/ca_select_proposal.ycp Tue Jun 14 14:53:17 2011 > @@ -49,6 +49,151 @@ > }); > return result; > } > + > + define list editAltNames( list<map> initial_set ){ > + term alt_dialog = `HBox( `VSpacing(10), > + `VBox (`HSpacing(10), > + `RadioButtonGroup(`id(`rb), > + `HBox( > + `RadioButton(`id("URI"), `opt(`notify), "URI", true), > + `RadioButton(`id("email"), `opt(`notify), "email"), > + `RadioButton(`id("DNS"), `opt(`notify), "DNS"), > + `RadioButton(`id("IP"), `opt(`notify), "IP"), > + `RadioButton(`id("RID"), `opt(`notify), "RID"), > + `RadioButton(`id("MS-UPN"), `opt(`notify), "MS-UPN"), > + `RadioButton(`id("K5PN"), `opt(`notify), "K5PN") > + )), > + `TextEntry(`id(`name), _("&Name:")), labels should not have trailing colons > + // push button label > + `HBox(`PushButton (`id(`cancel), `opt(`key_F9), Label::CancelButton()), > + `HStretch(), > + `PushButton (`id(`ok), `opt(`default, `key_F10), Label::OKButton()) > + ), > + `HSpacing(10) > + ), > + `VSpacing(10) > + ); > + UI::OpenDialog ( > + `VBox ( `Left (`CheckBox(`id(`id_adv_cri_subject_alt_name), > + _("critical"), "critical" what? Add a comment for translators so that they know whether it is "kritický" (certifikát) or "kritická" (chyba) or "kritické" (čtení kódu) > + CaMgm::prop_adv_cri_issuer_alt_name > + )), > + `VSpacing(1.5), > + `HBox( > + `VSpacing(5), > + `HWeight (3, `Table (`id (`id_adv_subject_alt_name), > + `header ( > + // To translators: table headers > + _("Kind"), > + _("Name") > + ) > + )), > + `HWeight (1, `VBox( > + `HBox ( `HWeight (1,`PushButton (`id (`add), > + `opt (`key_F3), > + Label::AddButton() > + ))), > + `HBox ( `HWeight (1,`PushButton (`id (`edit), > + `opt (`key_F3), > + Label::EditButton() > + ))), > + `HBox ( `HWeight (1,`PushButton (`id (`delete), > + `opt (`key_F5), > + Label::DeleteButton() > + ))), > + `VStretch() > + ))), > + `VSpacing(1.5), > + `ButtonBox(`PushButton(`id(`ok),`opt(`key_F10),Label::OKButton()), > + `PushButton (`id(`cancel),`opt(`key_F9,`default), Label::CancelButton()) > + ) > + )); > + any ret = nil; > + do { > + integer i = 0; > + list<term> table_list = maplist( map element, initial_set, ``{ > + i = i + 1; > + return `item (`id (i), element["kind"]:"", element["name"]:"" ); > + }); > + UI::ChangeWidget(`id(`id_adv_subject_alt_name), `Items, table_list); > + ret = UI::UserInput(); > + if (ret == `add || ret == `edit ){ > + string current_kind = ""; > + string current_name = ""; > + if (ret == `edit){ > + integer current_item = (integer)UI::QueryWidget(`id(`id_adv_subject_alt_name), `CurrentItem); > + term table_item = (term) UI::QueryWidget(`id(`id_adv_subject_alt_name), `Item(current_item)); > + if (table_item == nil) > + { > + Popup::Error (_("No item has been selected.")); > + continue; > + } > + > + current_kind = (string) table_item[1]:""; > + current_name = (string) table_item[2]:""; > + } > + UI::OpenDialog (`opt(`decorated ),alt_dialog); > + UI::SetFocus (`id(`ok)); > + if (ret == `edit){ > + UI::ChangeWidget(`id(`name), `Value, current_name); > + UI::ChangeWidget(`id(`rb), `CurrentButton, current_kind); > + } > + if(!CaMgm::adv_subject_alt_name_show_email) > + { > + UI::ChangeWidget(`id("email"), `Enabled, false); > + } else { > + UI::ChangeWidget(`id("email"), `Enabled, true); > + } > + while (true) > + { > + any ret2 = UI::UserInput(); > + if (ret2 == `ok) > + { > + map new_entry = (ret==`add) ? $[] : > + find(map m,initial_set, > + ``{return (m["kind"]:"" == current_kind && m["name"]:"" == current_name);}); > + new_entry["kind"] = (string) UI::QueryWidget(`id(`rb), `CurrentButton); > + new_entry["name"] = (string) UI::QueryWidget(`id(`name), `Value); > + if (ret == `add) > + { > + initial_set = add (initial_set, new_entry); > + } > + break; > + } > + else if (ret2 == `cancel) > + { > + break; > + } > + } > + UI::CloseDialog (); > + } else if (ret == `delete) { > + integer current_item = (integer)UI::QueryWidget(`id(`id_adv_subject_alt_name), `CurrentItem); > + term table_item = (term) UI::QueryWidget(`id(`id_adv_subject_alt_name), `Item(current_item)); > + > + if (table_item == nil) > + { > + Popup::Error (_("No item has been selected.")); > + continue; > + } > + string current_kind = (string) table_item[1]:""; > + string current_name = (string) table_item[2]:""; Actually 'foo[index]:default' already includes casting the result to the type of 'default', so (string) is superfluous here. > + > + if (Popup::YesNoHeadline ( > + // To translators: ContinueCancel Popup headline > + _("Delete"), > + // To translators: ContinueCancel Popup > + _("Really delete this entry?"))) > + { > + initial_set = filter( map element, initial_set, ``{ > + return (element["kind"]:"" != current_kind || element["name"]:"" != current_name ); > + }); > + } > + } > + } while ( ret != `ok && ret != `cancel ); ^^^ yuck. do we have a plan how to improve this "best UI practice"? > + boolean ca_crit = (boolean) UI::QueryWidget(`id(`id_adv_cri_subject_alt_name), `Value); > + UI::CloseDialog(); > + return (ret == `ok) ? [initial_set,ca_crit] : nil; > + } > > > /** > @@ -97,7 +242,7 @@ > ), > `HBox(`HWeight (1,`TextEntry(`id(`email), _("E-Mail"), CaMgm::prop_email)), > `HSpacing(2), > - `HWeight (1,`Empty()) > + `HWeight (1,`PushButton(`id(`alt),"&Edit Alternative Names")) > ) > ); > > @@ -119,12 +264,21 @@ > // Get the user input. > // > any ret = nil; > + list<map> temp_alt_names = CaMgm::prop_subject_alt_name_list; > + boolean temp_cri_alt = CaMgm::prop_adv_cri_issuer_alt_name; > > repeat > { > ret = Wizard::UserInput(); > > - if (ret == `next > + if (ret == `alt) > + { > + any ret = editAltNames(temp_alt_names); hey! how about "list names_and_criminalsorwhatever = ..." ? > + if (ret != nil) > + temp_alt_names = ((list)ret)[0]:temp_alt_names; > + temp_cri_alt = ((list)ret)[1]:temp_cri_alt; > + } > + else if (ret == `next > ||ret == `back) > { > confirmPassword = (string) UI::QueryWidget(`id(`pw2), `Value); > @@ -183,6 +337,9 @@ > CaMgm::prop_organisationUnit = (string) UI::QueryWidget(`id(`id_organisationUnit), `Value); > CaMgm::prop_locality = (string) UI::QueryWidget(`id(`id_locality), `Value); > CaMgm::prop_state = (string) UI::QueryWidget(`id(`id_state), `Value); > + CaMgm::prop_subject_alt_name_list = temp_alt_names; > + CaMgm::prop_adv_cri_issuer_alt_name = temp_cri_alt; > + y2milestone("alt names %1", CaMgm::prop_subject_alt_name_list); > } > } > } > > Modified: trunk/ca-management/src/new_cert_read_write.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/ca-management/src/new_cert_read_… > ============================================================================== > --- trunk/ca-management/src/new_cert_read_write.ycp (original) > +++ trunk/ca-management/src/new_cert_read_write.ycp Tue Jun 14 14:53:17 2011 > @@ -760,6 +760,7 @@ > foreach(map elem, CaMgm::prop_subject_alt_name_list, { > CaMgm::adv_subject_alt_name_list = add (CaMgm::adv_subject_alt_name_list, elem); > }); > + CaMgm::adv_cri_issuer_alt_name = CaMgm::prop_adv_cri_issuer_alt_name; > } > if (kind == "Client Certificate") > { > > -- > To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org > For additional commands, e-mail: yast-commit+help(a)opensuse.org > -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

3 2

[yast-devel] Re: [yast-commit] r64372 - /trunk/cluster/src/Cluster.ycp
by Martin Vidner 15 Jun '11

15 Jun '11

On Wed, Jun 15, 2011 at 03:23:29AM -0000, xwhu(a)svn2.opensuse.org wrote: > Author: xwhu > Date: Wed Jun 15 05:23:29 2011 > New Revision: 64372 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64372&view=rev > Log: > Fix quoting issues > > Modified: > trunk/cluster/src/Cluster.ycp > > Modified: trunk/cluster/src/Cluster.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/cluster/src/Cluster.ycp?rev=6437… > ============================================================================== > --- trunk/cluster/src/Cluster.ycp (original) > +++ trunk/cluster/src/Cluster.ycp Wed Jun 15 05:23:29 2011 > @@ -438,10 +438,10 @@ > sleep(sl); > > if (corokey != "") { > - map out = (map)SCR::Execute(.target.bash_output, "echo " + corokey + " | uudecode -o /etc/corosync/authkey"); > + map out = (map)SCR::Execute(.target.bash_output, "echo '" + corokey + "' | uudecode -o /etc/corosync/authkey"); if corokey can include apostrophes, then use: "echo '" + String::Quote(corokey) + "' |..." > } > if (csync2key != "") { > - map out = (map)SCR::Execute(.target.bash_output, "echo " + csync2key + " | uudecode -o /etc/corosync/authkey"); > + map out = (map)SCR::Execute(.target.bash_output, "echo '" + csync2key + "' | uudecode -o /etc/corosync/authkey"); > } > if (global_startopenais == true) { > SCR::Execute(.target.bash_output, "/sbin/chkconfig openais on"); > > -- > To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org > For additional commands, e-mail: yast-commit+help(a)opensuse.org > -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

1 0

[yast-devel] Re: [yast-commit] r64362 - in /trunk/cluster: agents/Makefile.am src/Cluster.ycp src/common.ycp src/dialogs.ycp src/helps.ycp src/wizards.ycp
by Martin Vidner 15 Jun '11

15 Jun '11

On Wed, Jun 15, 2011 at 03:14:11AM -0000, xwhu(a)svn2.opensuse.org wrote: > Author: xwhu > Date: Wed Jun 15 05:14:10 2011 > New Revision: 64362 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64362&view=rev > Log: > Add support of csync2 > > Modified: > trunk/cluster/agents/Makefile.am > trunk/cluster/src/Cluster.ycp > trunk/cluster/src/common.ycp > trunk/cluster/src/dialogs.ycp > trunk/cluster/src/helps.ycp > trunk/cluster/src/wizards.ycp > > Modified: trunk/cluster/agents/Makefile.am > URL: http://svn.opensuse.org/viewcvs/yast/trunk/cluster/agents/Makefile.am?rev=6… > ============================================================================== > --- trunk/cluster/agents/Makefile.am (original) > +++ trunk/cluster/agents/Makefile.am Wed Jun 15 05:14:10 2011 > @@ -2,6 +2,7 @@ > > agent_SCRIPTS = ag_openais > > -scrconf_DATA = openais.scr > +scrconf_DATA = openais.scr \ > + csync2_ha.scr > > EXTRA_DIST = $(agent_SCRIPTS) $(scrconf_DATA) > > Modified: trunk/cluster/src/Cluster.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/cluster/src/Cluster.ycp?rev=6436… > ============================================================================== > --- trunk/cluster/src/Cluster.ycp (original) > +++ trunk/cluster/src/Cluster.ycp Wed Jun 15 05:14:10 2011 > @@ -42,6 +42,9 @@ > import "PackageSystem"; > import "SuSEFirewall"; > > + > +global string csync2_key_file = "/etc/csync2/key_hagroup"; > + > /** > * Prototypes > */ > @@ -233,6 +236,30 @@ > SCR::Write(.openais, ""); > } > > +global list<string> csync2_host = []; > +global list<string> csync2_include = []; > + > +global void load_csync2_conf() > +{ > + csync2_host = (list<string>)SCR::Read(.csync2_ha.value.ha_group.host); > + csync2_include = (list<string>)SCR::Read(.csync2_ha.value.ha_group.include); > + > + if (csync2_host == nil) csync2_host = []; > + if (csync2_include == nil) csync2_include = []; > + y2milestone("read csync2 conf: csync2_host = %1", csync2_host); > + y2milestone("read csync2 conf: csync2_include = %1", csync2_include); > +} > + > +global void save_csync2_conf() > +{ > + y2milestone("write csync2 conf: csync2_host = %1", csync2_host); > + y2milestone("write csync2 conf: csync2_include = %1", csync2_include); > + > + SCR::Write(.csync2_ha.value.ha_group.host, csync2_host); > + SCR::Write(.csync2_ha.value.ha_group.include, csync2_include); > + SCR::Write(.csync2_ha.value.ha_group.key, [csync2_key_file]); > +} > + > /** > * Read all cluster settings > * @return true on success > @@ -271,7 +298,7 @@ > ); > > boolean ret = false; > - list<string> required_pack_list = ["pacemaker", "pacemaker-pygui"]; > + list<string> required_pack_list = ["pacemaker", "pacemaker-pygui", "csync2"]; > ret = PackageSystem::CheckAndInstallPackagesInteractive(required_pack_list); > if (ret == false) { > Report::Error(_("Cannot install required package")); > @@ -295,6 +322,7 @@ > if(false) Report::Error(_("Cannot read database1.")); > sleep(sl); > > + load_csync2_conf(); Please keep the established indentation rules: four columns per level, \t is 8 columns wide, preferably use 8 spaces instead. > // read another database > if(Abort()) return false; > Progress::NextStep(); > @@ -385,6 +413,8 @@ > SuSEFirewall::AddService(mcastport2, "UDP", "EXT"); > } > > + save_csync2_conf(); > + > // run SuSEconfig > SuSEFirewall::Write(); > if(Abort()) return false; > > Modified: trunk/cluster/src/common.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/cluster/src/common.ycp?rev=64362… > ============================================================================== > --- trunk/cluster/src/common.ycp (original) > +++ trunk/cluster/src/common.ycp Wed Jun 15 05:14:10 2011 > @@ -14,7 +14,8 @@ > list<string> DIALOG = [ > "communication", > "security", > - "service" > + "service", > + "csync2" > ]; > > map PARENT = $[ > @@ -23,7 +24,8 @@ > map NAME = $[ > "communication": _("Communication Channels"), > "security": _("Security"), > - "service": _("Service") > + "service": _("Service"), > + "csync2": _("Configure Csync2") > ]; > > void myHelp(string help) > > Modified: trunk/cluster/src/dialogs.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/cluster/src/dialogs.ycp?rev=6436… > ============================================================================== > --- trunk/cluster/src/dialogs.ycp (original) > +++ trunk/cluster/src/dialogs.ycp Wed Jun 15 05:14:10 2011 > @@ -45,6 +45,32 @@ > include "cluster/helps.ycp"; > include "cluster/common.ycp"; > > +list<string> csync2_suggest_files = []; > +string csync2_port = "30865"; > + > +// return `cacel or a string > +any text_input_dialog(string title, string value) > +{ > + any ret = nil; > + > + UI::OpenDialog( > + `MarginBox(1, 1, `VBox( > + `MinWidth(100, `TextEntry(`id(`text), title, value)), > + `VSpacing(1), > + `Right(`HBox( > + `PushButton(`id(`ok), _("OK")), > + `PushButton(`id(`cancel), _("Cancel")) > + )) > + )) > + ); > + > + ret = UI::UserInput(); > + if (ret == `ok) ret = UI::QueryWidget(`text, `Value); > + UI::CloseDialog(); > + return ret; > +} We should move this to Popup::TextEntry, improving it to return a string (nil on cancel). (really we don't have this already somewhere?) > + > + > boolean ValidateCommunication () > { > if (IP::Check((string)UI::QueryWidget(`id(`bindnetaddr1), `Value)) == false) { > @@ -554,6 +580,230 @@ > return ret; > } > > + > +term csync2_layout() > +{ > + return > + `VBox(`opt(`hvstretch), > + `HBox( > + `Frame(_("Sync Host"), > + `VBox( > + `SelectionBox(`id(`host_box), ""), > + `HBox( > + `PushButton(`id(`host_add), "Add"), > + `PushButton(`id(`host_del), "Del"), > + `PushButton(`id(`host_edit), "Edit") > + ) > + )), > + `HSpacing(), > + `Frame(_("Sync File"), > + `VBox( > + `SelectionBox(`id(`include_box), ""), > + `HBox( > + `PushButton(`id(`include_add), "Add"), > + `PushButton(`id(`include_del), "Del"), > + `PushButton(`id(`include_edit), "Edit"), > + `PushButton(`id(`include_suggest), "Add Suggested Files") Translation marks missing > + ) > + )) > + ), > + `HBox( > + `PushButton(`id(`generate_key), `opt(`hstretch), _("Generate Pre-Shared-Keys")), > + `PushButton(`id(`csync2_switch), `opt(`hstretch), "") > + ) > + ); > +} > + > + > +// return 1 if csync2 is not installed well > +// return 2 if csync2 is OFF > +// return 3 if csync2 is ON wow. how about boolean: nil, false, true? > +integer csync2_status() > +{ > + map ret = nil; > + > + ret = (map)SCR::Execute(.target.bash_output, "/sbin/chkconfig csync2"); see Service::Enabled > + y2milestone("chkconfig csync2 = %1", ret); > + if (issubstring(ret["stderr"]:"", "command not found") == true) return 1; > + if (issubstring(ret["stderr"]:"", "unknown service") == true) return 1; > + if (issubstring(ret["stdout"]:"", "off") == true) return 2; > + > + return 3; > +} > + > +void csync2_turn_off() > +{ > + SCR::Execute(.target.bash_output, "/sbin/chkconfig csync2 off"); Service::Disable > + if (SuSEFirewall::HaveService(csync2_port, "TCP", "EXT")) > + SuSEFirewall::RemoveService(csync2_port, "TCP", "EXT"); > +} > + > +void csync2_turn_on() > +{ > + SCR::Execute(.target.bash_output, "/sbin/chkconfig csync2 on"); Service::Enable > + if (!SuSEFirewall::HaveService(csync2_port, "TCP", "EXT")) > + SuSEFirewall::AddService(csync2_port, "TCP", "EXT"); > +} > + > +void fill_csync_entries() > +{ > + integer i = 0; > + integer ret = 0; > + integer current = 0; > + list<term> items = []; > + > + // remove duplicated elements > + Cluster::csync2_host = Cluster::csync2_host + []; > + Cluster::csync2_include = Cluster::csync2_include + []; > + > + i = 0; > + items = []; > + foreach(string value, Cluster::csync2_host, { > + items = add(items, `item(`id(i), value)); > + i = i + 1; > + }); > + current = (integer)UI::QueryWidget(`host_box, `CurrentItem); > + if (current == nil) current = 0; > + if (current >= i) current = i - 1; > + UI::ChangeWidget(`host_box, `Items, items); > + UI::ChangeWidget(`host_box, `CurrentItem, current); > + > + i = 0; > + items = []; > + foreach(string value, Cluster::csync2_include, { > + items = add(items, `item(`id(i), value)); > + i = i + 1; > + }); > + current = (integer)UI::QueryWidget(`include_box, `CurrentItem); > + if (current == nil) current = 0; > + if (current >= i) current = i - 1; > + UI::ChangeWidget(`include_box, `Items, items); > + UI::ChangeWidget(`include_box, `CurrentItem, current); > + > + ret = csync2_status(); > + UI::ChangeWidget(`id(`csync2_switch), `Enabled, (ret != 1)); > + if (ret == 1) > + UI::ChangeWidget(`id(`csync2_switch), `Label, "Csync2 Status Unknown"); > + if (ret == 2) > + UI::ChangeWidget(`id(`csync2_switch), `Label, "Turn csync2 ON"); > + if (ret == 3) > + UI::ChangeWidget(`id(`csync2_switch), `Label, "Turn csync2 OFF"); _() missing > +} > + > + > +any Csync2Dialog() > +{ > + any ret = nil; > + > + my_SetContents("csync2", csync2_layout()); > + > + > + while(true) { > + fill_csync_entries(); > + > + ret = UI::UserInput(); > + > + if (ret == `abort || ret == `cancel) { > + if (ReallyAbort()) break; > + continue; > + } > + > + if (ret == `next || ret == `back) { > + break; > + } > + > + if (ret == `wizardTree) > + ret = (string)UI::QueryWidget(`id(`wizardTree), `CurrentItem); > + > + if (contains(DIALOG, (string)ret)) { > + ret = symbolof(toterm(ret)); > + //SaveCsync2(); > + break; > + } else { > + Wizard::SelectTreeItem("csync2"); > + continue; > + } > + > + if (ret == `host_add) { > + ret = text_input_dialog(_("Input a new host"), ""); > + if (ret == `cancel) continue; > + Cluster::csync2_host = add(Cluster::csync2_host, (string)ret); > + } > + > + if (ret == `host_edit) { > + integer current = 0; > + string str = ""; > + > + current = (integer)UI::QueryWidget(`host_box, `CurrentItem); > + ret = text_input_dialog(_("Edit a host"), Cluster::csync2_host[current]:""); > + if (ret == `cancel) continue; > + Cluster::csync2_host[current] = (string)ret; > + } > + > + if (ret == `host_del) { > + integer current = 0; > + current = (integer)UI::QueryWidget(`host_box, `CurrentItem); > + Cluster::csync2_host = remove(Cluster::csync2_host, current); > + } > + > + if (ret == `include_add) { > + ret = text_input_dialog(_("Input a new sync file"), ""); > + if (ret == `cancel) continue; > + Cluster::csync2_include = add(Cluster::csync2_include, (string)ret); > + } > + > + if (ret == `include_edit) { > + integer current = 0; > + > + current = (integer)UI::QueryWidget(`include_box, `CurrentItem); > + ret = text_input_dialog(_("Edit a sync file"), Cluster::csync2_include[current]:""); > + if (ret == `cancel) continue; > + Cluster::csync2_include[current] = (string)ret; > + } > + > + if (ret == `include_del) { > + integer current = 0; > + current = (integer)UI::QueryWidget(`include_box, `CurrentItem); > + Cluster::csync2_include = remove(Cluster::csync2_include, current); > + } > + > + if (ret == `include_suggest) { > + Cluster::csync2_include = Cluster::csync2_include + csync2_suggest_files; > + } > + > + if (ret == `generate_key) { > + string key_file = Cluster::csync2_key_file; > + > + // key file exist > + if (SCR::Read(.target.size, key_file) > 0) { > + if (!Popup::YesNo(sformat(_("Key file %1 already exist.\nDo you want to overwrite it?"), key_file))) > + continue; > + > + // remove exist key file > + if (SCR::Execute(.target.remove, key_file) == false) { > + Popup::Message(sformat(_("Delete key file %1 failed."), key_file)); > + continue; > + } > + } > + > + // generate key file > + ret = SCR::Execute(.target.bash, sformat("csync2 -k %1", key_file)); > + if (ret == 0) > + Popup::Message(sformat(_("Key file %1 is generated.\nPlease copy this file to all members of the cluster."), key_file)); > + else > + Popup::Message("Key generation failed."); _() > + } > + > + if (ret == `csync2_switch) { > + string label = ""; > + label = (string)UI::QueryWidget(`csync2_switch, `Label); > + if (issubstring(label, "OFF")) csync2_turn_off(); > + if (issubstring(label, "ON")) csync2_turn_on(); > + } > + } > + > + return ret; > +} > /* EOF */ > } > > > Modified: trunk/cluster/src/helps.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/cluster/src/helps.ycp?rev=64362&… > ============================================================================== > --- trunk/cluster/src/helps.ycp (original) > +++ trunk/cluster/src/helps.ycp Wed Jun 15 05:14:10 2011 > @@ -50,6 +50,13 @@ > <big>Booting</big> Starting openais service during boot or not > <big>Start mgmtd</big> Mgmtd daemon is a standalone service daemon. It is required by the GUI > "), > + > + "csync2" : _(" > + <big>Sync Host</big> The hostnames used here must be the local hostnames of the cluster nodes. That means you must use exactly the same string as printed out by the hostname command. > + <big>Sync File</big> The full absolute filename to be synced. > + <big>Pre-Shared-Keys</big> Authentication is performed using the IP addresses and pre-shared-keys in Csync2. The key file is generated with csync2 -k /etc/csync2/key_hagroup. The file key_hagroup should be copied to all members of the cluster manually after it's created. > + "), > + > /* Read dialog help 1/2 */ > "read" : _("<big>Initializing cluster Configuration</big> > Please wait... > > Modified: trunk/cluster/src/wizards.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/cluster/src/wizards.ycp?rev=6436… > ============================================================================== > --- trunk/cluster/src/wizards.ycp (original) > +++ trunk/cluster/src/wizards.ycp Wed Jun 15 05:14:10 2011 > @@ -41,7 +41,8 @@ > map Aliases = $[ > "communication": ``(CommunicationDialog()), > "security": ``(SecurityDialog()), > - "service": ``(ServiceDialog()) > + "service": ``(ServiceDialog()), > + "csync2": ``(Csync2Dialog()) > ]; > > > @@ -86,7 +87,8 @@ > "ws_start" : "communication", > "communication": $[ `next:"security", `back:"communication", `abort:`abort ], > "security": $[ `next:"service", `back:"communication", `abort:`abort ], > - "service": $[ `next:`next, `back:"security", `abort:`abort ], > + "service": $[ `next:"csync2", `back:"security", `abort:`abort ], > + "csync2" : $[ `next:`next, `back:"service", `abort:`abort ], > ]; > > any ret = Sequencer::Run(Aliases, sequence); > > -- > To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org > For additional commands, e-mail: yast-commit+help(a)opensuse.org > -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

1 0

[yast-devel] Re: [yast-commit] r64385 - /trunk/ncurses/src/NCstyle.mono.h
by Martin Vidner 15 Jun '11

15 Jun '11

> URL: http://svn.opensuse.org/viewcvs/yast?rev=64385&view=rev > Log: > adjust value for progress bar background The commit message merely states WHAT is being changed, which is already apparent from the code itself. Please remember to say WHY a change is being made. > > Modified: > trunk/ncurses/src/NCstyle.mono.h > > Modified: trunk/ncurses/src/NCstyle.mono.h > URL: http://svn.opensuse.org/viewcvs/yast/trunk/ncurses/src/NCstyle.mono.h?rev=6… > ============================================================================== > --- trunk/ncurses/src/NCstyle.mono.h (original) > +++ trunk/ncurses/src/NCstyle.mono.h Wed Jun 15 11:47:03 2011 > @@ -93,7 +93,7 @@ > attrset->setAttr( NCstyle::RichTextIT, 0 | A_REVERSE ); > attrset->setAttr( NCstyle::RichTextBIT, 0 | A_REVERSE ); > attrset->setAttr( NCstyle::ProgbarCh, 0 | A_BOLD ); > - attrset->setAttr( NCstyle::ProgbarBgch, 4194481 ); > + attrset->setAttr( NCstyle::ProgbarBgch, 16160 ); > attrset->setAttr( NCstyle::TextCursor, 0 | A_REVERSE | A_BOLD ); > // InfoStyle > attrset = &styleSet[NCstyle::InfoStyle].getAttrLocal(); > @@ -156,7 +156,7 @@ > attrset->setAttr( NCstyle::RichTextIT, 0 | A_REVERSE ); > attrset->setAttr( NCstyle::RichTextBIT, 0 | A_REVERSE ); > attrset->setAttr( NCstyle::ProgbarCh, 0 | A_BOLD ); > - attrset->setAttr( NCstyle::ProgbarBgch, 4194481 ); > + attrset->setAttr( NCstyle::ProgbarBgch, 16160 ); > attrset->setAttr( NCstyle::TextCursor, 0 | A_REVERSE | A_BOLD ); > // WarnStyle > attrset = &styleSet[NCstyle::WarnStyle].getAttrLocal(); > @@ -219,7 +219,7 @@ > attrset->setAttr( NCstyle::RichTextIT, 0 | A_REVERSE ); > attrset->setAttr( NCstyle::RichTextBIT, 0 | A_REVERSE ); > attrset->setAttr( NCstyle::ProgbarCh, 0 | A_BOLD ); > - attrset->setAttr( NCstyle::ProgbarBgch, 4194481 ); > + attrset->setAttr( NCstyle::ProgbarBgch, 16160 ); > attrset->setAttr( NCstyle::TextCursor, 0 | A_REVERSE | A_BOLD ); > // PopupStyle > attrset = &styleSet[NCstyle::PopupStyle].getAttrLocal(); > @@ -282,7 +282,7 @@ > attrset->setAttr( NCstyle::RichTextIT, 0 | A_REVERSE ); > attrset->setAttr( NCstyle::RichTextBIT, 0 | A_REVERSE ); > attrset->setAttr( NCstyle::ProgbarCh, 0 | A_BOLD ); > - attrset->setAttr( NCstyle::ProgbarBgch, 4194481 ); > + attrset->setAttr( NCstyle::ProgbarBgch, 16160 ); > attrset->setAttr( NCstyle::TextCursor, 0 | A_REVERSE | A_BOLD ); > } > > > -- > To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org > For additional commands, e-mail: yast-commit+help(a)opensuse.org > -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

1 0

[yast-devel] Re: [yast-commit] r64329 - /trunk/yast2/library/control/src/ProductProfile.ycp
by Martin Vidner 15 Jun '11

15 Jun '11

On Tue, Jun 14, 2011 at 06:43:07AM -0000, jsuchome(a)svn2.opensuse.org wrote: > Author: jsuchome > Date: Tue Jun 14 08:43:07 2011 > New Revision: 64329 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64329&view=rev > Log: > module for handling product profiles (fate#310730) > > Added: > trunk/yast2/library/control/src/ProductProfile.ycp (with props) > > Added: trunk/yast2/library/control/src/ProductProfile.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/yast2/library/control/src/Produc… > ============================================================================== > --- trunk/yast2/library/control/src/ProductProfile.ycp (added) > +++ trunk/yast2/library/control/src/ProductProfile.ycp Tue Jun 14 08:43:07 2011 > @@ -0,0 +1,153 @@ > +/** > + * File: modules/ProductProfile.ycp > + * Package: yast2 > + * Summary: Functions for handling Product Profiles > + * Authors: Jiri Suchomel <jsuchome(a)suse.cz> > + * > + * $Id$ > + */ > +{ > + textdomain "base"; > + module "ProductProfile"; > + > + import "Mode"; > + import "Report"; > + import "YaPI::SubscriptionTools"; Where is this module? I guess yast2.rpm should have a dependency on it. > + > + // path to the profile file on the media > + string profile_path = "/profile.prod"; //FIXME the name > + > + // Result map of isCompliance call. > + // If map is not empty, contains reasons why system is not compliant. > + map<string,any> compliance = $[]; > + > + // profiles for all installed products From an earlier commit, it is apparent that the strings are filenames (as opposed to contents of the files). Please mention that in the comment. How is the variable initialized? Wouldn't it be better to use a function, like GetComplianceMap does with compliance? > + global list<string> all_profiles = []; > + > + // return the result of last compliance test > + global map<string,any> GetComplianceMap () { > + return compliance; > + } > + > + /** > + * Checks the profile compliance with the system. > + * @param if productId is not nil, check only compliance with given product > + * (once new product is added, function should be called to with new product ID) > + * @ret true if the system is compliant > + */ > + global boolean IsCompliant (integer productId) { > + > + list<string> profiles = []; > + list<map> products = []; > + list<string> sigkeys = []; > + > + // iterate all (or given) products and get the info about them > + foreach (map product, Pkg::ResolvableProperties ("", `product, ""), { > + > + integer src_id = product["source"]:-1; > + > + if (product["status"]:`none != `selected) > + return; > + > + if (productId != nil && src_id != productId) > + return; > + > + // generate product map: > + list<string> version_release = splitstring (product["version"]:"", "-"); > + products = add (products, $[ > + "arch" : product["arch"]:"", > + "name" : product["name"]:"", > + "version" : version_release[0]:"", > + "release" : version_release[1]:"", > + "vendor" : product["vendor"]:"" > + ]); > + > + // FIXME it might be needed to check to directory.yast as well for *.profile > + string profile = Pkg::SourceProvideOptionalFile (src_id, 1, profile_path); > + if (profile != nil) > + { > + profiles = add (profiles, profile); > + } > + > + // find the list of sigkeys Comments like these are asking for a function: list<string> SourceSigkeys (integer src_id) { ... } > + string dir_file = Pkg::SourceProvideOptionalFile (src_id, 1, "/directory.yast"); > + map out = (map) SCR::Execute (.target.bash_output, sformat ( > + "grep 'gpg-pubkey' %1 2>/dev/null", dir_file)); > + list<string> keys = filter (string k, splitstring (out["stdout"]:"", "\n"), ``(k != "")); > + > + y2internal ("keys for %1: %2", src_id, keys); > + > + sigkeys = (list<string>) union (sigkeys, keys); > + }); > + > + // remember profiles so they can be copied them to the installed system > + all_profiles = union (all_profiles, profiles); > + > + y2internal ("products: %1", products); > + > + if (profiles == []) > + { > + y2milestone ("no product profile present"); > + compliance = $[]; > + return true; > + } > + > + compliance = YaPI::SubscriptionTools::isCompliant (profiles, products, sigkeys); > + return (compliance == nil); > + } > + > + > + /** > + * Checks the profile compliance with the system. > + * If system is not complient, shows a popup with reasons and asks > + * to continue with the installation. > + * @ret Returns true if system is complient or user agrees to continue > + * although the complience test failed. > + * @param if productId is not nil, check only compliance with given product > + * (once new product is added, function should be called to with new product ID) > + */ > + global boolean CheckCompliance (integer productId) { > + > + // behavior for non-installation not defined yet > + if (!Mode::installation ()) > + { > + return true; > + } > + > + if (IsCompliant (productId)) > + { > + return true; > + } > + > + list<string> reasons = []; > + foreach (string key, any val, compliant, { > + y2internal ("key %1, val %2", key, val); > + if (is (val, map) && haskey ((map)val, "message")) > + { > + reasons = add (reasons, ((map)val)["message"]:""); > + } > + }); > + list<string> reasons = [ > + "Architecture(x86_64) is not one of: s390x, i586", > + "Number of CPU cores(2) exceeds 1" > + ]; Are you sure to redeclare reasons? Is this a forgotten example list? > + > + return Report::AnyQuestion ( > + // popup dialog caption > + _("Warning"), > + // popup message, %1 is list of problems > + sformat (_("The system is not compliant with provided profile: > + > +%1 > + > +Do you want to continue or abort the installation?"), > + mergestring (reasons, "\n") > + ), > + // button label > + _("&Continue Installation"), > + // button label > + _("&Abort Installation"), > + `no_button > + ); > + } > +} > > -- > To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org > For additional commands, e-mail: yast-commit+help(a)opensuse.org > -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

3 4

[yast-devel] Re: [yast-commit] r64357 - in /branches/SuSE-Code-11-SP2-Branch/apparmor: VERSION package/yast2-apparmor.changes
by Martin Vidner 14 Jun '11

14 Jun '11

On Tue, Jun 14, 2011 at 01:54:40PM -0000, jreidinger(a)svn2.opensuse.org wrote: > Author: jreidinger > Date: Tue Jun 14 15:54:40 2011 > New Revision: 64357 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64357&view=rev > Log: > changes > > Modified: > branches/SuSE-Code-11-SP2-Branch/apparmor/VERSION > branches/SuSE-Code-11-SP2-Branch/apparmor/package/yast2-apparmor.changes > > Modified: branches/SuSE-Code-11-SP2-Branch/apparmor/VERSION > URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/appar… > ============================================================================== > --- branches/SuSE-Code-11-SP2-Branch/apparmor/VERSION (original) > +++ branches/SuSE-Code-11-SP2-Branch/apparmor/VERSION Tue Jun 14 15:54:40 2011 > @@ -1 +1 @@ > -2.17.8 > +2.17.9 > > Modified: branches/SuSE-Code-11-SP2-Branch/apparmor/package/yast2-apparmor.changes > URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/appar… > ============================================================================== > --- branches/SuSE-Code-11-SP2-Branch/apparmor/package/yast2-apparmor.changes (original) > +++ branches/SuSE-Code-11-SP2-Branch/apparmor/package/yast2-apparmor.changes Tue Jun 14 15:54:40 2011 > @@ -1,6 +1,13 @@ > ------------------------------------------------------------------- > Thu Mar 11 15:47:28 CET 2010 - kmachalkova(a)suse.cz not really > +- Merge apparmor module into one desktop entry (FATE#305278) > +- fix typos in translatable strings (BNC#693326 ) 859! > +- 2.17.9 > + > +------------------------------------------------------------------- > +Thu Mar 11 15:47:28 CET 2010 - kmachalkova(a)suse.cz > + > - Backport of bnc#538245 bugfix. This solves untranslated tooltips > of AA modules (now bnc#583643) > - 2.17.8 > > -- > To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org > For additional commands, e-mail: yast-commit+help(a)opensuse.org > -- Martin Vidner, YaST developer ( http://xkcd.com/859/ Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

1 0

[yast-devel] Re: [yast-commit] r64343 - in /trunk/wagon: ./ VERSION package/yast2-wagon.changes src/clients/wagon_update_proposal.ycp src/config/cd_update.desktop src/modules/Wagon.ycp yast2-wagon.spec.in
by Martin Vidner 14 Jun '11

14 Jun '11

On Tue, Jun 14, 2011 at 12:00:23PM -0000, lslezak(a)svn2.opensuse.org wrote: > Author: lslezak > Date: Tue Jun 14 14:00:22 2011 > New Revision: 64343 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64343&view=rev > Log: > - implemented download in advance mode support (fate#308951) > - 2.21.0 > > Modified: > trunk/wagon/ (props changed) > trunk/wagon/VERSION > trunk/wagon/package/yast2-wagon.changes > trunk/wagon/src/clients/wagon_update_proposal.ycp > trunk/wagon/src/config/cd_update.desktop (props changed) > trunk/wagon/src/modules/Wagon.ycp (contents, props changed) > trunk/wagon/yast2-wagon.spec.in > > Modified: trunk/wagon/VERSION > URL: http://svn.opensuse.org/viewcvs/yast/trunk/wagon/VERSION?rev=64343&r1=64342… > ============================================================================== > --- trunk/wagon/VERSION (original) > +++ trunk/wagon/VERSION Tue Jun 14 14:00:22 2011 > @@ -1 +1 @@ > -2.20.3 > +2.21.0 > > Modified: trunk/wagon/package/yast2-wagon.changes > URL: http://svn.opensuse.org/viewcvs/yast/trunk/wagon/package/yast2-wagon.change… > ============================================================================== > --- trunk/wagon/package/yast2-wagon.changes (original) > +++ trunk/wagon/package/yast2-wagon.changes Tue Jun 14 14:00:22 2011 > @@ -1,4 +1,10 @@ > ------------------------------------------------------------------- > +Tue Jun 14 08:42:02 UTC 2011 - lslezak(a)suse.cz > + > +- implemented download in advance mode support (fate#308951) > +- 2.21.0 > + > +------------------------------------------------------------------- > Fri Feb 18 14:22:27 CET 2011 - locilka(a)suse.cz > > - yast2-wagon control files have been moved to separate packages > > Modified: trunk/wagon/src/clients/wagon_update_proposal.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/wagon/src/clients/wagon_update_p… > ============================================================================== > --- trunk/wagon/src/clients/wagon_update_proposal.ycp (original) > +++ trunk/wagon/src/clients/wagon_update_proposal.ycp Tue Jun 14 14:00:22 2011 > @@ -20,6 +20,7 @@ > > import "Wagon"; > import "Report"; > + import "PackageInstallation"; > > string func = (string) WFM::Args(0); > map param = (map) WFM::Args(1); > @@ -40,11 +41,23 @@ > } > > Wagon::ProposeDUP(); > + Wagon::ProposeDownloadMode(); > > ret = Wagon::ProposalSummary(); > > } else if (func == "AskUser") { > - Report::Message (_("There is nothing to set.")); > + string chosen_id = param["chosen_id"]:""; > + > + // toggle the download mode status > + if (chosen_id == Wagon::GetDownloadModeLink()) > + { > + PackageInstallation::SetDownloadInAdvance(!PackageInstallation::DownloadInAdvance()); > + } > + else > + { > + Report::Message (_("There is nothing to set.")); > + } > + > ret = $[ "workflow_sequence" : `next ]; > > } else if (func == "Description") { > > Modified: trunk/wagon/src/modules/Wagon.ycp > URL: http://svn.opensuse.org/viewcvs/yast/trunk/wagon/src/modules/Wagon.ycp?rev=… > ============================================================================== > --- trunk/wagon/src/modules/Wagon.ycp (original) > +++ trunk/wagon/src/modules/Wagon.ycp Tue Jun 14 14:00:22 2011 > @@ -31,6 +31,8 @@ > import "Update"; > import "FileUtils"; > import "String"; > + import "Packages"; > + import "PackageInstallation"; > > boolean running_by_applet = false; > > @@ -75,6 +77,13 @@ > return update_workflow_type; > } > > + const string download_mode_link = "wagon-download_in_advance"; > + > + global define string GetDownloadModeLink() > + { > + return download_mode_link; > + } > + > string migration_method = nil; > > global string migration_method_file = sformat ("%1/wagon_migration_method", Directory::vardir); > @@ -379,6 +388,68 @@ > return true; > } > > + string FindMountPoint(string dir, list<string> dirs) > + { > + while (dir != nil && dir != "" && !contains(dirs, dir)) > + { > + // strip the last path component and try it again > + list<string> comps = splitstring(dir, "/"); > + comps = remove(comps, size(comps) - 1); > + dir = mergestring(comps, "/"); > + } > + > + return dir; > + } ^ I *think* that this function fails if it should return "/", returning "" instead. Unit tests? A descriptive comment for the function, mentioning the corner cases? > + > + map<string, any> CheckDownloadSpace() > + { > + map<string, list> du = Pkg::TargetGetDU(); > + list<string> mounts = maplist(string dir, list info, du, {return dir;}); > + > + map<string,any> zconfig = Pkg::ZConfig(); > + string pkg_path = zconfig["repo_packages_path"]:""; > + > + string packages_mount = FindMountPoint(pkg_path, mounts); > + y2milestone("Packages will we downloaded to mountpoint %1", packages_mount); > + > + // download size in bytes > + integer download_size = Packages::CountSizeToBeDownloaded(); > + y2milestone("Size of packages to download: %1MB", download_size >> 20); > + > + // du contains maps: $[ "dir" : [ total, used, pkgusage, readonly ], .... ] > + integer after_install = du[packages_mount, 2]:0; > + integer total = du[packages_mount, 0]:0; > + > + symbol result = `ok; > + string message = ""; > + > + if (after_install + download_size < total) > + { > + result = `error; > + message = "Not enough free space"; _() missing? > + y2error("Not enough free space for download in advance upgrade"); > + } > + else if (after_install + download_size < total - 100<<20) > + { > + result = `warning; > + message = "Not enough free space"; _() ? > + y2error("Not enough free space for download in advance upgrade"); good luck with distinguishing the two situations in y2log some months later ;-) > + } > + > + return $[ "result" : result, "message" : message ]; > + } > + > + global void ProposeDownloadMode() > + { > + if (PackageInstallation::DownloadInAdvance() == nil) > + { > + map<string, any> dwspace = CheckDownloadSpace(); > + > + PackageInstallation::SetDownloadInAdvance(dwspace["result"]:nil == `ok); > + y2milestone("Proposed download in advance mode: %1", PackageInstallation::DownloadInAdvance()); > + } > + } > + > global map <string,any> MinimizeProductMap (map <string,any> product) { > if (haskey (product, "license")) product["license"] = "..."; > if (haskey (product, "description")) product["description"] = "..."; > @@ -510,9 +581,14 @@ > > ret = "<ul>\n" + ret + "</ul>\n"; > > + ret = ret + "<ul><li>\n" + _("Download all packages before upgrade: ") + > + sformat("<a href=\"%1\">%2</a>", download_mode_link, (PackageInstallation::DownloadInAdvance() ? _("Enabled") : _("Disabled"))) > + + "</li></ul>\n"; > + > map <string, any> summary = $[ > "preformatted_proposal" : ret, > - // help text > + "links" : [ download_mode_link ], > + // help text, TODO FIXME: replace the inner by WTF? Is this a broken text unfixable because it is translated already? OK, but this is the trunk. > "help" : _("To change the update settings, go to Packages Proposal section."), > ]; > > > Modified: trunk/wagon/yast2-wagon.spec.in > URL: http://svn.opensuse.org/viewcvs/yast/trunk/wagon/yast2-wagon.spec.in?rev=64… > ============================================================================== > --- trunk/wagon/yast2-wagon.spec.in (original) > +++ trunk/wagon/yast2-wagon.spec.in Tue Jun 14 14:00:22 2011 > @@ -10,7 +10,8 @@ > Requires: yast2-pkg-bindings >= 2.19.1 > > # Called in proposal and in code > -Requires: yast2-packager yast2-add-on > +Requires: yast2-packager >= 2.21.2 > +Requires: yast2-add-on > > # Counting packages directly in packages proposal (BNC #573482) > Requires: yast2-update >= 2.18.7 > > -- > To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org > For additional commands, e-mail: yast-commit+help(a)opensuse.org > -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

1 0

[yast-devel] Re: [yast-commit] r64312 - /branches/SuSE-Code-11-SP2-Branch/ldap-server/src/agent/SlapdConfigAgent.cc
by Martin Vidner 14 Jun '11

14 Jun '11

On Fri, Jun 10, 2011 at 09:08:26AM -0000, rhafer(a)svn2.opensuse.org wrote: > Author: rhafer > Date: Fri Jun 10 11:08:25 2011 > New Revision: 64312 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64312&view=rev > Log: > fixed return value for undefined update referral > > Modified: > branches/SuSE-Code-11-SP2-Branch/ldap-server/src/agent/SlapdConfigAgent.cc > > Modified: branches/SuSE-Code-11-SP2-Branch/ldap-server/src/agent/SlapdConfigAgent.cc > URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/ldap-… > ============================================================================== > --- branches/SuSE-Code-11-SP2-Branch/ldap-server/src/agent/SlapdConfigAgent.cc (original) > +++ branches/SuSE-Code-11-SP2-Branch/ldap-server/src/agent/SlapdConfigAgent.cc Fri Jun 10 11:08:25 2011 > @@ -961,10 +961,7 @@ > resMap.add( YCPString("target"), YCPString( updateUrl.getHost() ) ); > resMap.add( YCPString("port"), YCPInteger( updateUrl.getPort() ) ); > } > - else > - { > - resMap.add( YCPString("updateref"), YCPMap() ); > - } > + resMap = YCPNull(); > return resMap; > } This seems wrong, you're overwriting the resMap from the "if" branch. -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

2 1

[yast-devel] YaST SDK for SLE 11 SP2
by Josef Reidinger 14 Jun '11

14 Jun '11

Hi, I create SDK for development of YAST for SP2[0]. It helps to not mess your machine and also easier testing your module on SP1 machine, so you don't have problem with too much recent software. If you miss something just ping me. I welcome any comments, complains or questions. Pepa P.S. try checkout_yast_svn script for easy svn checkout P.P.S. - if your machine is public accessible don't forget to change password for root and tux user ( default is linux ) [0] http://susegallery.com/a/F0D5dp/josefs-sled-11-sp1-yast-sdk -- Josef Reidinger Appliance Toolkit team maintaining parts of webyast and SLMS author of rubygems - studio_api and net_observer (coauthor) -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

2 2

[yast-devel] Ruby Bindings: Examples
by Lukas Ocilka 13 Jun '11

13 Jun '11

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Hi, I've just tried all the examples from http://svn.opensuse.org/svn/yast/trunk/ruby-bindings/examples/ruby/ and it seems that none of them work. At least not for me. What's the current status of this package? What's the status of calling Ruby from YCP and vice versa? Thanks in advance Lukas - -- Lukas Ocilka, Appliances Department, SUSE LINUX s.r.o. MD: Jeff Hawn, Jennifer Guild, Alena Hendrichova -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/ iD8DBQFN9f+MVSqMdRCqTiwRAuy0AKCP8eHxxr/9wpDL3k1m0qeQ1znzawCfROSf qqz2WGl4cQlUId/vGS2zzQg= =+b9F -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

2 2

[yast-devel] Re: [yast-commit] r64314 - in /branches/SuSE-Code-11-SP2-Branch/autoinstallation: package/autoyast2.changes src/include/io.ycp
by Lukas Ocilka 13 Jun '11

13 Jun '11

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Dne 10.6.2011 15:22, ug(a)svn2.opensuse.org napsal(a): > Author: ug > Date: Fri Jun 10 15:22:36 2011 > New Revision: 64315 > > Modified: > branches/SuSE-Code-11-SP2-Branch/autoinstallation/package/autoyast2.changes > branches/SuSE-Code-11-SP2-Branch/autoinstallation/src/include/io.ycp > > Modified: branches/SuSE-Code-11-SP2-Branch/autoinstallation/package/autoyast2.changes > URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP2-Branch/autoi… > ============================================================================== > --- branches/SuSE-Code-11-SP2-Branch/autoinstallation/package/autoyast2.changes (original) > +++ branches/SuSE-Code-11-SP2-Branch/autoinstallation/package/autoyast2.changes Fri Jun 10 15:22:36 2011 > @@ -1,10 +1,4 @@ > ------------------------------------------------------------------- > -Fri Jun 10 15:06:40 CEST 2011 - ug(a)suse.de > - > -- merged the code for autoyast=device and autoyast=usb to reduce > - redundancy and fix the "already mounted" bug for USB (bnc#565558) > - > -------------------------------------------------------------------- Just an idea, while you are refactoring... wouldn't it be better to move a particular schema-related code to functions instead of having it in one big function (287 lines long)? if ( Scheme == "http" || Scheme == "https" ) { return GetHTTP(...); } else if ( Scheme == "ftp" ) { return GetFTP(...); } else if (Scheme == "file") { return GetFile(...); } else if ... This was almost the first thing I had to do with the code in YaST Backup as it used looooong `if, else if` statements with even looooonger `while` loops. That made the code uneasy to understand and debug. I know that YCP is not an object-oriented language and that makes is harder but it can still be done with, for instance, pointers to variables. BTW, splitting into more functions makes it easier to test too. Bye Lukas - -- Lukas Ocilka, Appliances Department, SUSE LINUX s.r.o. MD: Jeff Hawn, Jennifer Guild, Alena Hendrichova -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/ iD8DBQFN9b43VSqMdRCqTiwRAgvlAKCEK42IoZ7YEkyg+8V6iFy4OsP/AgCdHIQp XWQisjCpvBmCsl0vj9igrxE= =Pw4a -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] GsoC Weekly Report - Separating libYUI
by N B Prashanth 11 Jun '11

11 Jun '11

Hi, I am attaching my GsoC weekly report along with this mail. It can also be found at nbprashanth.wordpress.org (http://nbprashanth.wordpress.com/2011/06/11/gsoc-report-2/) ------------------------------------------------------------- Hi. The last week was full of changes for libYUI. To start off, the packages are no longer being built locally and the build process is now handled by OBS. This makes libYUI available on a wide range of operating systems and architectures. Currently, the build at OBS is being done for Fedora 14/15 as well as Ubuntu 10.10/11.04 for both x86 and x86_64 architectures. Interested people can find the OBS repository at http://download.opensuse.org/repositories/home:/nbprashanth/. This means that the binaries will no longer be availalbe at the gitorious repository(gitorious.org/libyui). Secondly, the initial documentation for the library is hosted at http://doc.opensuse.org/projects/libyui/HEAD/. This documentation is subjected to frequent updates and will change regularly (weekly updates planned). Again, to avoid duplications, the gitorious repository will not contain the documentation anymore. People who test the new packages are requested to submit bug reports (here or at OBS) in order to help improve libYUI. ------------------------------------------------------------- -- Regards, N.B.Prashanth -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] GsoC Weekly Report - Separating libYUI
by N.B.Prashanth 11 Jun '11

11 Jun '11

Hi, I am attaching my GsoC weekly report along with this mail. It can also be found at nbprashanth.wordpress.org (http://nbprashanth.wordpress.com/2011/06/11/gsoc-report-2/) ------------------------------------------------------------- Hi. The last week was full of changes for libYUI. To start off, the packages are no longer being built locally and the build process is now handled by OBS. This makes libYUI available on a wide range of operating systems and architectures. Currently, the build at OBS is being done for Fedora 14/15 as well as Ubuntu 10.10/11.04 for both x86 and x86_64 architectures. Interested people can find the OBS repository at http://download.opensuse.org/repositories/home:/nbprashanth/. This means that the binaries will no longer be availalbe at the gitorious repository(gitorious.org/libyui). Secondly, the initial documentation for the library is hosted at http://doc.opensuse.org/projects/libyui/HEAD/. This documentation is subjected to frequent updates and will change regularly (weekly updates planned). Again, to avoid duplications, the gitorious repository will not contain the documentation anymore. People who test the new packages are requested to submit bug reports (here or at OBS) in order to help improve libYUI. ------------------------------------------------------------- -- Regards, N.B.Prashanth -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] Re: [yast-commit] r64276 - in /branches/tmp/lslezak/SP2/packager: agents/Makefile.am agents/etc_zypp_conf.scr package/yast2-packager.changes src/modules/PackageSlideShow.ycp src/modules/SlideShowCallbacks.ycp
by Martin Vidner 09 Jun '11

09 Jun '11

Code review! On Thu, Jun 09, 2011 at 12:16:30PM -0000, lslezak(a)svn2.opensuse.org wrote: > Author: lslezak > Date: Thu Jun 9 14:16:29 2011 > New Revision: 64276 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64276&view=rev > Log: > - correctly display download progress in DownloadInAdvance > libzypp mode (bnc#616708) (backported commits 62714, > partly 54940) > > Added: > branches/tmp/lslezak/SP2/packager/agents/etc_zypp_conf.scr > Modified: > branches/tmp/lslezak/SP2/packager/agents/Makefile.am > branches/tmp/lslezak/SP2/packager/package/yast2-packager.changes > branches/tmp/lslezak/SP2/packager/src/modules/PackageSlideShow.ycp > branches/tmp/lslezak/SP2/packager/src/modules/SlideShowCallbacks.ycp > > Added: branches/tmp/lslezak/SP2/packager/agents/etc_zypp_conf.scr > URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/lslezak/SP2/packager/agen… > ============================================================================== > --- branches/tmp/lslezak/SP2/packager/agents/etc_zypp_conf.scr (added) > +++ branches/tmp/lslezak/SP2/packager/agents/etc_zypp_conf.scr Thu Jun 9 14:16:29 2011 > @@ -0,0 +1,130 @@ > +/** > + * File: > + * etc_zypp_conf.scr > + * Summary: > + * SCR Agent for reading/writing /etc/zypp/zypp.conf file > + * using the ini-agent > + * Access: > + * read/write > + * Authors: > + * Ladislav Slezak <lslezak(a)suse.cz> > + * Example: > + * > + * Read list of available files > + * SCR::Dir(.etc.zypp_conf) > + * > + * Read list of sections in a file > + * SCR::Dir(.etc.policykit.section."/etc/PolicyKit/privilege.d/hal-power-hibernate.privilege") > + * (["Privilege"]) > + * > + * Read list of options in a section > + * SCR::Dir(.etc.policykit.value."/etc/PolicyKit/privilege.d/hal-power-hibernate.privilege"."Privilege") > + * (["RequiredPrivileges", "SufficientPrivileges", "Allow", "Deny", "CanObtain", "CanGrant", "ObtainRequireRoot"]) > + * > + * Read an option > + * SCR::Read(.etc.policykit.value."/etc/PolicyKit/privilege.d/hal-power-hibernate.privilege"."Privilege"."RequiredPrivileges") > + * ("desktop-console") > + * > + * > + * Write an option > + * SCR::Write(.etc.policykit.value."/etc/PolicyKit/privilege.d/hal-power-hibernate.privilege"."Privilege"."RequiredPrivileges", "") > + * (true) The above example usage is irrelevant. > + * > + * > + * Flush the changes > + * SCR::Write(.etc.zypp_conf, nil) > + * > + * > + * Fore more information see the INI-agent documentation > + * > + * $Id$ > + */ > + > +.etc.zypp_conf > + > +`ag_ini( > + `IniAgent( "/etc/zypp/zypp.conf" , $[ > + "options" : [ "no_nested_sections", "line_can_continue", "comments_last" ], > + "comments": [ > + "^[ \t]*$", // empty line > + "^[ \t]*#.*$", // comment char > + ], > + "sections" : [ > + $[ > + "begin" : [ "^[ \t]*\\[[ \t]*(.*[^ \t])[ \t]*\\][ \t]*", "[%s]" ], > + ] > + ], > + "params" : [ > + $[ > + "match" : [ "^[ \t]*([^=]*[^ \t=])[ \t]*=[ \t]*(.*[^ \t]|)[ \t]*$" , "%s=%s"], > + ] > + ] > + ] > + ) > +) > + The file contents is duplicated. > +/** > + * File: > + * etc_zypp_conf.scr [...] > + "match" : [ "^[ \t]*([^=]*[^ \t=])[ \t]*=[ \t]*(.*[^ \t]|)[ \t]*$" , "%s=%s"], > + ] > + ] > + ] > + ) > +) > + -- Martin Vidner, YaST developer http://en.opensuse.org/User:Mvidner Kuracke oddeleni v restauraci je jako fekalni oddeleni v bazenu

2 1

Re: [yast-devel] Re: [yast-commit] r64270 - in /trunk/theme: ./ openSUSE/icons/ openSUSE/icons/256x256/ openSUSE/icons/256x256/apps/ openSUSE/icons/64x64/apps/ openSUSE/icons/scalable/ openSUSE/icons/scalable/apps/ package/
by Thomas Goettlicher 09 Jun '11

09 Jun '11

On Thursday, June 09, 2011 11:43:54 am Lukas Ocilka wrote: > Dne 8.6.2011 13:32, tgoettlicher(a)svn2.opensuse.org napsal(a): > > Author: tgoettlicher > > Date: Wed Jun 8 13:32:10 2011 > > New Revision: 64270 > > > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64270&view=rev > > Log: > > - Include high resolution png images (256x256) and scalable svg > > > > images to make icons look better in gnome-shell > > > > ... > > > > ------------------------------------------------------------------- > > > > +Tue Jun 7 19:26:01 UTC 2011 - badshah400(a)gmail.com > > + > > +- Include high resolution png images (256x256) and scalable svg > > + images to make icons look better in gnome-shell > > +- 2.21.4 > > + > > +------------------------------------------------------------------- > > Thomas, If it is because of the gnome-shell, what do you think of > splitting some parts to a sub-package for decreasing the default size? The new 256x256 icons consume 8MB. I think it's not worth to split packages. Cheers, Thomas > Such sub-package would then be installed only if needed. > > Example usage (from YaST SVN): > Supplements: samba > Supplements: packageand(yast2-wagon:branding-openSUSE) > Supplements: packageand(yast2-wagon:branding-SLE) > Supplements: packageand(yast2-storage:yast2-qt) > Supplements: kdebase3 > Supplements: kdebase4-session > > Thanks and Bye > Lukas -- Thomas Goettlicher SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg) Maxfeldstraße 5 90409 Nürnberg Germany -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

2 1

[yast-devel] Re: [yast-commit] r64267 - in /branches/tmp/dmacvicar/zc10/ncurses/src: NCPkgTable.cc NCPkgTable.h
by Lukas Ocilka 09 Jun '11

09 Jun '11

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Dne 7.6.2011 17:13, dmacvicar(a)svn2.opensuse.org napsal(a): > Author: dmacvicar > Date: Tue Jun 7 17:13:36 2011 > New Revision: 64267 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64267&view=rev > Log: > remove unused code Yes, one of the things I forgot to mention in the 'Changing the Attitude to Coding YaST' as an easy hack is: * Removing unused code Good to see that someone is actually doing that! :) Thanks Lukas - -- Lukas Ocilka, Appliances Department, SUSE LINUX s.r.o. MD: Jeff Hawn, Jennifer Guild, Alena Hendrichova -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/ iD8DBQFN8JbjVSqMdRCqTiwRAiSNAJ9SmAwNaSdO1LmnJfA2x873cMwvAQCeNbty dGM7e/1sa3VxjmlaL/dIXSk= =FwnO -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] Re: [yast-commit] r64270 - in /trunk/theme: ./ openSUSE/icons/ openSUSE/icons/256x256/ openSUSE/icons/256x256/apps/ openSUSE/icons/64x64/apps/ openSUSE/icons/scalable/ openSUSE/icons/scalable/apps/ package/
by Lukas Ocilka 09 Jun '11

09 Jun '11

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Dne 8.6.2011 13:32, tgoettlicher(a)svn2.opensuse.org napsal(a): > Author: tgoettlicher > Date: Wed Jun 8 13:32:10 2011 > New Revision: 64270 > > URL: http://svn.opensuse.org/viewcvs/yast?rev=64270&view=rev > Log: > - Include high resolution png images (256x256) and scalable svg > images to make icons look better in gnome-shell > ... > ------------------------------------------------------------------- > +Tue Jun 7 19:26:01 UTC 2011 - badshah400(a)gmail.com > + > +- Include high resolution png images (256x256) and scalable svg > + images to make icons look better in gnome-shell > +- 2.21.4 > + > +------------------------------------------------------------------- Thomas, If it is because of the gnome-shell, what do you think of splitting some parts to a sub-package for decreasing the default size? Such sub-package would then be installed only if needed. Example usage (from YaST SVN): Supplements: samba Supplements: packageand(yast2-wagon:branding-openSUSE) Supplements: packageand(yast2-wagon:branding-SLE) Supplements: packageand(yast2-storage:yast2-qt) Supplements: kdebase3 Supplements: kdebase4-session Thanks and Bye Lukas - -- Lukas Ocilka, Appliances Department, SUSE LINUX s.r.o. MD: Jeff Hawn, Jennifer Guild, Alena Hendrichova -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.16 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/ iD8DBQFN8JXaVSqMdRCqTiwRAqyLAJ9t77qN1bjDiaVRLefLuWqU7Uw4YACeNCGa Sue782s29isr/Zyvioveyes= =IXui -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] Issues with WebYaST on 11.4
by Andrew Wafaa 08 Jun '11

08 Jun '11

Aloha all, Before I fire up bugzilla, I thought I would ask about some issues I'm having at the moment with WebYaST. On a new install of 11.4 I added the WebYaST repo, and then installed the webyast pattern. From what I can tell all went well. I started both yastws & yastwc without issue and connected via the browser. I was then greeted with a big fat red light advising me that "Registration is missing" and that No Status available". The status issue was easily resolved by restarting collectd, I have however not been able to resolve the Registration issue - is this a SLE item only and as such not needed on 11.4? I have clicked on Register and confirmed the details are correct, selected Save but I still get the error. Any advice please? Thanks, Andy -- Andrew Wafaa IRC: FunkyPenguin GPG: 0x3A36312F openSUSE: Get It, Discover It, Create It at http://www.opensuse.org -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

3 4

[yast-devel] Separation libYUI - GsoC Project Report
by N.B.Prashanth 07 Jun '11

07 Jun '11

Hi, I am working on the libYUI project this year under the mentorship of Martin Vidner. This is my report for the second week of GsoC (May 23 - June 04). I am maintaining a gitorious repository at gitorious.org/libyui and a blog at nbprashanth.wordpress.com. All weekly reports and other updates are updated on the blog as well. ----------------------------------------------------------------------- REPORT (As on the blog) This is my GsoC report for the first two weeks (May 23 – June 04). I will also in brief state the goals of the project and the future plans. My project for this years GsoC is “Separating libYUI” for the OpenSUSE organization. My mentor for the project is Martin Vidner and the abstract can be found at : http://www.google-melange.com/gsoc/project/google/gsoc2011/nbprashanth/22003 Let me begin by laying out the main goals of this project. The project aims to : Separate YUI from the YaST framework. Port YUI to other major Operating Systems such as Ubuntu and Fedora. Remove the libzypp dependency for the gtk pluging and hence making YUI SUSE-independent. Improve documentation with doxygen. Provide small tutorials. To provide a large and non-trivial example to show the capability of YUI. The work done so far is listed below : libYUI has been successfully compiled on Ubuntu 11.04 and Fedora 15. Initial packages for libYUI along with ncurses and Qt plugins have been made. They are available in my gitorious account : gitorious.org/libyui Small scale examples are being added to this blog. I will try to maintain a weekly schedule. Currently, I am trying to eliminate the libzypp dependency for the Gtk plugin. This is important as libzypp is highly OpenSUSE centric. The packages are currently in the testing phase. So all interested people, please feel free to test and post any bugs you may find in the comments section. Please note, this is an initial version. Patches have been made for small bugs like “long long” warnings and enumeration ending with a ‘,’. More serious bugs, if reported will be tended to when encountered. Another important change that I would like to implement is the use of a uniform build system. Currently, some parts use CMake while others use autotools. Though this does not make much of a difference to the outside world, I think it will be easier for developers and porters in the long run. But this will be done later on unless there is an urgent need for it. Since my end semester exams ended on May 27th, no work was done during the first week of GsoC. Hence, this can be regarded as a summary of my work for the second week (May 29 – June 04). Also, please note that I will maintain weekly reports on the blog henceforth. Please feel free to add comments and suggestions below. ----------------------------------------------------------------------- In addition to the above, i have uploaded the docs and a build script. The build script is just a simple shell script to automate building rpms. The reason for this is that I would be able to get a 64 bit rpm from volunteers and updating rpms later on would be easier. Similarly, I will work on 64 bit packages for Ubuntu soon. -- Regards, N.B.Prashanth -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

3 2

[yast-devel] Separation libYUI - GsoC Project Report
by N.B.Prashanth 06 Jun '11

06 Jun '11

Hi, I am working on the libYUI project this year under the mentorship of Martin Vidner. This is my report for the second week of GsoC (May 23 - June 04). I am maintaining a gitorious repository at gitorious.org/libyui and a blog at nbprashanth.wordpress.com. All weekly reports and other updates are updated on the blog as well. ----------------------------------------------------------------------- REPORT (As on the blog) This is my GsoC report for the first two weeks (May 23 – June 04). I will also in brief state the goals of the project and the future plans. My project for this years GsoC is “Separating libYUI” for the OpenSUSE organization. My mentor for the project is Martin Vidner and the abstract can be found at : http://www.google-melange.com/gsoc/project/google/gsoc2011/nbprashanth/22003 Let me begin by laying out the main goals of this project. The project aims to : 1. Separate YUI from the YaST framework. 2. Port YUI to other major Operating Systems such as Ubuntu and Fedora. 3. Remove the libzypp dependency for the gtk pluging and hence making YUI SUSE-independent. 4. Improve documentation with doxygen. 5. Provide small tutorials. 6. To provide a large and non-trivial example to show the capability of YUI. The work done so far is listed below : 1. libYUI has been successfully compiled on Ubuntu 11.04 and Fedora 15. 2. Initial packages for libYUI along with ncurses and Qt plugins have been made. They are available in my gitorious account : gitorious.org/libyui 3. Small scale examples are being added to this blog. I will try to maintain a weekly schedule. 4. Currently, I am trying to eliminate the libzypp dependency for the Gtk plugin. This is important as libzypp is highly OpenSUSE centric. The packages are currently in the testing phase. So all interested people, please feel free to test and post any bugs you may find in the comments section. Please note, this is an initial version. Patches have been made for small bugs like “long long” warnings and enumeration ending with a ‘,’. More serious bugs, if reported will be tended to when encountered. Another important change that I would like to implement is the use of a uniform build system. Currently, some parts use CMake while others use autotools. Though this does not make much of a difference to the outside world, I think it will be easier for developers and porters in the long run. But this will be done later on unless there is an urgent need for it. Since my end semester exams ended on May 27th, no work was done during the first week of GsoC. Hence, this can be regarded as a summary of my work for the second week (May 29 – June 04). Also, please note that I will maintain weekly reports on the blog henceforth. Please feel free to add comments and suggestions below. ----------------------------------------------------------------------- In addition to the above, i have uploaded the docs and a build script. The build script is just a simple shell script to automate building rpms. The reason for this is that I would be able to get a 64 bit rpm from volunteers and updating rpms later on would be easier. Similarly, I will work on 64 bit packages for Ubuntu soon. -- Regards, N.B.Prashanth -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

1 0

[yast-devel] liby2util-devel
by Togan Muftuoglu 06 Jun '11

06 Jun '11

Hi, According to <http://doc.opensuse.org/projects/YaST/openSUSE11.3/tutorials/general.html> liby2util-devel package is needed for development environment. However no such package exists according to zypper. toganm@desktop:~> sudo zypper se liby2util-devel Loading repository data... Reading installed packages... No packages found. Is the package renamed to something ? Thanks Togan -- To unsubscribe, e-mail: yast-devel+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-devel+help(a)opensuse.org

2 4