January 2010 - YaST Commits - openSUSE Mailing Lists

[yast-commit] r60254 - in /branches/SuSE-Code-11-SP1-Branch/yast2: VERSION package/yast2.changes scripts/yast2
by mvidner＠svn.opensuse.org 07 Jan '10

07 Jan '10

Author: mvidner Date: Thu Jan 7 10:46:44 2010 New Revision: 60254 URL: http://svn.opensuse.org/viewcvs/yast?rev=60254&view=rev Log: YaST would not start from the GNOME menu (Unknown option -S) bnc#568424. Duplicate of bnc#537470, so just copied r58631 Modified: branches/SuSE-Code-11-SP1-Branch/yast2/VERSION branches/SuSE-Code-11-SP1-Branch/yast2/package/yast2.changes branches/SuSE-Code-11-SP1-Branch/yast2/scripts/yast2 Modified: branches/SuSE-Code-11-SP1-Branch/yast2/VERSION URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/VERSION (original) +++ branches/SuSE-Code-11-SP1-Branch/yast2/VERSION Thu Jan 7 10:46:44 2010 @@ -1 +1 @@ -2.17.79 +2.17.80 Modified: branches/SuSE-Code-11-SP1-Branch/yast2/package/yast2.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/package/yast2.changes (original) +++ branches/SuSE-Code-11-SP1-Branch/yast2/package/yast2.changes Thu Jan 7 10:46:44 2010 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Thu Jan 7 10:43:13 CET 2010 - mvidner(a)suse.cz + +- YaST would not start from the GNOME menu (Unknown option -S) bnc#568424. +- 2.17.80 + +------------------------------------------------------------------- Thu Jan 7 08:20:23 UTC 2010 - lslezak(a)suse.cz - URL.ycp - escape non-ASCII characters in URL, added URLRecode.pm Modified: branches/SuSE-Code-11-SP1-Branch/yast2/scripts/yast2 URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/scripts/yast2 (original) +++ branches/SuSE-Code-11-SP1-Branch/yast2/scripts/yast2 Thu Jan 7 10:46:44 2010 @@ -232,10 +232,13 @@ # Pass the arguments to WFM::Args(): # Surround all args by (...). args may contain spaces. # Use -S to avoid YCP escaping (bnc#382883) -a=("$@") -b=("${a[@]/#/(}") -c=("${b[@]/%/)}") -set -- -S "${c[@]}" +# But avoid a lone -S (cc-gnome dies, bnc#537470) +if [ $# -gt 0 ]; then + a=("$@") + b=("${a[@]/#/(}") + c=("${b[@]/%/)}") + set -- -S "${c[@]}" +fi case "`basename $0`" in YaST|yast|yast1|zast) -- 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] r60253 - /tags/branch-Code-11-SP1-2_17_79/yast2/
by lslezak＠svn.opensuse.org 07 Jan '10

07 Jan '10

Author: lslezak Date: Thu Jan 7 09:59:27 2010 New Revision: 60253 URL: http://svn.opensuse.org/viewcvs/yast?rev=60253&view=rev Log: Created tag branch-Code-11-SP1-2_17_79 for yast2 Added: tags/branch-Code-11-SP1-2_17_79/yast2/ (props changed) - copied from r60252, branches/SuSE-Code-11-SP1-Branch/yast2/ -- 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] r60252 - in /branches/SuSE-Code-11-SP1-Branch/yast2: ./ library/types/src/ library/types/testsuite/tests/ package/
by lslezak＠svn.opensuse.org 07 Jan '10

07 Jan '10

