December 2009 - YaST Commits - openSUSE Mailing Lists

[yast-commit] r59909 - /branches/SuSE-Code-11-SP1-Branch/ldap-server/src/agent/SlapdConfigAgent.cc
by rhafer＠svn.opensuse.org 02 Dec '09

02 Dec '09

Author: rhafer Date: Wed Dec 2 16:57:48 2009 New Revision: 59909 URL: http://svn.opensuse.org/viewcvs/yast?rev=59909&view=rev Log: Don't write out portnumbers when using defaults Modified: branches/SuSE-Code-11-SP1-Branch/ldap-server/src/agent/SlapdConfigAgent.cc Modified: branches/SuSE-Code-11-SP1-Branch/ldap-server/src/agent/SlapdConfigAgent.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/ldap-… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/ldap-server/src/agent/SlapdConfigAgent.cc (original) +++ branches/SuSE-Code-11-SP1-Branch/ldap-server/src/agent/SlapdConfigAgent.cc Wed Dec 2 16:57:48 2009 @@ -1697,7 +1697,10 @@ LDAPUrl prvuri; prvuri.setScheme(protocol); prvuri.setHost(target); - prvuri.setPort(port); + if ( ( protocol == "ldap" && port != 389 ) || ( protocol == "ldaps" && port != 636 ) ) + { + prvuri.setPort(port); + } sr->setType( type ); sr->setProvider( prvuri ); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r59908 - in /trunk/gtk: ChangeLog src/YGPackageSelectorPluginStub.cc src/YGPushButton.cc src/YGUI.h src/pkg/YGPackageSelector.cc
by rpmcruz＠svn.opensuse.org 02 Dec '09

02 Dec '09

Author: rpmcruz Date: Wed Dec 2 16:48:53 2009 New Revision: 59908 URL: http://svn.opensuse.org/viewcvs/yast?rev=59908&view=rev Log: * src/YGPackageSelector.cc: fix online update launch. Modified: trunk/gtk/ChangeLog trunk/gtk/src/YGPackageSelectorPluginStub.cc trunk/gtk/src/YGPushButton.cc trunk/gtk/src/YGUI.h trunk/gtk/src/pkg/YGPackageSelector.cc Modified: trunk/gtk/ChangeLog URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/ChangeLog?rev=59908&r1=59907… ============================================================================== --- trunk/gtk/ChangeLog (original) +++ trunk/gtk/ChangeLog Wed Dec 2 16:48:53 2009 @@ -3,6 +3,9 @@ * src/YGUtils.cc: bug fix 549943: HTML to XHTML conversor was tripping in the breakline. + * src/YGPackageSelector.cc: fix online update + launch. + 2009-12-01 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> * src/YGUtils.cc: bug fix 559226: Allow '&' to be escaped. Modified: trunk/gtk/src/YGPackageSelectorPluginStub.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGPackageSelectorPluginS… ============================================================================== --- trunk/gtk/src/YGPackageSelectorPluginStub.cc (original) +++ trunk/gtk/src/YGPackageSelectorPluginStub.cc Wed Dec 2 16:48:53 2009 @@ -25,7 +25,7 @@ } void YGUI::pkgSelectorSize (int *width, int *height) -{ *width = 700; *height = 800; } +{ *width = 700; *height = 750; } class YGPackageSelectorPluginStub : public YPackageSelectorPlugin { Modified: trunk/gtk/src/YGPushButton.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGPushButton.cc?rev=5990… ============================================================================== --- trunk/gtk/src/YGPushButton.cc (original) +++ trunk/gtk/src/YGPushButton.cc Wed Dec 2 16:48:53 2009 @@ -122,7 +122,6 @@ struct inner { static void realize_cb (GtkWidget *widget) { - GtkWidget *window = gtk_widget_get_toplevel (widget); gtk_widget_grab_default (widget); gtk_widget_grab_focus (widget); } Modified: trunk/gtk/src/YGUI.h URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/YGUI.h?rev=59908&r1=5990… ============================================================================== --- trunk/gtk/src/YGUI.h (original) +++ trunk/gtk/src/YGUI.h Wed Dec 2 16:48:53 2009 @@ -67,9 +67,10 @@ int m_default_width, m_default_height; bool pkgSelectorParse (const char *arg); - void pkgSelectorSize (int *width, int *height); public: + void pkgSelectorSize (int *width, int *height); + // Helpers for internal use [ visibility hidden ] bool setFullscreen() const { return m_fullscreen; } bool hasWM() const { return m_have_wm; } Modified: trunk/gtk/src/pkg/YGPackageSelector.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/pkg/YGPackageSelector.cc… ============================================================================== --- trunk/gtk/src/pkg/YGPackageSelector.cc (original) +++ trunk/gtk/src/pkg/YGPackageSelector.cc Wed Dec 2 16:48:53 2009 @@ -2403,7 +2403,9 @@ bool onlineUpdate = onlineUpdateMode(); YGDialog *dialog = YGDialog::currentDialog(); dialog->setCloseCallback (confirm_cb, this); - dialog->setMinSize (700, 800); // enlarge + int width, height; + YGUI::ui()->pkgSelectorSize (&width, &height); + dialog->setMinSize (width, height); YGtkWizard *wizard = YGTK_WIZARD (getWidget()); ygtk_wizard_set_header_icon (wizard, @@ -2523,6 +2525,7 @@ if (GTK_WIDGET_VISIBLE (m_progressbar)) { gtk_widget_hide (m_progressbar); while (g_main_context_iteration (NULL, FALSE)) ; + //gtk_widget_set_sensitive (getWidget(), TRUE); } YGUI::ui()->normalCursor(); } @@ -2530,6 +2533,7 @@ if (progress == 0) YGUI::ui()->busyCursor(); else { // progress=0 may be to trigger cursor only + //gtk_widget_set_sensitive (getWidget(), FALSE); gtk_widget_show (m_progressbar); gtk_progress_bar_set_fraction (GTK_PROGRESS_BAR (m_progressbar), progress); } @@ -2545,21 +2549,21 @@ YPackageSelector * YGPackageSelectorPluginImpl::createPackageSelector (YWidget *parent, long modeFlags) { - modeFlags &= YPkg_SearchMode; + modeFlags |= YPkg_SearchMode; return new YGPackageSelector (parent, modeFlags); } YWidget * YGPackageSelectorPluginImpl::createPatternSelector (YWidget *parent, long modeFlags) { - modeFlags ^= YPkg_SearchMode; + modeFlags &= YPkg_SearchMode; return new YGPackageSelector (parent, modeFlags); } YWidget * YGPackageSelectorPluginImpl::createSimplePatchSelector (YWidget *parent, long modeFlags) { - modeFlags &= YPkg_OnlineUpdateMode; + modeFlags |= YPkg_OnlineUpdateMode; return new YGPackageSelector (parent, modeFlags); } -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r59907 - in /trunk/printer/src: Printer.ycp basicadd.ycp basicmodify.ycp connectionwizard.ycp
by jsmeix＠svn.opensuse.org 02 Dec '09

