Merge pull request #27 from yast/no_internal_log

do not use y2internal as it is reserved for component system internals
yast · Oct 21, 2013 · be23b6d · be23b6d
2 parents 9558fe7 + d475ecf
commit be23b6d
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 8 deletions.
diff --git a/src/clients/repositories.rb b/src/clients/repositories.rb
@@ -1114,7 +1114,7 @@ def SummaryDialog
 
           if Ops.less_than(id, 0) && @repository_view &&
               @displayed_service == ""
-            Builtins.y2internal(
+            Builtins.y2error(
               "Unable to determine repository id, broken repository?"
             )
             next
@@ -1399,7 +1399,7 @@ def SummaryDialog
                 ReposFromService(@displayed_service, @sourceStatesOut) :
               @serviceStatesOut
 
-            Builtins.y2internal("data: %1", data)
+            Builtins.y2milestone("data: %1", data)
 
             Builtins.foreach(data) do |src_state|
               if Ops.get_boolean(src_state, "enabled", false) &&
@@ -1658,7 +1658,7 @@ def SummaryDialog
               new_keep = Convert.to_boolean(
                 UI.QueryWidget(Id(:keeppackages), :Value)
               )
-              Builtins.y2internal("New keep packages option: %1", new_keep)
+              Builtins.y2milestone("New keep packages option: %1", new_keep)
 
               Ops.set(sourceState, "keeppackages", new_keep)
               Ops.set(@sourceStatesOut, global_current, sourceState)

diff --git a/src/clients/sw_single.rb b/src/clients/sw_single.rb
@@ -504,7 +504,7 @@ def StartSWSingle
 
       # an extra step is needed in dir inst mode
       if Installation.destdir != "/"
-        Builtins.y2internal("Extra step is needed")
+        Builtins.y2milestone("Extra step is needed")
         # %1 is path to the target system (e.g. /tmp/dirinstall
         Builtins.sformat(
           _("Reset the target system to %1"),

diff --git a/src/modules/PackageSlideShow.rb b/src/modules/PackageSlideShow.rb
@@ -1343,7 +1343,7 @@ def SlideDisplayStart(pkg_name, pkg_location, pkg_summary, pkg_size, deleting)
 
       # remove path
       pkg_filename = StripPath(pkg_location)
-      Builtins.y2internal("pkg_name: %1", pkg_name)
+      Builtins.y2milestone("pkg_name: %1", pkg_name)
 
       if deleting
         pkg_size = -1

diff --git a/src/modules/Packages.rb b/src/modules/Packages.rb
@@ -1560,7 +1560,7 @@ def IntegrateServicePack(show_popup, base_url)
           AddOnProduct.UpdateInstSys(Ops.add(spdir, "/y2update.tgz"))
         end
         @theSources = Builtins.add(@theSources, sp_source)
-        Builtins.y2internal(
+        Builtins.y2milestone(
           "Service pack repository: %1, changing to URL: %2",
           sp_source,
           base_url

diff --git a/src/modules/SourceDialogs.rb b/src/modules/SourceDialogs.rb
@@ -2184,7 +2184,7 @@ def SelectStore(key, event)
           @_url = "commrepos://"
         end
       else
-        Builtins.y2internal("Unexpected repo type %1", selected)
+        Builtins.y2error("Unexpected repo type %1", selected)
       end
 
       nil

diff --git a/src/modules/SpaceCalculation.rb b/src/modules/SpaceCalculation.rb
@@ -699,7 +699,7 @@ def get_partition_info
                     )
                     Builtins.foreach(partition) do |p|
                       if Ops.get_string(p, "name", "") == tmpdir
-                        Builtins.y2internal("P: %1", p)
+                        Builtins.y2milestone("P: %1", p)
                         free_size = Ops.multiply(
                           Builtins.tointeger(Ops.get_string(p, "free", "0")),
                           1024