Author: lslezak Date: Thu Jan 7 09:59:20 2010 New Revision: 60252 URL: http://svn.opensuse.org/viewcvs/yast?rev=60252&view=rev Log: - URL.ycp - escape non-ASCII characters in URL, added URLRecode.pm module (bnc#446395) - 2.17.79 Added: branches/SuSE-Code-11-SP1-Branch/yast2/library/types/src/URLRecode.pm branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URLRecode.err branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URLRecode.out branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URLRecode.ycp Modified: branches/SuSE-Code-11-SP1-Branch/yast2/VERSION branches/SuSE-Code-11-SP1-Branch/yast2/library/types/src/URL.ycp branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URL.out branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URL.ycp branches/SuSE-Code-11-SP1-Branch/yast2/package/yast2.changes Modified: branches/SuSE-Code-11-SP1-Branch/yast2/VERSION URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/VERSION (original) +++ branches/SuSE-Code-11-SP1-Branch/yast2/VERSION Thu Jan 7 09:59:20 2010 @@ -1 +1 @@ -2.17.78 +2.17.79 Modified: branches/SuSE-Code-11-SP1-Branch/yast2/library/types/src/URL.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/library/types/src/URL.ycp (original) +++ branches/SuSE-Code-11-SP1-Branch/yast2/library/types/src/URL.ycp Thu Jan 7 09:59:20 2010 @@ -17,6 +17,7 @@ import "Hostname"; import "String"; import "IP"; +import "URLRecode"; /** * TODO: @@ -203,9 +204,9 @@ pth = substring (pth, 1); } } - tokens["path"] = UnEscapeString(pth, transform_map_filename); - tokens["query"] = UnEscapeString(rawtokens[5]:"", transform_map_query); - tokens["fragment"] = UnEscapeString(rawtokens[7]:"", transform_map_passwd); + tokens["path"] = URLRecode::UnEscape(pth); + tokens["query"] = URLRecode::UnEscape(rawtokens[5]:""); + tokens["fragment"] = URLRecode::UnEscape(rawtokens[7]:""); /* Extract username:pass@host:port */ list userpass = regexptokenize(rawtokens[2]:"", "^" + @@ -219,8 +220,8 @@ ); y2debug("userpass=%1", userpass); - tokens["user"] = UnEscapeString(userpass[1]:"", transform_map_passwd); - tokens["pass"] = UnEscapeString(userpass[3]:"", transform_map_passwd); + tokens["user"] = URLRecode::UnEscape(userpass[1]:""); + tokens["pass"] = URLRecode::UnEscape(userpass[3]:""); tokens["port"] = userpass[7]:""; if(userpass[5]:"" != "") @@ -318,6 +319,8 @@ string url = ""; string userpass = ""; + y2debug("URL::Build(): input: %1", tokens); + if(regexpmatch(tokens["scheme"]:"", "^[[:alpha:]]*$")) { /* if (tokens["scheme"]:"" == "samba") url="smb"; @@ -326,12 +329,12 @@ y2debug("url: %1", url); if (tokens["user"]:"" != "") { - userpass = EscapeString(tokens["user"]:"", transform_map_passwd); + userpass = URLRecode::EscapePassword(tokens["user"]:""); y2milestone("Escaped username '%1' => '%2'", tokens["user"]:"", userpass); } if (size(userpass) != 0 && tokens["pass"]:"" != "" ) { - userpass = sformat("%1:%2", userpass, EscapeString(tokens["pass"]:"", transform_map_passwd) ); + userpass = sformat("%1:%2", userpass, URLRecode::EscapePassword(tokens["pass"]:"") ); } if (size(userpass) > 0 ) userpass = userpass + "@"; @@ -355,21 +358,22 @@ // path is not empty and doesn't start with "/" if (tokens["path"]:"" != "" && ! regexpmatch (tokens["path"]:"", "^/")) - url = sformat("%1/%2", url, EscapeString(tokens["path"]:"", transform_map_filename)); + url = sformat("%1/%2", url, URLRecode::EscapePath(tokens["path"]:"")); // patch is not empty and starts with "/" else if (tokens["path"]:"" != "" && regexpmatch (tokens["path"]:"", "^/")) { while (substring (tokens["path"]:"", 0, 2) == "//") tokens["path"] = substring (tokens["path"]:"", 1); if (tokens["scheme"]:"" == "ftp") { - url = sformat("%1/%%2f%2", url, substring(EscapeString(tokens["path"]:"", transform_map_filename), 1)); + url = sformat("%1/%%2f%2", url, substring(URLRecode::EscapePath(tokens["path"]:""), 1)); } else { - url = sformat("%1%2", url, EscapeString(tokens["path"]:"", transform_map_filename)); + url = sformat("%1%2", url, URLRecode::EscapePath(tokens["path"]:"")); } } y2debug("url: %1", url); + map<string,string> query_map = MakeMapFromParams(tokens["query"]:""); if (tokens["scheme"]:"" == "smb" && size(tokens["domain"]:"")>0 && query_map["workgroup"]:"" != tokens["domain"]:"") @@ -380,9 +384,10 @@ } if (tokens["query"]:"" != "" ) - url = sformat("%1?%2", url, EscapeString(tokens["query"]:"", transform_map_query)); + url = sformat("%1?%2", url, URLRecode::EscapeQuery(tokens["query"]:"")); + if (tokens["fragment"]:"" != "" ) - url = sformat("%1#%2", url, EscapeString(tokens["fragment"]:"", transform_map_passwd)); + url = sformat("%1#%2", url, URLRecode::EscapePassword(tokens["fragment"]:"")); y2debug("url: %1", url); if (!Check(url)) { @@ -390,6 +395,8 @@ return ""; } + y2debug("URL::Build(): result: %1", url); + return url; } Added: branches/SuSE-Code-11-SP1-Branch/yast2/library/types/src/URLRecode.pm URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/library/types/src/URLRecode.pm (added) +++ branches/SuSE-Code-11-SP1-Branch/yast2/library/types/src/URLRecode.pm Thu Jan 7 09:59:20 2010 @@ -0,0 +1,87 @@ +#!/usr/bin/perl -w +# +# Author: Ladislav SlezÃ¡k <lslezak(a)novell.com> +# +# $Id:$ +# +# URLRecode.pm +# +# This is a replacement for URI::Encode perl module which cannot be used in inst-sys +# and to decrease the package dependencies +# + +package URLRecode; + +use strict; +use YaST::YCP qw(:LOGGING Boolean sformat);; + +our %TYPEINFO; +use strict; + +# local cache for char -> hex string conversion +our %escape_cache; + +# fill the cache +sub InitCache +{ + for(0..255) {$escape_cache{chr($_)} = sprintf("%%%02x", $_);} +} + + +# Escape password, user name and fragment part of URL string +# @param input input string +# @return string Escaped string +BEGIN{ $TYPEINFO{EscapePassword} = ["function", "string", "string"];} +sub EscapePassword +{ + my ($self, $escaped) = @_; + + if (!defined %escape_cache) { InitCache(); } + + $escaped =~ s/([^A-Za-z0-9\\-_.!~*'()])/$escape_cache{$1}/ge; + return $escaped; +} + +# Escape path part of URL string +# @param input input string +# @return string Escaped string +BEGIN{ $TYPEINFO{EscapePath} = ["function", "string", "string"];} +sub EscapePath +{ + my ($self, $escaped) = @_; + + if (!defined %escape_cache) { InitCache() }; + + $escaped =~ s/([^A-Za-z0-9\-_.!~*'()\/])/$escape_cache{$1}/ge; + return $escaped; +} + +# Escape query part of URL string +# @param input input string +# @return string Escaped string +BEGIN{ $TYPEINFO{EscapeQuery} = ["function", "string", "string"];} +sub EscapeQuery +{ + my ($self, $escaped) = @_; + + if (!defined %escape_cache) { InitCache(); } + + $escaped =~ s/([^A-Za-z0-9\\-_.!~*'()\/:=&])/$escape_cache{$1}/ge; + return $escaped; +} + + +# UnEscape an URL string, replace %<Hexnum><HexNum> sequences +# by character +# @param input input string +# @return string Unescaped string +BEGIN{ $TYPEINFO{UnEscape} = ["function", "string", "string"];} +sub UnEscape +{ + my ($self, $input) = @_; + + $input =~ s/%([0-9A-Fa-f]{2})/chr(hex($1))/ge; + return $input +} + +1; Modified: branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URL.out URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URL.out (original) +++ branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URL.out Thu Jan 7 09:59:20 2010 @@ -21,6 +21,8 @@ Return ftp://ftp.example.com/%2fpath/to/dir Return ftp://ftp.example.com/%2fpath/to/dir Return ftp://ftp.example.com/%2fpath/to/dir +Return ftp://ftp.example.com/%2fpath/to/dir?param1=val1&param2=val2 +Return dir:///path/to/%c4%9b%c5%a1%c4%8d%c5%99%c5%be%c3%bd%c3%a1%c3%ad%c3%a9/dir Return ftp://[2001:de8:0:f123::1]/%2fpath/to/dir Return http://[2001:de8:0:f123::1]:8080/path/to/dir Return $["fragment":"", "host":"2001:de8:0:f123::1", "pass":"", "path":"/path/to/dir", "port":"", "query":"", "scheme":"http", "user":""] Modified: branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URL.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URL.ycp (original) +++ branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URL.ycp Thu Jan 7 09:59:20 2010 @@ -10,6 +10,13 @@ { include "testsuite.ycp"; +map READ = $[ + "target" : $[ + "tmpdir" : "/tmp" + ] +]; +TESTSUITE_INIT ([READ], nil); + import "URL"; TEST(``(URL::Parse("http://name:pass@www.suse.cz:80/path/index.html?question#part")), [], nil); @@ -50,6 +57,10 @@ TEST(``(URL::Build($["scheme":"ftp","host":"ftp.example.com","path":"/path/to/dir"])), [], nil); TEST(``(URL::Build($["scheme":"ftp","host":"ftp.example.com","path":"//path/to/dir"])), [], nil); TEST(``(URL::Build($["scheme":"ftp","host":"ftp.example.com","path":"///path/to/dir"])), [], nil); +TEST(``(URL::Build($["scheme":"ftp","host":"ftp.example.com","path":"///path/to/dir", "query" : "param1=val1&param2=val2" ])), [], nil); + +// bnc #446395 - non-ASCII chars in path must be escaped +TEST(``(URL::Build($["scheme":"dir", "path":"/path/to/ÄÅ¡ÄÅÅ¾Ã½Ã¡ÃÃ©/dir"])), [], nil); // IPv6 tests TEST(``(URL::Build($["scheme":"ftp","host":"2001:de8:0:f123::1","path":"///path/to/dir"])), [], nil); Added: branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URLRecode.err URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== (empty) Added: branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URLRecode.out URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URLRecode.out (added) +++ branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URLRecode.out Thu Jan 7 09:59:20 2010 @@ -0,0 +1,7 @@ +Return nil +Return +Return /%40%23%24%25%5e%26/dir/%c4%8d%c3%bd%c4%9b%c5%a1%c4%8d%c3%bd%c3%a1/file +Return true +Return %20!%40%23%24%25%5e%26*()/%3f%2b%3d +Return %20!%40%23%24%25^%26*()%2f%3f%2b%3d +Return %20!%40%23%24%25^&*()/%3f%2b= Added: branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URLRecode.ycp URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URLRecode.ycp (added) +++ branches/SuSE-Code-11-SP1-Branch/yast2/library/types/testsuite/tests/URLRecode.ycp Thu Jan 7 09:59:20 2010 @@ -0,0 +1,27 @@ +/** + * Testsuite for URLRecode.pm module + * + * $Id: URLRecode.ycp 57027 2009-04-29 10:24:13Z lslezak $ + */ + +{ + +include "testsuite.ycp"; +import "URLRecode"; + + +TEST(``(URLRecode::EscapePath(nil)), [], nil); +TEST(``(URLRecode::EscapePath("")), [], nil); + +string test_path = "/@#$%^&/dir/ÄÃ½ÄÅ¡ÄÃ½Ã¡/file"; + +TEST(``(URLRecode::EscapePath(test_path)), [], nil); +TEST(``(URLRecode::UnEscape(URLRecode::EscapePath(test_path)) == test_path), [], nil); + + +TEST(``(URLRecode::EscapePath(" !@#$%^&*()/?+=")), [], nil); +TEST(``(URLRecode::EscapePassword(" !@#$%^&*()/?+=")), [], nil); +TEST(``(URLRecode::EscapeQuery(" !@#$%^&*()/?+=")), [], nil); + +/* EOF */ +} Modified: branches/SuSE-Code-11-SP1-Branch/yast2/package/yast2.changes URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/yast2… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/yast2/package/yast2.changes (original) +++ branches/SuSE-Code-11-SP1-Branch/yast2/package/yast2.changes Thu Jan 7 09:59:20 2010 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Thu Jan 7 08:20:23 UTC 2010 - lslezak(a)suse.cz + +- URL.ycp - escape non-ASCII characters in URL, added URLRecode.pm + module (bnc#446395) +- 2.17.79 + +------------------------------------------------------------------- Tue Jan 5 15:07:54 UTC 2010 - lslezak(a)suse.cz - added missing UI::SetProductName() call - display the proper -- 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] r60251 - /trunk/gtk/src/pkg/YGPackageSelector.cc
by rpmcruz＠svn.opensuse.org 07 Jan '10