02 Dec '09

Author: jsmeix Date: Wed Dec 2 16:43:55 2009 New Revision: 59907 URL: http://svn.opensuse.org/viewcvs/yast?rev=59907&view=rev Log: Implemented that the current URI from the Connection Wizard gets preselected in the Add/Modify dialogs. Modified: trunk/printer/src/Printer.ycp trunk/printer/src/basicadd.ycp trunk/printer/src/basicmodify.ycp trunk/printer/src/connectionwizard.ycp Modified: trunk/printer/src/Printer.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/printer/src/Printer.ycp?rev=5990… ============================================================================== --- trunk/printer/src/Printer.ycp (original) +++ trunk/printer/src/Printer.ycp Wed Dec 2 16:43:55 2009 @@ -1165,23 +1165,25 @@ sorted_connection_items = sort( term one_item, term another_item, connection_items, - { // Have the entries which are "created by the connection wizard" topmost: + { // Have entries which are "created by the connection wizard" + // or "modified by the connection wizard" topmost: // Have short local variable names for shorter comparison expression // and avoid multiple complicated accesses of same elements in an list: string this = one_item[3]:""; string that = another_item[3]:""; - // This is an entry where the info is "created by the connection wizard" + string connection_wizard_match = "by the connection wizard"; + // This is an entry where the info is "created/modified by the connection wizard" // and that is no such entry: - if( ( this == "created by the connection wizard" ) - && ( that != "created by the connection wizard" ) + if( issubstring( this, connection_wizard_match ) + && ( ! issubstring( that, connection_wizard_match ) ) ) return true; - // This is an entry where the info is not "created by the connection wizard" + // This is an entry where the info is not "created/modified by the connection wizard" // but that is such an entry: - if( ( this != "created by the connection wizard" ) - && ( that == "created by the connection wizard" ) + if( ( ! issubstring( this, connection_wizard_match ) ) + && issubstring( that, connection_wizard_match ) ) return false; - // If both are "created by the connection wizard" - // or if both are not "created by the connection wizard" + // If both are "created/modified by the connection wizard" + // or if both are not "created/modified by the connection wizard" // have entries with a valid model name topmost: // Have short local variable names for shorter comparison expression // and avoid multiple complicated accesses of same elements in an list: Modified: trunk/printer/src/basicadd.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/printer/src/basicadd.ycp?rev=599… ============================================================================== --- trunk/printer/src/basicadd.ycp (original) +++ trunk/printer/src/basicadd.ycp Wed Dec 2 16:43:55 2009 @@ -451,7 +451,7 @@ // (there is only "Kyocera" but not "Kyocera Mita" in the known_manufacturers list). foreach( string known_manufacturer, Printer::known_manufacturers, - { if( manufacturer == tolower( known_manufacturer ) ) + { if( tolower( manufacturer ) == tolower( known_manufacturer ) ) { driver_filter_string = "^" + driver_filter_string; break; } Modified: trunk/printer/src/basicmodify.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/printer/src/basicmodify.ycp?rev=… ============================================================================== --- trunk/printer/src/basicmodify.ycp (original) +++ trunk/printer/src/basicmodify.ycp Wed Dec 2 16:43:55 2009 @@ -78,17 +78,38 @@ Wizard::HideAbortButton(); string model = ""; list <string> description_words = splitstring( description, " " ); - if( size( description_words ) > 2 ) - { model = description_words[0]:"" + " " - + description_words[1]:"" + " " - + description_words[2]:""; - } - if( size( description_words ) == 2 ) - { model = description_words[0]:"" + " " - + description_words[1]:""; - } - if( size( description_words ) == 1 ) - { model = description_words[0]:""; + // In Printer.ycp and basicmodify.ycp there are lines like + // description = model + " with driver " + description; + // new_description = model + " with driver " + new_description; + // Avoid having "with" and/or "driver" in the model string: + description_words = filter( string word, + description_words, + { return( "with" != word && "driver" != word ); + } + ); + // Use the queue description only if its first word is actually a known manufacturer: + boolean description_starts_with_known_manufacturer = false; + foreach( string known_manufacturer, + Printer::known_manufacturers, + { if( tolower( description_words[0]:"" ) == tolower( known_manufacturer ) ) + { description_starts_with_known_manufacturer = true; + break; + } + } + ); + if( description_starts_with_known_manufacturer ) + { if( size( description_words ) > 2 ) + { model = description_words[0]:"" + " " + + description_words[1]:"" + " " + + description_words[2]:""; + } + if( size( description_words ) == 2 ) + { model = description_words[0]:"" + " " + + description_words[1]:""; + } + if( size( description_words ) == 1 ) + { model = description_words[0]:""; + } } string driver_filter_input_text = model; // Match anywhere in the NickName entry in the PPD: @@ -461,6 +482,12 @@ // so that this item can be preselected: y2milestone( "Preselected connection: '%1'", Printer::connections[Printer::selected_connections_index]:$[] ); UI::ChangeWidget( `connection_selection, `CurrentItem, `id(Printer::selected_connections_index) ); + // In contrast to BasicAddDialog fake user input of a connection_selection here only + // when a connection was preselected because initially drivers are shown which + // match to the description of the queue which is to be modified here + // but when the user had used the Connection Wizard, a different connection + // becomes usually preselected and this requires different drivers to be shown: + UI::FakeUserInput( `connection_selection ); } any ret = nil; while(true) @@ -789,7 +816,7 @@ // (there is only "Kyocera" but not "Kyocera Mita" in the known_manufacturers list). foreach( string known_manufacturer, Printer::known_manufacturers, - { if( manufacturer == tolower( known_manufacturer ) ) + { if( tolower( manufacturer ) == tolower( known_manufacturer ) ) { driver_filter_string = "^" + driver_filter_string; break; } @@ -808,17 +835,42 @@ } else { description_words = splitstring( description, " " ); - if( size( description_words ) > 2 ) - { model = description_words[0]:"" + " " - + description_words[1]:"" + " " - + description_words[2]:""; - } - if( size( description_words ) == 2 ) - { model = description_words[0]:"" + " " - + description_words[1]:""; + // In Printer.ycp and basicmodify.ycp there are lines like + // description = model + " with driver " + description; + // new_description = model + " with driver " + new_description; + // Avoid having "with" and/or "driver" in the model string: + description_words = filter( string word, + description_words, + { return( "with" != word && "driver" != word ); + } + ); + // Use the queue description only if its first word is actually a known manufacturer: + boolean description_starts_with_known_manufacturer = false; + foreach( string known_manufacturer, + Printer::known_manufacturers, + { if( tolower( description_words[0]:"" ) == tolower( known_manufacturer ) ) + { description_starts_with_known_manufacturer = true; + break; + } + } + ); + if( description_starts_with_known_manufacturer ) + { if( size( description_words ) > 2 ) + if( size( description_words ) > 2 ) + { model = description_words[0]:"" + " " + + description_words[1]:"" + " " + + description_words[2]:""; + } + if( size( description_words ) == 2 ) + { model = description_words[0]:"" + " " + + description_words[1]:""; + } + if( size( description_words ) == 1 ) + { model = description_words[0]:""; + } } - if( size( description_words ) == 1 ) - { model = description_words[0]:""; + else + { model = ""; } driver_filter_input_text = model; // Replace each sequence of spaces in driver_filter_input_text @@ -864,7 +916,7 @@ // (there is only "Kyocera" but not "Kyocera Mita" in the known_manufacturers list). foreach( string known_manufacturer, Printer::known_manufacturers, - { if( manufacturer == tolower( known_manufacturer ) ) + { if( tolower( manufacturer ) == tolower( known_manufacturer ) ) { driver_filter_string = "^" + driver_filter_string; break; } Modified: trunk/printer/src/connectionwizard.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/printer/src/connectionwizard.ycp… ============================================================================== --- trunk/printer/src/connectionwizard.ycp (original) +++ trunk/printer/src/connectionwizard.ycp Wed Dec 2 16:43:55 2009 @@ -159,17 +159,47 @@ && "unknown" != tolower( current_model_info ) ) { list <string> description_words = splitstring( current_model_info, " " ); - if( size( description_words ) > 2 ) - { current_model_info = description_words[0]:"" + " " - + description_words[1]:"" + " " - + description_words[2]:""; - } - if( size( description_words ) == 2 ) - { current_model_info = description_words[0]:"" + " " - + description_words[1]:""; + // In Printer.ycp and basicmodify.ycp there are lines like + // description = model + " with driver " + description; + // new_description = model + " with driver " + new_description; + // Avoid having "with" and/or "driver" in the current_model_info: + description_words = filter( string word, + description_words, + { return( "with" != word && "driver" != word ); + } + ); + // Use the queue description only if its first word is actually a known manufacturer: + boolean description_starts_with_known_manufacturer = false; + foreach( string known_manufacturer, + Printer::known_manufacturers, + { if( tolower( description_words[0]:"" ) == tolower( known_manufacturer ) ) + { description_starts_with_known_manufacturer = true; + break; + } + } + ); + if( description_starts_with_known_manufacturer ) + { if( size( description_words ) > 2 + && "with" != description_words[1]:"" + && "with" != description_words[2]:"" + && "driver" != description_words[2]:"" + ) + { current_model_info = description_words[0]:"" + " " + + description_words[1]:"" + " " + + description_words[2]:""; + } + if( size( description_words ) == 2 + && "with" != description_words[1]:"" + ) + { current_model_info = description_words[0]:"" + " " + + description_words[1]:""; + } + if( size( description_words ) == 1 ) + { current_model_info = description_words[0]:""; + } } - if( size( description_words ) == 1 ) - { current_model_info = description_words[0]:""; + else + { current_model_info = ""; } } } @@ -1845,6 +1875,13 @@ case(`lpd): case(`lpr): string lpd_hostname = (string)UI::QueryWidget( `hostname, `Value ); + if( ! issubstring( lpd_hostname, "@" ) ) + { // It would be wrong to do URIpercentEncoding(lpd_hostname) + // when it contains a '@' because a lpd URI can be of the form + // lpd://username@ip-address-or-hostname/... + // see https://bugzilla.novell.com/show_bug.cgi?id=512549 + lpd_hostname = URIpercentEncoding( lpd_hostname ); + } string lpd_queue = (string)UI::QueryWidget( `port_or_queue, `Value ); string lpd_uri_options = (string)UI::QueryWidget( `uri_options, `Value ); if( "" != filterchars( lpd_hostname, Printer::alnum_chars ) @@ -1856,14 +1893,14 @@ // must stay as is and not be percent encoded because lpd_uri_options // contains all options like 'option1=value1&option2=value2'. connection_uri = sformat( "lpd://%1/%2?%3", - URIpercentEncoding( lpd_hostname ), + lpd_hostname, URIpercentEncoding( lpd_queue ), lpd_uri_options ); } else { connection_uri = sformat( "lpd://%1/%2", - URIpercentEncoding( lpd_hostname ), + lpd_hostname, URIpercentEncoding( lpd_queue ) ); } @@ -1875,6 +1912,14 @@ break; case(`cups): string cups_hostname = (string)UI::QueryWidget( `hostname, `Value ); + if( ! issubstring( cups_hostname, "@" ) ) + { // It would be wrong to do URIpercentEncoding(cups_hostname) + // when it contains a '@' because a ipp/http URI can be of the form + // ipp://username:password@ip-address-or-hostname/... + // http://username:password@ip-address-or-hostname/... + // see https://bugzilla.novell.com/show_bug.cgi?id=512549 + cups_hostname = URIpercentEncoding( cups_hostname ); + } string cups_queue = (string)UI::QueryWidget( `queue, `Value ); string cups_uri_options = (string)UI::QueryWidget( `uri_options, `Value ); if( "" != filterchars( cups_hostname, Printer::alnum_chars ) @@ -1886,14 +1931,14 @@ // must stay as is and not be percent encoded because cups_uri_options // contains all options like 'option1=value1&option2=value2'. connection_uri = sformat( "ipp://%1/printers/%2?%3", - URIpercentEncoding( cups_hostname ), + cups_hostname, URIpercentEncoding( cups_queue ), cups_uri_options ); } else { connection_uri = sformat( "ipp://%1/printers/%2", - URIpercentEncoding( cups_hostname ), + cups_hostname, URIpercentEncoding( cups_queue ) ); } @@ -2240,22 +2285,76 @@ return true; } -boolean storeQueue( symbol selected ) +boolean UpdateConnectionsList( symbol selected ) { if( "" == connection_uri ) { return false; } if( "" == connection_model ) { connection_model = "Unknown"; } - y2internal( "storeQueue storing type '%1' with uri '%2' for model '%3'", selected, connection_uri, connection_model ); - Printer::connections = add( Printer::connections, - $[ "uri" : connection_uri, - "model" : connection_model, - "deviceID" : "", - "info" : "created by the connection wizard", - "class" : "ConnectionWizardDialog" - ] - ); + // Avoid duplicate URIs because Printer::current_device_uri must point to a unique URI + // otherwise preselection in BasicAddDialog and BasicModifyDialog would be ambiguous. + // If an URI already exists, update its matching entry in the Printer::connections list. + // Usually there should not exist duplicate URIs in the Printer::connections list + // (in contrast duplicate URIs in the Printer::queues list are perfectly o.k.) + // but if duplicate URIs exist in Printer::connections, all those entries are updated. + boolean uri_already_exists = false; +/* + // I do not know if it is allowed to modify one same list "in place" via foreach: + integer connections_index = -1; + foreach( map< string, string > connection_entry, + Printer::connections, + { // Set the connections_index to the index number of the current connection_entry: + connections_index = connections_index + 1; + // If the URI already exists, update its matching entry in the Printer::connections list: + if( connection_uri == connection_entry["uri"]:"" ) + { uri_already_exists = true; + y2internal( "UpdateConnectionsList: modify connections list type '%1' with uri '%2' for model '%3'", selected, connection_uri, connection_model ); + Printer::connections[connections_index,"model"] = connection_model; + // Leave te values for deviceID, info, and class unchanged because when the URI is the same + // those values still match to the URI and therefore should also stay the same. + } + } + ); +*/ + Printer::connections = maplist( map< string, string > connection_entry, + Printer::connections, + { if( connection_uri == connection_entry["uri"]:"" ) + { uri_already_exists = true; + y2internal( "UpdateConnectionsList: modify connections list type '%1' with uri '%2' for model '%3'", selected, connection_uri, connection_model ); + return $[ "uri" : connection_uri, + "model" : connection_model, + "deviceID" : "", + "info" : "modified by the connection wizard", + "class" : "ConnectionWizardDialog" + ]; + } + else + { return connection_entry; + } + } + ); + if( ! uri_already_exists ) + { y2internal( "UpdateConnectionsList: adding connection type '%1' with uri '%2' for model '%3'", selected, connection_uri, connection_model ); + Printer::connections = add( Printer::connections, + $[ "uri" : connection_uri, + "model" : connection_model, + "deviceID" : "", + "info" : "created by the connection wizard", + "class" : "ConnectionWizardDialog" + ] + ); + } + // Set Printer::current_device_uri so that BasicAddDialog and BasicModifyDialog + // can preselect the currently used connection when calling Printer::ConnectionItems(...) + // which sets Printer::selected_connections_index if it matches to Printer::current_device_uri. + // It makes sense to have the currently used connection from the ConnectionWizardDialog + // preselected also in BasicModifyDialog because when the user launched the Connection Wizard + // from the BasicModifyDialog he wants to change the connection so that it is sufficiently safe + // to have the changed connection preselected when he is back in the BasicModifyDialog + // so that the connection would get changed in the system without explicite selection + // by the user in the BasicModifyDialog if the user clicks "OK" in the BasicModifyDialog. + Printer::current_device_uri = connection_uri; return true; } @@ -2274,7 +2373,7 @@ && validateModel( selected ) ) { y2milestone( "ConnectionWizardDialog writing settings" ); - storeQueue( selected ); + UpdateConnectionsList( selected ); } else { y2error( "ConnectionWizardDialog: Could not validate for '%1'", selected ); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r59906 - in /branches/SuSE-Linux-11_0-Branch/backup: package/yast2-backup.changes src/backup.ycp
by locilka＠svn.opensuse.org 02 Dec '09

02 Dec '09

Author: locilka Date: Wed Dec 2 16:20:43 2009 New Revision: 59906 URL: http://svn.opensuse.org/viewcvs/yast?rev=59906&view=rev Log: - Fixed Mode when running in cron, 'commandline' is used instead of 'testsuite' (bnc #548427). Modified: branches/SuSE-Linux-11_0-Branch/backup/package/yast2-backup.changes branches/SuSE-Linux-11_0-Branch/backup/src/backup.ycp Modified: branches/SuSE-Linux-11_0-Branch/backup/package/yast2-backup.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Linux-11_0-Branch/backup… ============================================================================== --- branches/SuSE-Linux-11_0-Branch/backup/package/yast2-backup.changes (original) +++ branches/SuSE-Linux-11_0-Branch/backup/package/yast2-backup.changes Wed Dec 2 16:20:43 2009 @@ -3,6 +3,8 @@ - Fixed infinite loop in case of old archives to keep set to zero (bnc #550996). +- Fixed Mode when running in cron, 'commandline' is used instead of + 'testsuite' (bnc #548427). - 2.16.8 ------------------------------------------------------------------- Modified: branches/SuSE-Linux-11_0-Branch/backup/src/backup.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Linux-11_0-Branch/backup… ============================================================================== --- branches/SuSE-Linux-11_0-Branch/backup/src/backup.ycp (original) +++ branches/SuSE-Linux-11_0-Branch/backup/src/backup.ycp Wed Dec 2 16:20:43 2009 @@ -20,7 +20,7 @@ y2milestone(" *** Backup module started *** "); import "Wizard"; -//import "Mode"; +import "Mode"; import "Backup"; import "Report"; import "Sequencer"; @@ -135,6 +135,11 @@ y2milestone("Starting in cron mode"); Backup::cron_mode = true; + // BNC #548427: Mode acts like it was in a testsuite + // if it actually running in cron mode + y2milestone ("Adjusting Mode UI to: commandline"); + Mode::SetUI ("commandline"); + if (args[1]:nil != nil) { Backup::cron_profile = regexpsub(args[1]:"", "^[ \t]*profile[ \t]*=[ \t]*(.*)", "\\1"); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r59905 - in /branches/SuSE-Code-11-SP1-Branch/backup: package/yast2-backup.changes src/backup.ycp
by locilka＠svn.opensuse.org 02 Dec '09