07 Jan '10

Author: rpmcruz Date: Thu Jan 7 03:37:46 2010 New Revision: 60251 URL: http://svn.opensuse.org/viewcvs/yast?rev=60251&view=rev Log: Something to enable in the future. Modified: trunk/gtk/src/pkg/YGPackageSelector.cc 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 Thu Jan 7 03:37:46 2010 @@ -2845,6 +2845,10 @@ : m_link_popup (NULL) { m_text = ygtk_rich_text_new(); +/* + ygtk_rich_text_set_text (YGTK_RICH_TEXT (m_text), + _("<i>Write a little introductory text here</i>")); +*/ YGtkPackageView *view = ygtk_package_view_new (TRUE); m_pkg_view = GTK_WIDGET (view); //gtk_widget_set_size_request (m_pkg_view, -1, 200); -- 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] r60250 - in /trunk/gtk: ChangeLog src/YGPackageSelectorPluginStub.cc src/pkg/YGPackageSelector.cc src/pkg/ygtkpackageview.cc src/pkg/yzyppwrapper.cc src/pkg/yzyppwrapper.h
by rpmcruz＠svn.opensuse.org 07 Jan '10

07 Jan '10

Author: rpmcruz Date: Thu Jan 7 01:36:18 2010 New Revision: 60250 URL: http://svn.opensuse.org/viewcvs/yast?rev=60250&view=rev Log: * src/pkg/yzyppwrapper.h/cc: added API to read changes log. * src/pkg/YGPackageSelector.cc: improved look of --undo-old-style. * src/pkg/YGPackageSelector.cc: as requested by Atri, added previous changes to the changes sidebar. See flag --undo-log-all and --undo-log-changed. * src/pkg/ygtkpackageview.cc: changed action button label/stock behavior. * src/pkg/ygtkpackageview.cc: as suggested by Atri, added flag --action-col-label to allow disabling action button labels. * src/pkg/ygtkpackageview.cc: added --colorful-rows. Modified: trunk/gtk/ChangeLog trunk/gtk/src/YGPackageSelectorPluginStub.cc trunk/gtk/src/pkg/YGPackageSelector.cc trunk/gtk/src/pkg/ygtkpackageview.cc trunk/gtk/src/pkg/yzyppwrapper.cc trunk/gtk/src/pkg/yzyppwrapper.h Modified: trunk/gtk/ChangeLog URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/ChangeLog?rev=60250&r1=60249… ============================================================================== --- trunk/gtk/ChangeLog (original) +++ trunk/gtk/ChangeLog Thu Jan 7 01:36:18 2010 @@ -1,3 +1,21 @@ +2010-01-07 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> + + * src/pkg/yzyppwrapper.h/cc: added API to read changes log. + + * src/pkg/YGPackageSelector.cc: improved look of --undo-old-style. + + * src/pkg/YGPackageSelector.cc: as requested by Atri, + added previous changes to the changes sidebar. See flag + --undo-log-all and --undo-log-changed. + + * src/pkg/ygtkpackageview.cc: changed action button label/stock + behavior. + + * src/pkg/ygtkpackageview.cc: as suggested by Atri, added + flag --action-col-label to allow disabling action button labels. + + * src/pkg/ygtkpackageview.cc: added --colorful-rows. + 2009-12-29 Ricardo Cruz <rpmcruz(a)alunos.dcc.fc.up.pt> * src/pkg/YGPackageSelector.cc: added --status-top 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 Thu Jan 7 01:36:18 2010 @@ -14,10 +14,12 @@ bool search_entry_side = true, search_entry_top = false, dynamic_sidebar = false, categories_side = true, categories_top = false, status_side = false, status_top = false, status_tabs = true, status_tabs_as_actions = false, - undo_side = false, undo_tab = true, undo_old_style = false, status_col = false, - action_col = true, action_col_as_button = true, action_col_as_check = false, - version_col = true, single_line_rows = false, details_start_hide = true, - toolbar_top = false, toolbar_yast = false, arrange_by = false; + undo_side = false, undo_tab = true, undo_old_style = false, undo_log_all = false, + undo_log_changed = false, status_col = false, action_col = true, + action_col_as_button = true, action_col_as_check = false, action_col_label = true, + version_col = true, single_line_rows = false, colorful_rows = false, + details_start_hide = true, toolbar_top = false, toolbar_yast = false, + arrange_by = false; struct Arg { const char *arg; @@ -36,12 +38,16 @@ { "undo-side", &undo_side }, { "undo-tab", &undo_tab }, { "undo-old-style", &undo_old_style }, + { "undo-log-all", &undo_log_all }, + { "undo-log-changed", &undo_log_changed }, { "status-col", &status_col }, { "action-col", &action_col }, { "action-col-as-button", &action_col_as_button }, { "action-col-as-check", &action_col_as_check }, + { "action-col-label", &action_col_label }, { "version-col", &version_col }, { "single-line-rows", &single_line_rows }, + { "colorful-rows", &colorful_rows }, { "details-start-hide", &details_start_hide }, { "toolbar-top", &toolbar_top }, { "toolbar-yast", &toolbar_yast }, 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 Thu Jan 7 01:36:18 2010 @@ -29,9 +29,9 @@ // experiments: extern bool search_entry_side, search_entry_top, dynamic_sidebar, categories_side, categories_top, status_side, status_top, status_tabs, status_tabs_as_actions, - undo_side, undo_tab, undo_old_style, status_col, action_col, action_col_as_button, - action_col_as_check, version_col, single_line_rows, details_start_hide, - toolbar_top, toolbar_yast, arrange_by; + undo_side, undo_tab, undo_old_style, undo_log_all, undo_log_changed, status_col, + action_col, action_col_as_button, action_col_as_check, version_col, + single_line_rows, details_start_hide, toolbar_top, toolbar_yast, arrange_by; //** UI components -- split up for re-usability, but mostly for readability @@ -168,6 +168,43 @@ #endif }; +static void small_button_style_set_cb (GtkWidget *button, GtkStyle *prev_style) +{ + // based on gedit + int width, height; + gtk_icon_size_lookup_for_settings (gtk_widget_get_settings (button), + GTK_ICON_SIZE_MENU, &width, &height); + gtk_widget_set_size_request (button, width+2, height+2); +} + +static GtkWidget *create_small_button (const char *stock_icon, const char *tooltip) +{ + static bool first_time = true; + if (first_time) { + first_time = false; + gtk_rc_parse_string ( + "style \"small-button-style\"\n" + "{\n" + " GtkWidget::focus-padding = 0\n" + " GtkWidget::focus-line-width = 0\n" + " xthickness = 0\n" + " ythickness = 0\n" + "}\n" + "widget \"*.small-button\" style \"small-button-style\""); + } + + GtkWidget *button = gtk_button_new(); + gtk_widget_set_name (button, "small-button"); + gtk_button_set_focus_on_click (GTK_BUTTON (button), FALSE); + gtk_button_set_relief (GTK_BUTTON (button), GTK_RELIEF_NONE); + gtk_widget_set_tooltip_text (button, tooltip); + GtkWidget *image = gtk_image_new_from_stock (stock_icon, GTK_ICON_SIZE_MENU); + gtk_button_set_image (GTK_BUTTON (button), image); + g_signal_connect (G_OBJECT (button), "style-set", + G_CALLBACK (small_button_style_set_cb), NULL); + return button; +} + class ChangesPane : public Ypp::PkgList::Listener { struct Entry { @@ -179,13 +216,10 @@ m_label = gtk_label_new (""); gtk_misc_set_alignment (GTK_MISC (m_label), 0, 0.5); gtk_label_set_ellipsize (GTK_LABEL (m_label), PANGO_ELLIPSIZE_END); - m_button = gtk_button_new(); - gtk_widget_set_tooltip_text (m_button, _("Undo")); - GtkWidget *undo_image = gtk_image_new_from_stock (GTK_STOCK_UNDO, GTK_ICON_SIZE_MENU); - gtk_button_set_image (GTK_BUTTON (m_button), undo_image); + m_button = create_small_button (GTK_STOCK_UNDO, _("Undo")); m_box = gtk_hbox_new (FALSE, 6); gtk_box_pack_start (GTK_BOX (m_box), m_label, TRUE, TRUE, 0); - gtk_box_pack_start (GTK_BOX (m_box), m_button, FALSE, FALSE, 0); + gtk_box_pack_start (GTK_BOX (m_box), m_button, FALSE, TRUE, 0); gtk_widget_show_all (m_box); modified (package); g_signal_connect (G_OBJECT (m_button), "clicked", @@ -232,46 +266,120 @@ } }; -GtkWidget *m_box, *m_entries_box; + struct LogEntry { + GtkWidget *m_box, *m_label, *m_button; + Ypp::Log::Entry::Action m_action; + GtkWidget *getWidget() { return m_box; } + + LogEntry (Ypp::Package *package, Ypp::Log::Entry::Action action) + : m_action (action) + { + std::string label = action == Ypp::Log::Entry::INSTALL_ACTION ? + _("install") : _("remove"); + label += " " + package->name(); + m_label = gtk_label_new (label.c_str()); + gtk_misc_set_alignment (GTK_MISC (m_label), 0, 0.5); + gtk_label_set_ellipsize (GTK_LABEL (m_label), PANGO_ELLIPSIZE_END); + if (canUndo (package, action)) { + m_button = create_small_button (GTK_STOCK_UNDO, _("Undo")); + m_box = gtk_hbox_new (FALSE, 6); + gtk_box_pack_start (GTK_BOX (m_box), m_label, TRUE, TRUE, 0); + gtk_box_pack_start (GTK_BOX (m_box), m_button, FALSE, TRUE, 0); + g_signal_connect (G_OBJECT (m_button), "clicked", + G_CALLBACK (undo_clicked_cb), package); + } + else + m_box = m_label; + gtk_widget_show_all (m_box); + } + + static void undo_clicked_cb (GtkButton *button, Ypp::Package *package) + { +#if 0 + if (m_action == Ypp::Log::Entry::INSTALL_ACTION) + package->remove(); + else + package->install (0); +#endif + if (package->isInstalled()) + package->remove(); + else + package->install (0); + gtk_widget_hide (GTK_WIDGET (button)); + } + + static bool canUndo (Ypp::Package *package, Ypp::Log::Entry::Action action) + { + return (action == Ypp::Log::Entry::INSTALL_ACTION && package->isInstalled()) + || (action == Ypp::Log::Entry::REMOVE_ACTION && !package->isInstalled()); + } + }; + +GtkWidget *m_handle_box, *m_empty_label, *m_entries_box; Ypp::PkgList m_pool; GList *m_entries; public: GtkWidget *getWidget() - { return m_box; } + { return m_handle_box; } ChangesPane() : m_entries (NULL) { - GtkWidget *heading = gtk_label_new (_("Changes:")); + GtkWidget *heading = gtk_label_new (_("To perform now:")); YGUtils::setWidgetFont (heading, PANGO_STYLE_NORMAL, PANGO_WEIGHT_BOLD, PANGO_SCALE_MEDIUM); gtk_misc_set_alignment (GTK_MISC (heading), 0, 0.5); - m_entries_box = gtk_vbox_new (FALSE, 4); + m_entries_box = gtk_vbox_new (FALSE, 2); + + m_empty_label = gtk_label_new (_("Package changes will\nbe listed here.")); + YGUtils::setWidgetFont (m_empty_label, PANGO_STYLE_ITALIC, PANGO_WEIGHT_NORMAL, PANGO_SCALE_MEDIUM); + gtk_label_set_justify (GTK_LABEL (m_empty_label), GTK_JUSTIFY_CENTER); + gtk_misc_set_alignment (GTK_MISC (m_empty_label), 0.5, 0.5); + + GtkWidget *vbox = gtk_vbox_new (FALSE, 6); + gtk_container_set_border_width (GTK_CONTAINER (vbox), 4); + gtk_box_pack_start (GTK_BOX (vbox), heading, FALSE, TRUE, 0); + gtk_box_pack_start (GTK_BOX (vbox), m_empty_label, TRUE, TRUE, 0); + gtk_box_pack_start (GTK_BOX (vbox), m_entries_box, TRUE, TRUE, 0); + + struct inner { + static gboolean expose_cb (GtkWidget *widget, GdkEventExpose *event) + { + cairo_t *cr = gdk_cairo_create (widget->window); + GdkColor color = { 0, 255 << 8, 255 << 8, 255 << 8 }; + gdk_cairo_set_source_color (cr, &color); + cairo_rectangle (cr, event->area.x, event->area.y, + event->area.width, event->area.height); + cairo_fill (cr); + cairo_destroy (cr); + return FALSE; + } + }; + g_signal_connect (G_OBJECT (vbox), "expose-event", + G_CALLBACK (inner::expose_cb), NULL); GtkWidget *scroll = gtk_scrolled_window_new (NULL, NULL); gtk_scrolled_window_set_policy (GTK_SCROLLED_WINDOW (scroll), GTK_POLICY_NEVER, GTK_POLICY_AUTOMATIC); - gtk_scrolled_window_add_with_viewport (GTK_SCROLLED_WINDOW (scroll), - m_entries_box); + gtk_scrolled_window_add_with_viewport (GTK_SCROLLED_WINDOW (scroll), vbox); GtkWidget *port = gtk_bin_get_child (GTK_BIN (scroll)); gtk_viewport_set_shadow_type (GTK_VIEWPORT (port), GTK_SHADOW_NONE); - GtkWidget *vbox = gtk_vbox_new (FALSE, 6); - gtk_container_set_border_width (GTK_CONTAINER (vbox), 4); - gtk_box_pack_start (GTK_BOX (vbox), heading, FALSE, TRUE, 0); - gtk_box_pack_start (GTK_BOX (vbox), scroll, TRUE, TRUE, 0); - +/* ygtk_wizard_set_information_expose_hook (vbox, &vbox->allocation); ygtk_wizard_set_information_expose_hook (m_entries_box, &m_entries_box->allocation); - +*/ int width = YGUtils::getCharsWidth (vbox, 32); gtk_widget_set_size_request (vbox, width, -1); gtk_widget_show_all (vbox); - m_box = gtk_handle_box_new(); - gtk_container_add (GTK_CONTAINER (m_box), vbox); - gtk_handle_box_set_handle_position (GTK_HANDLE_BOX (m_box), GTK_POS_TOP); - gtk_handle_box_set_snap_edge (GTK_HANDLE_BOX (m_box), GTK_POS_RIGHT); + GtkWidget *frame = gtk_frame_new (NULL); + gtk_container_add (GTK_CONTAINER (frame), scroll); + + m_handle_box = gtk_handle_box_new(); + gtk_container_add (GTK_CONTAINER (m_handle_box), frame); + gtk_handle_box_set_handle_position (GTK_HANDLE_BOX (m_handle_box), GTK_POS_TOP); + gtk_handle_box_set_snap_edge (GTK_HANDLE_BOX (m_handle_box), GTK_POS_RIGHT); Ypp::QueryBase *query = new Ypp::QueryProperty ("to-modify", true); /* if (pkg_selector->onlineUpdateMode()) @@ -281,6 +389,25 @@ for (int i = 0; i < m_pool.size(); i++) ChangesPane::entryInserted (m_pool, i, m_pool.get (i)); m_pool.addListener (this); + + if (undo_log_all || undo_log_changed) { + GtkWidget *log_box = gtk_vbox_new (FALSE, 2); + GtkWidget *heading = gtk_label_new (_("Past changes:")); + YGUtils::setWidgetFont (heading, PANGO_STYLE_NORMAL, PANGO_WEIGHT_BOLD, PANGO_SCALE_MEDIUM); + gtk_misc_set_alignment (GTK_MISC (heading), 0, 0.5); + gtk_box_pack_start (GTK_BOX (vbox), heading, FALSE, TRUE, 0); + gtk_box_pack_start (GTK_BOX (vbox), log_box, TRUE, TRUE, 0); + + for (int i = 0; Ypp::get()->getLog()->getEntry (i); i++) { + Ypp::Log::Entry *entry = Ypp::get()->getLog()->getEntry (i); + if (entry->package) + if (undo_log_all || LogEntry::canUndo (entry->package, entry->action)) { + // FIXME: free these stuff + LogEntry *log_entry = new LogEntry (entry->package, entry->action); + gtk_box_pack_start (GTK_BOX (log_box), log_entry->getWidget(), FALSE, TRUE, 0); + } + } + } } ~ChangesPane() @@ -293,6 +420,7 @@ void startHack() // call after init, after you did a show_all in the dialog { UpdateVisible(); + // ugly: signal modified for all entries to allow them to hide undo buttons GList *i; int index; @@ -303,8 +431,10 @@ void UpdateVisible() { - if (details_start_hide) - m_entries != NULL ? gtk_widget_show (m_box) : gtk_widget_hide (m_box); + if (details_start_hide && !(undo_log_all || undo_log_changed)) + m_entries ? gtk_widget_show (m_handle_box) : gtk_widget_hide (m_handle_box); + if (m_entries) + gtk_widget_hide (m_empty_label); } virtual void entryInserted (const Ypp::PkgList list, int index, Ypp::Package *package) @@ -3078,13 +3208,16 @@ GtkWidget *packages_pane = gtk_hpaned_new(); gtk_paned_pack1 (GTK_PANED (packages_pane), packages_button_box, TRUE, FALSE); + ChangesPane *changes_pane = 0; GtkWidget *undo_widget; if (undo_side) { - if (undo_old_style) - undo_widget = (new ChangesPane())->getWidget(); + if (undo_old_style) { + changes_pane = new ChangesPane(); + undo_widget = changes_pane->getWidget(); + } else undo_widget = undo_view->createView (this, false); - gtk_paned_pack2 (GTK_PANED (packages_pane), undo_widget, FALSE, FALSE); + gtk_paned_pack2 (GTK_PANED (packages_pane), undo_widget, FALSE, TRUE); gtk_paned_set_position (GTK_PANED (packages_pane), 350); } @@ -3167,15 +3300,14 @@ m_widget = vbox; gtk_widget_show_all (m_widget); - if (details_start_hide) { + if (details_start_hide) gtk_widget_hide (m_details->getWidget()); - if (undo_old_style) - gtk_widget_hide (undo_widget); - } else m_details->setPackage (NULL); if (!categories_side && !search_entry_side && !status_side) gtk_widget_hide (side_vbox); + if (changes_pane) + changes_pane->startHack(); } ~UI() Modified: trunk/gtk/src/pkg/ygtkpackageview.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/pkg/ygtkpackageview.cc?r… ============================================================================== --- trunk/gtk/src/pkg/ygtkpackageview.cc (original) +++ trunk/gtk/src/pkg/ygtkpackageview.cc Thu Jan 7 01:36:18 2010 @@ -20,7 +20,7 @@ #include <string.h> extern bool status_col, action_col, action_col_as_button, action_col_as_check, - version_col, single_line_rows; + action_col_label, version_col, colorful_rows, single_line_rows; //** Icons resources @@ -116,8 +116,8 @@ TO_INSTALL_PROP, TO_UPGRADE_PROP, TO_REMOVE_PROP, TO_MODIFY_PROP, // internal STYLE_PROP, WEIGHT_PROP, SENSITIVE_PROP, CHECK_VISIBLE_PROP, - FOREGROUND_PROP, BACKGROUND_PROP, XPAD_PROP, - INSTALL_LABEL_PROP, REMOVE_LABEL_PROP, INSTALL_STOCK_PROP, REMOVE_STOCK_PROP, + FOREGROUND_PROP, VERSION_FOREGROUND_PROP, BACKGROUND_PROP, XPAD_PROP, + ACTION_LABEL_PROP, ACTION_STOCK_PROP, // misc PTR_PROP, TOTAL_PROPS }; @@ -136,11 +136,10 @@ case INSTALLED_VERSION_PROP: case AVAILABLE_VERSION_PROP: case FOREGROUND_PROP: + case VERSION_FOREGROUND_PROP: case BACKGROUND_PROP: - case INSTALL_LABEL_PROP: - case REMOVE_LABEL_PROP: - case INSTALL_STOCK_PROP: - case REMOVE_STOCK_PROP: + case ACTION_LABEL_PROP: + case ACTION_STOCK_PROP: return G_TYPE_STRING; case TO_INSTALL_PROP: case TO_UPGRADE_PROP: @@ -173,13 +172,12 @@ case SIZE_PROP: case INSTALLED_VERSION_PROP: case AVAILABLE_VERSION_PROP: - case INSTALL_LABEL_PROP: - case REMOVE_LABEL_PROP: - case INSTALL_STOCK_PROP: - case REMOVE_STOCK_PROP: + case ACTION_LABEL_PROP: + case ACTION_STOCK_PROP: g_value_set_string (value, g_strdup ("")); break; case FOREGROUND_PROP: + case VERSION_FOREGROUND_PROP: case BACKGROUND_PROP: g_value_set_string (value, NULL); break; @@ -412,12 +410,10 @@ g_value_set_boolean (value, modified); break; } - case INSTALL_LABEL_PROP: - case REMOVE_LABEL_PROP: + case ACTION_LABEL_PROP: g_value_set_string (value, g_strdup (_("All"))); break; - case INSTALL_STOCK_PROP: - case REMOVE_STOCK_PROP: + case ACTION_STOCK_PROP: g_value_set_string (value, g_strdup (GTK_STOCK_SELECT_ALL)); break; default: @@ -516,15 +512,32 @@ g_value_set_boolean (value, sensitive); break; } -/* case STYLE_PROP: { - PangoStyle style = PANGO_STYLE_NORMAL; - if (package->isAuto()) - style = PANGO_STYLE_ITALIC; - g_value_set_int (value, style); + if (colorful_rows) { + PangoStyle style = PANGO_STYLE_NORMAL; + if (package->isAuto()) + style = PANGO_STYLE_ITALIC; + g_value_set_int (value, style); + } + else + _getValueDefault (col, value); + break; + } + case VERSION_FOREGROUND_PROP: + case FOREGROUND_PROP: { + if (colorful_rows) { + const char *color = NULL; + if (!package->isInstalled()) + color = "darkgray"; + if (col == VERSION_FOREGROUND_PROP) + if (package->hasUpgrade()) + color = "blue"; + g_value_set_string (value, color); + } + else + _getValueDefault (col, value); break; } -*/ case WEIGHT_PROP: { bool highlight = segment->list.highlight (package); int weight = highlight ? PANGO_WEIGHT_BOLD : PANGO_WEIGHT_NORMAL; @@ -536,35 +549,25 @@ g_value_set_int (value, xpad); break; } - case INSTALL_LABEL_PROP: { - const char *label = _("Add"); -/* if (package->toModify()) + case ACTION_LABEL_PROP: { + const char *label; + if (package->toModify()) label = _("Undo"); - else if (package->hasUpgrade()) - label = _("Upgrade");*/ - if (package->hasUpgrade()) - label = _("Upgrade"); - g_value_set_string (value, g_strdup (label)); - break; - } - case REMOVE_LABEL_PROP: { - const char *label = _("Remove"); -/* if (package->toModify()) - label = _("Undo");*/ + else if (package->isInstalled()) + label = _("Remove"); + else + label = _("Add"); g_value_set_string (value, g_strdup (label)); break; } - case INSTALL_STOCK_PROP: { - const char *stock = GTK_STOCK_ADD; -/* if (package->toModify()) - stock = GTK_STOCK_UNDO;*/ - g_value_set_string (value, g_strdup (stock)); - break; - } - case REMOVE_STOCK_PROP: { - const char *stock = GTK_STOCK_REMOVE; -/* if (package->toModify()) - stock = GTK_STOCK_UNDO;*/ + case ACTION_STOCK_PROP: { + const char *stock; + if (package->toModify()) + stock = GTK_STOCK_UNDO; + else if (package->isInstalled()) + stock = GTK_STOCK_REMOVE; + else + stock = GTK_STOCK_ADD; g_value_set_string (value, g_strdup (stock)); break; } @@ -891,49 +894,50 @@ void appendButtonColumn (const char *header, const std::string &prop) { - int modelCol = translateProperty (prop); - int labelCol, stockCol; - switch (modelCol) { - case TO_INSTALL_PROP: case TO_UPGRADE_PROP: default: - labelCol = INSTALL_LABEL_PROP; - stockCol = INSTALL_STOCK_PROP; - break; - case TO_REMOVE_PROP: - labelCol = REMOVE_LABEL_PROP; - stockCol = REMOVE_STOCK_PROP; - break; - } + int labelCol = ACTION_LABEL_PROP, stockCol = ACTION_STOCK_PROP; GtkTreeView *view = GTK_TREE_VIEW (m_view); GtkCellRenderer *renderer = ygtk_cell_renderer_button_new(); GtkTreeViewColumn *column = gtk_tree_view_column_new_with_attributes ( - header, renderer, "active", modelCol, + header, renderer,// "active", modelCol, "visible", CHECK_VISIBLE_PROP, "sensitive", SENSITIVE_PROP, "cell-background", BACKGROUND_PROP, - "text", labelCol, -// "stock-id", stockCol, NULL); + if (action_col_label) + gtk_tree_view_column_add_attribute (column, renderer, "text", labelCol); + gboolean button_images; + g_object_get (G_OBJECT (gtk_settings_get_default()), "gtk-button-images", &button_images, NULL); + bool show_icon = !action_col_label || button_images; + if (show_icon) + gtk_tree_view_column_add_attribute (column, renderer, "stock-id", stockCol); g_signal_connect (G_OBJECT (renderer), "toggled", G_CALLBACK (renderer_toggled_cb), this); PangoRectangle rect; int width = 0; - const char *text[] = { _("Add"), _("Remove") }; - for (int i = 0; i < 2; i++) { - PangoLayout *layout = gtk_widget_create_pango_layout (m_view, text[i]); - pango_layout_get_pixel_extents (layout, NULL, &rect); - width = MAX (width, rect.width); - g_object_unref (G_OBJECT (layout)); + if (action_col_label) { + const char *text[] = { _("Add"), _("Remove") }; + for (int i = 0; i < 2; i++) { + PangoLayout *layout = gtk_widget_create_pango_layout (m_view, text[i]); + pango_layout_get_pixel_extents (layout, NULL, &rect); + width = MAX (width, rect.width); + g_object_unref (G_OBJECT (layout)); + } } width += 16; -// width += 20; // icon + if (show_icon) { + int icon_width; + gtk_icon_size_lookup_for_settings (gtk_widget_get_settings (m_view), + GTK_ICON_SIZE_MENU, &icon_width, NULL); + width += icon_width; + } gtk_tree_view_column_set_sizing (column, GTK_TREE_VIEW_COLUMN_FIXED); gtk_tree_view_column_set_fixed_width (column, width); gtk_tree_view_append_column (view, column); - Action action = columnAction (modelCol); + Action action = columnAction (TO_INSTALL_PROP); g_object_set_data (G_OBJECT (renderer), "action", GINT_TO_POINTER (action)); if (m_activate_action == NONE_ACTION) m_activate_action = action; @@ -967,9 +971,12 @@ "sensitive", SENSITIVE_PROP, "style", STYLE_PROP, "weight", WEIGHT_PROP, - "foreground", FOREGROUND_PROP, "cell-background", BACKGROUND_PROP, NULL); + int foregroundCol = FOREGROUND_PROP; + if (col == AVAILABLE_VERSION_PROP) + foregroundCol = VERSION_FOREGROUND_PROP; + gtk_tree_view_column_add_attribute (column, renderer, "foreground", foregroundCol); g_object_set_data_full (G_OBJECT (column), "property", g_strdup (prop.c_str()), g_free); if (identAuto) gtk_tree_view_column_add_attribute (column, renderer, "xpad", XPAD_PROP); @@ -1166,15 +1173,18 @@ static void apply (Ypp::Package *package, Action action, bool enable) { - if (enable) + if (package->toModify()) + package->undo(); + else +// if (enable) switch (action) { case INSTALL_ACTION: package->install (0); break; case REMOVE_ACTION: package->remove(); break; case UNDO_ACTION: package->undo(); break; case NONE_ACTION: break; } - else - package->undo(); +/* else + package->undo();*/ } static gboolean apply_iter_cb (GtkTreeModel *model, Modified: trunk/gtk/src/pkg/yzyppwrapper.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/pkg/yzyppwrapper.cc?rev=… ============================================================================== --- trunk/gtk/src/pkg/yzyppwrapper.cc (original) +++ trunk/gtk/src/pkg/yzyppwrapper.cc Thu Jan 7 01:36:18 2010 @@ -1,10 +1,7 @@ /******************************************************************** * YaST2-GTK - http://en.opensuse.org/YaST2-GTK * ********************************************************************/ - -/* - Textdomain "yast2-gtk" - */ +/* Textdomain "yast2-gtk" */ /* Ypp, zypp wrapper */ // check the header file for information about this wrapper @@ -37,7 +34,6 @@ #include "YGUtils.h" //** Zypp shortcuts - typedef zypp::ResPoolProxy ZyppPool; inline ZyppPool zyppPool() { return zypp::getZYpp()->poolProxy(); } typedef zypp::ui::Selectable::Ptr ZyppSelectable; @@ -55,11 +51,10 @@ { return zypp::dynamic_pointer_cast <const zypp::Pattern> (obj); } //** Utilities - // converts a set of tree representation in a form of a strings to a tree structure. // String tree representations are, for instance, filenames: /dir1/dir2/file struct StringTree { - typedef int (*Compare)(const char *, const char *); + typedef int (* Compare)(const char *, const char *); Compare compare; char delim; const char *trans_domain; @@ -208,7 +203,6 @@ std::vector <Repository *> repos; const Repository *favoriteRepo; int favoriteRepoPriority; - Disk *disk; Interface *interface; GSList *pkg_listeners; bool inTransaction; @@ -2225,6 +2219,20 @@ } }; +//** Disk + +static Ypp::Disk *_disk; + +Ypp::Disk *Ypp::getDisk() +{ + if (!_disk) + _disk = new Disk(); + return _disk; +} + +void finish_disk() +{ delete _disk; _disk = 0; } + Ypp::Disk::Disk() { impl = new Impl(); } Ypp::Disk::~Disk() @@ -2244,6 +2252,8 @@ return NULL; } +//** Categories + Ypp::Node *Ypp::getFirstCategory2 (Ypp::Package::Type type) { impl->getPackages (type); @@ -2325,7 +2335,7 @@ //** Ypp top methods & internal connections Ypp::Impl::Impl() -: favoriteRepo (NULL), disk (NULL), interface (NULL), +: favoriteRepo (NULL), interface (NULL), pkg_listeners (NULL), inTransaction (false), transactions (NULL) { for (int i = 0; i < Package::TOTAL_TYPES; i++) { @@ -2355,7 +2365,6 @@ // don't delete pools as they don't actually belong to Ypp, just listeners g_slist_free (pkg_listeners); - delete disk; } const Ypp::Repository *Ypp::Impl::getRepository (int nb) @@ -2424,13 +2433,6 @@ return i; } -Ypp::Disk *Ypp::Impl::getDisk() -{ - if (!disk) - disk = new Disk(); - return disk; -} - bool Ypp::Impl::acceptLicense (Ypp::Package *package, const std::string &license) { if (interface) @@ -2582,11 +2584,132 @@ g_slist_free (transactions); transactions = NULL; inTransaction = false; - if (disk) - disk->impl->packageModified(); + if (_disk) + _disk->impl->packageModified(); interface->loading (1); } +//** Log + +#include <zypp/parser/HistoryLogReader.h> +typedef zypp::HistoryItem::Ptr ZyppHistoryItem; +typedef zypp::HistoryItemInstall *ZyppHistoryItemInstall; +typedef zypp::HistoryItemRemove *ZyppHistoryItemRemove; +inline ZyppHistoryItemInstall tryCastToZyppHistoryItemInstall (ZyppHistoryItem item) +{ return dynamic_cast <zypp::HistoryItemInstall *> (item.get()); } +inline ZyppHistoryItemRemove tryCastToZyppHistoryItemRemove (ZyppHistoryItem item) +{ return dynamic_cast <zypp::HistoryItemRemove *> (item.get()); } + +struct Ypp::Log::Impl +{ + std::vector <Log::Entry *> entries; + Impl(); + ~Impl(); + void parse (const ZyppHistoryItem &item); +}; + +struct HistoryParseHandler { + Ypp::Log::Impl *log; + HistoryParseHandler (Ypp::Log::Impl *log) : log (log) {} + + bool operator()(const ZyppHistoryItem &item) { + log->parse (item); + return true; + } +}; + +Ypp::Log::Impl::Impl() +{ + entries.reserve (512); + HistoryParseHandler handler (this); + const char *filename = "/var/log/zypp/history"; + zypp::parser::HistoryLogReader reader (filename, boost::ref (handler)); + try { + reader.readAll(); + } + catch (const zypp::Exception &e) { + yuiWarning () << "Error: Could not load log file" << filename << ": " + << e.asUserHistory() << std::endl; + } +} + +Ypp::Log::Impl::~Impl() +{ + for (std::vector <Log::Entry *>::iterator it = entries.begin(); + it != entries.end(); it++) + delete *it; +} + +void Ypp::Log::Impl::parse (const ZyppHistoryItem &item) +{ + Entry::Action action; + if (item->action.toEnum() == zypp::HistoryActionID::INSTALL_e) + action = Entry::INSTALL_ACTION; + else if (item->action.toEnum() == zypp::HistoryActionID::REMOVE_e) + action = Entry::REMOVE_ACTION; + else + return; + + Entry *entry = new Entry(); + entry->action = action; + entry->date = item->date; + if (action == Entry::INSTALL_ACTION) { + ZyppHistoryItemInstall _item = tryCastToZyppHistoryItemInstall (item); + if (_item) { + entry->package_str = _item->name; + entry->version_str = _item->edition.asString(); + } + } + else { + ZyppHistoryItemRemove _item = tryCastToZyppHistoryItemRemove (item); + if (_item) { + entry->package_str = _item->name; + entry->version_str = _item->edition.asString(); + } + } +//fprintf (stderr, "parse: %s\n", entry->package_str.c_str()); + entry->package = Ypp::get()->getPackages (Ypp::Package::PACKAGE_TYPE).find (entry->package_str); + entry->version = NULL; + if (entry->package) { + const Package::Version *version = entry->package->getInstalledVersion(); + if (version && version->number == entry->version_str) + entry->version = version; + for (int i = 0; entry->package->getAvailableVersion (i); i++) { + version = entry->package->getAvailableVersion (i); + if (version->number == entry->version_str) + entry->version = version; + } + } + entries.push_back (entry); +} + +Ypp::Log::Log() +{ impl = new Impl(); } + +Ypp::Log::~Log() +{ delete impl; } + +Ypp::Log::Entry *Ypp::Log::getEntry (int nb) +{ + if (nb >= (signed) impl->entries.size()) + return NULL; + return impl->entries[impl->entries.size()-nb-1]; // reverse order +} + +static Ypp::Log *_log = 0; + +Ypp::Log *Ypp::getLog() +{ + if (!_log) + _log = new Log(); + return _log; +} + +void finish_log() +{ delete _log; _log = 0; } + +//** Top + Ypp::Ypp() { impl = new Impl(); @@ -2599,6 +2722,8 @@ { setFavoriteRepository (NULL); delete impl; + finish_log(); + finish_disk(); } const Ypp::Repository *Ypp::getRepository (int nb) @@ -2623,9 +2748,6 @@ const Ypp::Repository *Ypp::favoriteRepository() { return impl->favoriteRepo; } -Ypp::Disk *Ypp::getDisk() -{ return impl->getDisk(); } - void Ypp::setInterface (Ypp::Interface *interface) { impl->interface = interface; Modified: trunk/gtk/src/pkg/yzyppwrapper.h URL: http://svn.opensuse.org/viewcvs/yast/trunk/gtk/src/pkg/yzyppwrapper.h?rev=6… ============================================================================== --- trunk/gtk/src/pkg/yzyppwrapper.h (original) +++ trunk/gtk/src/pkg/yzyppwrapper.h Thu Jan 7 01:36:18 2010 @@ -258,6 +258,29 @@ void *impl; }; + // Log + struct Log { + struct Entry { + enum Action { INSTALL_ACTION, REMOVE_ACTION }; + Action action; + std::string date; + std::string package_str; + std::string version_str; + + // attemps to match the strings: could be null + Package *package; + const Package::Version *version; // only may be set for INSTALL + }; + + Entry *getEntry (int nb); + + struct Impl; + Impl *impl; + Log(); + ~Log(); + }; + Log *getLog(); + // Module static Ypp *get(); static void finish(); -- 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] r60249 - in /branches/SuSE-Code-11-SP1-Branch/storage/libstorage/src: Storage.cc StorageDefines.h
by aschnell＠svn.opensuse.org 06 Jan '10