02 Dec '09

Author: locilka Date: Wed Dec 2 16:19:42 2009 New Revision: 59905 URL: http://svn.opensuse.org/viewcvs/yast?rev=59905&view=rev Log: - Fixed Mode when running in cron, 'commandline' is used instead of 'testsuite' (bnc #548427). Modified: branches/SuSE-Code-11-SP1-Branch/backup/package/yast2-backup.changes branches/SuSE-Code-11-SP1-Branch/backup/src/backup.ycp Modified: branches/SuSE-Code-11-SP1-Branch/backup/package/yast2-backup.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/backu… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/backup/package/yast2-backup.changes (original) +++ branches/SuSE-Code-11-SP1-Branch/backup/package/yast2-backup.changes Wed Dec 2 16:19:42 2009 @@ -3,6 +3,8 @@ - Fixed infinite loop in case of old archives to keep set to zero (bnc #550996). +- Fixed Mode when running in cron, 'commandline' is used instead of + 'testsuite' (bnc #548427). - 2.17.7 ------------------------------------------------------------------- Modified: branches/SuSE-Code-11-SP1-Branch/backup/src/backup.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/backu… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/backup/src/backup.ycp (original) +++ branches/SuSE-Code-11-SP1-Branch/backup/src/backup.ycp Wed Dec 2 16:19:42 2009 @@ -20,7 +20,7 @@ y2milestone(" *** Backup module started *** "); import "Wizard"; -//import "Mode"; +import "Mode"; import "Backup"; import "Report"; import "Sequencer"; @@ -135,6 +135,11 @@ y2milestone("Starting in cron mode"); Backup::cron_mode = true; + // BNC #548427: Mode acts like it was in a testsuite + // if it actually running in cron mode + y2milestone ("Adjusting Mode UI to: commandline"); + Mode::SetUI ("commandline"); + if (args[1]:nil != nil) { Backup::cron_profile = regexpsub(args[1]:"", "^[ \t]*profile[ \t]*=[ \t]*(.*)", "\\1"); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r59904 - in /trunk/backup: package/yast2-backup.changes src/backup.ycp
by locilka＠svn.opensuse.org 02 Dec '09

02 Dec '09

Author: locilka Date: Wed Dec 2 16:18:27 2009 New Revision: 59904 URL: http://svn.opensuse.org/viewcvs/yast?rev=59904&view=rev Log: - Fixed Mode when running in cron, 'commandline' is used instead of 'testsuite' (bnc #548427). Modified: trunk/backup/package/yast2-backup.changes trunk/backup/src/backup.ycp Modified: trunk/backup/package/yast2-backup.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/backup/package/yast2-backup.chan… ============================================================================== --- trunk/backup/package/yast2-backup.changes (original) +++ trunk/backup/package/yast2-backup.changes Wed Dec 2 16:18:27 2009 @@ -3,6 +3,8 @@ - Fixed infinite loop in case of old archives to keep set to zero (bnc #550996). +- Fixed Mode when running in cron, 'commandline' is used instead of + 'testsuite' (bnc #548427). - 2.19.0 ------------------------------------------------------------------- Modified: trunk/backup/src/backup.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/backup/src/backup.ycp?rev=59904&… ============================================================================== --- trunk/backup/src/backup.ycp (original) +++ trunk/backup/src/backup.ycp Wed Dec 2 16:18:27 2009 @@ -20,7 +20,7 @@ y2milestone(" *** Backup module started *** "); import "Wizard"; -//import "Mode"; +import "Mode"; import "Backup"; import "Report"; import "Sequencer"; @@ -146,6 +146,11 @@ y2milestone("Starting in cron mode"); Backup::cron_mode = true; + // BNC #548427: Mode acts like it was in a testsuite + // if it actually running in cron mode + y2milestone ("Adjusting Mode UI to: commandline"); + Mode::SetUI ("commandline"); + if (args[1]:nil != nil) { Backup::cron_profile = regexpsub(args[1]:"", "^[ \t]*profile[ \t]*=[ \t]*(.*)", "\\1"); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r59903 - in /branches/SuSE-Linux-11_2-Branch/backup: package/yast2-backup.changes src/backup.ycp
by locilka＠svn.opensuse.org 02 Dec '09