06 Jan '10

Author: aschnell Date: Wed Jan 6 20:18:01 2010 New Revision: 60249 URL: http://svn.opensuse.org/viewcvs/yast?rev=60249&view=rev Log: - added logging Modified: branches/SuSE-Code-11-SP1-Branch/storage/libstorage/src/Storage.cc branches/SuSE-Code-11-SP1-Branch/storage/libstorage/src/StorageDefines.h Modified: branches/SuSE-Code-11-SP1-Branch/storage/libstorage/src/Storage.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/stora… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/storage/libstorage/src/Storage.cc (original) +++ branches/SuSE-Code-11-SP1-Branch/storage/libstorage/src/Storage.cc Wed Jan 6 20:18:01 2010 @@ -148,6 +148,8 @@ getline( File, line ); File.close(); y2mil( "kernel version:" << line ); + + SystemCmd(LSBIN " -1 /lib/modules"); } Modified: branches/SuSE-Code-11-SP1-Branch/storage/libstorage/src/StorageDefines.h URL: http://svn.opensuse.org/viewcvs/yast/branches/SuSE-Code-11-SP1-Branch/stora… ============================================================================== --- branches/SuSE-Code-11-SP1-Branch/storage/libstorage/src/StorageDefines.h (original) +++ branches/SuSE-Code-11-SP1-Branch/storage/libstorage/src/StorageDefines.h Wed Jan 6 20:18:01 2010 @@ -75,5 +75,7 @@ #define BLKIDBIN "/sbin/blkid" +#define LSBIN "/bin/ls" + #endif -- 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] r60248 - /tags/stable-2_19_0/pkg-bindings/
by lslezak＠svn.opensuse.org 06 Jan '10