02 Dec '09

Author: locilka Date: Wed Dec 2 16:16:30 2009 New Revision: 59903 URL: http://svn.opensuse.org/viewcvs/yast?rev=59903&view=rev Log: - Fixed Mode when running in cron, 'commandline' is used instead of 'testsuite' (bnc #548427). Modified: branches/SuSE-Linux-11_2-Branch/backup/package/yast2-backup.changes branches/SuSE-Linux-11_2-Branch/backup/src/backup.ycp Modified: branches/SuSE-Linux-11_2-Branch/backup/package/yast2-backup.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Linux-11_2-Branch/backup… ============================================================================== --- branches/SuSE-Linux-11_2-Branch/backup/package/yast2-backup.changes (original) +++ branches/SuSE-Linux-11_2-Branch/backup/package/yast2-backup.changes Wed Dec 2 16:16:30 2009 @@ -3,6 +3,8 @@ - Fixed infinite loop in case of old archives to keep set to zero (bnc #550996). +- Fixed Mode when running in cron, 'commandline' is used instead of + 'testsuite' (bnc #548427). - 2.18.5 ------------------------------------------------------------------- Modified: branches/SuSE-Linux-11_2-Branch/backup/src/backup.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Linux-11_2-Branch/backup… ============================================================================== --- branches/SuSE-Linux-11_2-Branch/backup/src/backup.ycp (original) +++ branches/SuSE-Linux-11_2-Branch/backup/src/backup.ycp Wed Dec 2 16:16:30 2009 @@ -20,7 +20,7 @@ y2milestone(" *** Backup module started *** "); import "Wizard"; -//import "Mode"; +import "Mode"; import "Backup"; import "Report"; import "Sequencer"; @@ -146,6 +146,11 @@ y2milestone("Starting in cron mode"); Backup::cron_mode = true; + // BNC #548427: Mode acts like it was in a testsuite + // if it actually running in cron mode + y2milestone ("Adjusting Mode UI to: commandline"); + Mode::SetUI ("commandline"); + if (args[1]:nil != nil) { Backup::cron_profile = regexpsub(args[1]:"", "^[ \t]*profile[ \t]*=[ \t]*(.*)", "\\1"); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r59902 - in /branches/SuSE-Code-11-Branch/backup: package/yast2-backup.changes src/backup.ycp
by locilka＠svn.opensuse.org 02 Dec '09

02 Dec '09

Author: locilka Date: Wed Dec 2 16:08:26 2009 New Revision: 59902 URL: http://svn.opensuse.org/viewcvs/yast?rev=59902&view=rev Log: - Fixed Mode when running in cron, 'commandline' is used instead of 'testsuite' (bnc #548427). Modified: branches/SuSE-Code-11-Branch/backup/package/yast2-backup.changes branches/SuSE-Code-11-Branch/backup/src/backup.ycp Modified: branches/SuSE-Code-11-Branch/backup/package/yast2-backup.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-Branch/backup/pa… ============================================================================== --- branches/SuSE-Code-11-Branch/backup/package/yast2-backup.changes (original) +++ branches/SuSE-Code-11-Branch/backup/package/yast2-backup.changes Wed Dec 2 16:08:26 2009 @@ -3,6 +3,8 @@ - Fixed infinite loop in case of old archives to keep set to zero (bnc #550996). +- Fixed Mode when running in cron, 'commandline' is used instead of + 'testsuite' (bnc #548427). - 2.17.6.1 ------------------------------------------------------------------- Modified: branches/SuSE-Code-11-Branch/backup/src/backup.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-Branch/backup/sr… ============================================================================== --- branches/SuSE-Code-11-Branch/backup/src/backup.ycp (original) +++ branches/SuSE-Code-11-Branch/backup/src/backup.ycp Wed Dec 2 16:08:26 2009 @@ -20,7 +20,7 @@ y2milestone(" *** Backup module started *** "); import "Wizard"; -//import "Mode"; +import "Mode"; import "Backup"; import "Report"; import "Sequencer"; @@ -135,6 +135,11 @@ y2milestone("Starting in cron mode"); Backup::cron_mode = true; + // BNC #548427: Mode acts like it was in a testsuite + // if it actually running in cron mode + y2milestone ("Adjusting Mode UI to: commandline"); + Mode::SetUI ("commandline"); + if (args[1]:nil != nil) { Backup::cron_profile = regexpsub(args[1]:"", "^[ \t]*profile[ \t]*=[ \t]*(.*)", "\\1"); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r59901 - in /trunk/backup: VERSION package/yast2-backup.changes src/Backup.ycp
by locilka＠svn.opensuse.org 02 Dec '09