06 Jan '10

Author: lslezak Date: Wed Jan 6 20:14:04 2010 New Revision: 60248 URL: http://svn.opensuse.org/viewcvs/yast?rev=60248&view=rev Log: Created tag stable-2_19_0 for pkg-bindings Added: tags/stable-2_19_0/pkg-bindings/ - copied from r60247, trunk/pkg-bindings/ -- 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] r60247 - in /trunk/pkg-bindings: VERSION package/yast2-pkg-bindings.changes src/Package.cc src/Resolvable_Properties.cc
by lslezak＠svn.opensuse.org 06 Jan '10

06 Jan '10

Author: lslezak Date: Wed Jan 6 20:13:58 2010 New Revision: 60247 URL: http://svn.opensuse.org/viewcvs/yast?rev=60247&view=rev Log: - Pkg::ResolvableProperties() - always return "transact_by" flag, Pkg::PkgApplReset() - do not reset "transact_by" flag for non-transacting resolvables (bnc#450786, bnc#471340) - 2.19.0 Modified: trunk/pkg-bindings/VERSION trunk/pkg-bindings/package/yast2-pkg-bindings.changes trunk/pkg-bindings/src/Package.cc trunk/pkg-bindings/src/Resolvable_Properties.cc Modified: trunk/pkg-bindings/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/pkg-bindings/VERSION?rev=60247&r… ============================================================================== --- trunk/pkg-bindings/VERSION (original) +++ trunk/pkg-bindings/VERSION Wed Jan 6 20:13:58 2010 @@ -1 +1 @@ -2.18.11 +2.19.0 Modified: trunk/pkg-bindings/package/yast2-pkg-bindings.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/pkg-bindings/package/yast2-pkg-b… ============================================================================== --- trunk/pkg-bindings/package/yast2-pkg-bindings.changes (original) +++ trunk/pkg-bindings/package/yast2-pkg-bindings.changes Wed Jan 6 20:13:58 2010 @@ -1,4 +1,12 @@ ------------------------------------------------------------------- +Wed Jan 6 17:50:26 UTC 2010 - lslezak(a)suse.cz + +- Pkg::ResolvableProperties() - always return "transact_by" flag, + Pkg::PkgApplReset() - do not reset "transact_by" flag for + non-transacting resolvables (bnc#450786, bnc#471340) +- 2.19.0 + +------------------------------------------------------------------- Thu Dec 3 08:19:45 UTC 2009 - lslezak(a)suse.cz - return more details when a service refresh fails (bnc#558049) Modified: trunk/pkg-bindings/src/Package.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/pkg-bindings/src/Package.cc?rev=… ============================================================================== --- trunk/pkg-bindings/src/Package.cc (original) +++ trunk/pkg-bindings/src/Package.cc Wed Jan 6 20:13:58 2010 @@ -1727,7 +1727,10 @@ for_(it, proxy.byKindBegin<T>(), proxy.byKindEnd<T>()) { zypp::ui::Selectable::Ptr s = (*it); - if (s) s->theObj().status().resetTransact(level); + if (s && (level == zypp::ResStatus::USER || s->theObj().status().transacts())) + { + s->theObj().status().resetTransact(level); + } } } Modified: trunk/pkg-bindings/src/Resolvable_Properties.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/pkg-bindings/src/Resolvable_Prop… ============================================================================== --- trunk/pkg-bindings/src/Resolvable_Properties.cc (original) +++ trunk/pkg-bindings/src/Resolvable_Properties.cc Wed Jan 6 20:13:58 2010 @@ -193,14 +193,12 @@ if (status.isToBeInstalled()) { stat = "selected"; - info->add(YCPString("transact_by"), YCPSymbol(TransactToString(status.getTransactByValue()))); } else if (status.isInstalled() || status.isSatisfied()) { if (status.isToBeUninstalled()) { stat = "removed"; - info->add(YCPString("transact_by"), YCPSymbol(TransactToString(status.getTransactByValue()))); } else { @@ -212,6 +210,8 @@ stat = "available"; } + info->add(YCPString("transact_by"), YCPSymbol(TransactToString(status.getTransactByValue()))); + info->add(YCPString("status"), YCPSymbol(stat)); // is the resolvable locked? (Locked or Taboo in the UI) -- 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] r60246 - /tags/stable-2_19_2/packager/
by lslezak＠svn.opensuse.org 06 Jan '10

06 Jan '10

Author: lslezak Date: Wed Jan 6 20:10:34 2010 New Revision: 60246 URL: http://svn.opensuse.org/viewcvs/yast?rev=60246&view=rev Log: Created tag stable-2_19_2 for packager Added: tags/stable-2_19_2/packager/ - copied from r60245, trunk/packager/ -- 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] r60245 - in /trunk/packager: VERSION package/yast2-packager.changes src/modules/Packages.ycp
by lslezak＠svn.opensuse.org 06 Jan '10

06 Jan '10

Author: lslezak Date: Wed Jan 6 20:10:25 2010 New Revision: 60245 URL: http://svn.opensuse.org/viewcvs/yast?rev=60245&view=rev Log: - do not reselect patterns which have been deselected by user (bnc#450786, bnc#471340) - 2.19.2 Modified: trunk/packager/VERSION trunk/packager/package/yast2-packager.changes trunk/packager/src/modules/Packages.ycp Modified: trunk/packager/VERSION URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/VERSION?rev=60245&r1=60… ============================================================================== --- trunk/packager/VERSION (original) +++ trunk/packager/VERSION Wed Jan 6 20:10:25 2010 @@ -1 +1 @@ -2.19.1 +2.19.2 Modified: trunk/packager/package/yast2-packager.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/package/yast2-packager.… ============================================================================== --- trunk/packager/package/yast2-packager.changes (original) +++ trunk/packager/package/yast2-packager.changes Wed Jan 6 20:10:25 2010 @@ -1,4 +1,11 @@ ------------------------------------------------------------------- +Wed Jan 6 17:48:18 UTC 2010 - lslezak(a)suse.cz + +- do not reselect patterns which have been deselected by user + (bnc#450786, bnc#471340) +- 2.19.2 + +------------------------------------------------------------------- Mon Jan 4 16:18:08 CET 2010 - locilka(a)suse.cz - Fixed accepting/declining the multilicense agreement. Modified: trunk/packager/src/modules/Packages.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/packager/src/modules/Packages.yc… ============================================================================== --- trunk/packager/src/modules/Packages.ycp (original) +++ trunk/packager/src/modules/Packages.ycp Wed Jan 6 20:10:25 2010 @@ -1649,7 +1649,15 @@ { y2milestone ("Selecting system patterns %1", system_patterns); foreach (string p, system_patterns, { - Pkg::ResolvableInstall (p, `pattern); + map prop = Pkg::ResolvableProperties(p, `pattern, "")[0]:$[]; + if (prop["status"]:nil == `available && prop["transact_by"]:nil == `user) + { + y2milestone("Ignoring deselected pattern '%1'", p); + } + else + { + Pkg::ResolvableInstall (p, `pattern); + } }); } else -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0