02 Dec '09

Author: locilka Date: Wed Dec 2 16:03:27 2009 New Revision: 59901 URL: http://svn.opensuse.org/viewcvs/yast?rev=59901&view=rev Log: - Fixed infinite loop in case of old archives to keep set to zero (bnc #550996). - 2.19.0 Modified: trunk/backup/VERSION trunk/backup/package/yast2-backup.changes trunk/backup/src/Backup.ycp Modified: trunk/backup/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/backup/VERSION?rev=59901&r1=5990… ============================================================================== --- trunk/backup/VERSION (original) +++ trunk/backup/VERSION Wed Dec 2 16:03:27 2009 @@ -1 +1 @@ -2.18.4 +2.19.0 Modified: trunk/backup/package/yast2-backup.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/backup/package/yast2-backup.chan… ============================================================================== --- trunk/backup/package/yast2-backup.changes (original) +++ trunk/backup/package/yast2-backup.changes Wed Dec 2 16:03:27 2009 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Wed Dec 2 15:41:11 CET 2009 - locilka(a)suse.cz + +- Fixed infinite loop in case of old archives to keep set to zero + (bnc #550996). +- 2.19.0 + +------------------------------------------------------------------- Wed Jun 24 13:35:51 CEST 2009 - locilka(a)suse.cz - Fixed 'bytes' units in help text (bnc #513438). Modified: trunk/backup/src/Backup.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/backup/src/Backup.ycp?rev=59901&… ============================================================================== --- trunk/backup/src/Backup.ycp (original) +++ trunk/backup/src/Backup.ycp Wed Dec 2 16:03:27 2009 @@ -1883,10 +1883,10 @@ y2milestone("Old archives: %1", files); - if (size(files) >= max && max >= 0) + if (size(files) > 0 && size(files) >= max && max >= 0) { // remove the old archives - while(size(files) >= max) + while(size(files) > 0 && size(files) >= max) { string oldarchive = files[size(files) - 1]:"__DUMMY__"; -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r59900 - in /branches/SuSE-Linux-11_2-Branch/backup: VERSION package/yast2-backup.changes src/Backup.ycp
by locilka＠svn.opensuse.org 02 Dec '09

02 Dec '09

Author: locilka Date: Wed Dec 2 16:02:15 2009 New Revision: 59900 URL: http://svn.opensuse.org/viewcvs/yast?rev=59900&view=rev Log: - Fixed infinite loop in case of old archives to keep set to zero (bnc #550996). - 2.18.5 Modified: branches/SuSE-Linux-11_2-Branch/backup/VERSION branches/SuSE-Linux-11_2-Branch/backup/package/yast2-backup.changes branches/SuSE-Linux-11_2-Branch/backup/src/Backup.ycp Modified: branches/SuSE-Linux-11_2-Branch/backup/VERSION URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Linux-11_2-Branch/backup… ============================================================================== --- branches/SuSE-Linux-11_2-Branch/backup/VERSION (original) +++ branches/SuSE-Linux-11_2-Branch/backup/VERSION Wed Dec 2 16:02:15 2009 @@ -1 +1 @@ -2.18.4 +2.18.5 Modified: branches/SuSE-Linux-11_2-Branch/backup/package/yast2-backup.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Linux-11_2-Branch/backup… ============================================================================== --- branches/SuSE-Linux-11_2-Branch/backup/package/yast2-backup.changes (original) +++ branches/SuSE-Linux-11_2-Branch/backup/package/yast2-backup.changes Wed Dec 2 16:02:15 2009 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Wed Dec 2 15:41:11 CET 2009 - locilka(a)suse.cz + +- Fixed infinite loop in case of old archives to keep set to zero + (bnc #550996). +- 2.18.5 + +------------------------------------------------------------------- Wed Jun 24 13:35:51 CEST 2009 - locilka(a)suse.cz - Fixed 'bytes' units in help text (bnc #513438). Modified: branches/SuSE-Linux-11_2-Branch/backup/src/Backup.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Linux-11_2-Branch/backup… ============================================================================== --- branches/SuSE-Linux-11_2-Branch/backup/src/Backup.ycp (original) +++ branches/SuSE-Linux-11_2-Branch/backup/src/Backup.ycp Wed Dec 2 16:02:15 2009 @@ -1883,10 +1883,10 @@ y2milestone("Old archives: %1", files); - if (size(files) >= max && max >= 0) + if (size(files) > 0 && size(files) >= max && max >= 0) { // remove the old archives - while(size(files) >= max) + while(size(files) > 0 && size(files) >= max) { string oldarchive = files[size(files) - 1]:"__DUMMY__"; -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0