Hello community,

here is the log from the commit of package yast2-pkg-bindings for 
openSUSE:Factory checked in at 2017-10-28 14:19:53
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-pkg-bindings (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-pkg-bindings.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-pkg-bindings"

Sat Oct 28 14:19:53 2017 rev:193 rq:537206 version:4.0.4

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-pkg-bindings/yast2-pkg-bindings.changes    
2017-10-07 17:49:03.107560435 +0200
+++ 
/work/SRC/openSUSE:Factory/.yast2-pkg-bindings.new/yast2-pkg-bindings.changes   
    2017-10-28 14:19:52.591517805 +0200
@@ -1,0 +2,21 @@
+Fri Oct 27 14:40:23 UTC 2017 - [email protected]
+
+- Pkg.ResolvableProperties: return the "register_flavor" product
+  property (related to bsc#896224)
+- 4.0.4
+
+-------------------------------------------------------------------
+Tue Oct 24 14:51:16 UTC 2017 - [email protected]
+
+- Report error when repository refresh fails during a service
+  refresh (bsc#1064210)
+- 4.0.3
+
+-------------------------------------------------------------------
+Mon Oct 23 11:57:37 UTC 2017 - [email protected]
+
+- Do not crash after a callback is set, unset, then called
+  (bsc#1063459)
+- 4.0.2
+
+-------------------------------------------------------------------

Old:
----
  yast2-pkg-bindings-4.0.1.tar.bz2

New:
----
  yast2-pkg-bindings-4.0.4.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-pkg-bindings-devel-doc.spec ++++++
--- /var/tmp/diff_new_pack.G4WE2j/_old  2017-10-28 14:19:53.387488702 +0200
+++ /var/tmp/diff_new_pack.G4WE2j/_new  2017-10-28 14:19:53.387488702 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-pkg-bindings-devel-doc
-Version:        4.0.1
+Version:        4.0.4
 Release:        0
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
 Source0:        yast2-pkg-bindings-%{version}.tar.bz2

++++++ yast2-pkg-bindings.spec ++++++
--- /var/tmp/diff_new_pack.G4WE2j/_old  2017-10-28 14:19:53.411487825 +0200
+++ /var/tmp/diff_new_pack.G4WE2j/_new  2017-10-28 14:19:53.415487678 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-pkg-bindings
-Version:        4.0.1
+Version:        4.0.4
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

++++++ yast2-pkg-bindings-4.0.1.tar.bz2 -> yast2-pkg-bindings-4.0.4.tar.bz2 
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-pkg-bindings-4.0.1/package/yast2-pkg-bindings-devel-doc.spec 
new/yast2-pkg-bindings-4.0.4/package/yast2-pkg-bindings-devel-doc.spec
--- old/yast2-pkg-bindings-4.0.1/package/yast2-pkg-bindings-devel-doc.spec      
2017-09-27 15:42:21.373200836 +0200
+++ new/yast2-pkg-bindings-4.0.4/package/yast2-pkg-bindings-devel-doc.spec      
2017-10-27 16:58:52.673477779 +0200
@@ -16,7 +16,7 @@
 #
 
 Name:           yast2-pkg-bindings-devel-doc
-Version:        4.0.1
+Version:        4.0.4
 Release:        0
 License:        GPL-2.0
 Group:          Documentation/HTML
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-pkg-bindings-4.0.1/package/yast2-pkg-bindings.changes 
new/yast2-pkg-bindings-4.0.4/package/yast2-pkg-bindings.changes
--- old/yast2-pkg-bindings-4.0.1/package/yast2-pkg-bindings.changes     
2017-09-27 15:42:21.373200836 +0200
+++ new/yast2-pkg-bindings-4.0.4/package/yast2-pkg-bindings.changes     
2017-10-27 16:58:52.673477779 +0200
@@ -1,4 +1,25 @@
 -------------------------------------------------------------------
+Fri Oct 27 14:40:23 UTC 2017 - [email protected]
+
+- Pkg.ResolvableProperties: return the "register_flavor" product
+  property (related to bsc#896224)
+- 4.0.4
+
+-------------------------------------------------------------------
+Tue Oct 24 14:51:16 UTC 2017 - [email protected]
+
+- Report error when repository refresh fails during a service
+  refresh (bsc#1064210)
+- 4.0.3
+
+-------------------------------------------------------------------
+Mon Oct 23 11:57:37 UTC 2017 - [email protected]
+
+- Do not crash after a callback is set, unset, then called
+  (bsc#1063459)
+- 4.0.2
+
+-------------------------------------------------------------------
 Mon Sep 25 10:30:05 UTC 2017 - [email protected]
 
 - Add a CompareVersions function (related to fate#323273)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-pkg-bindings-4.0.1/package/yast2-pkg-bindings.spec 
new/yast2-pkg-bindings-4.0.4/package/yast2-pkg-bindings.spec
--- old/yast2-pkg-bindings-4.0.1/package/yast2-pkg-bindings.spec        
2017-09-27 15:42:21.373200836 +0200
+++ new/yast2-pkg-bindings-4.0.4/package/yast2-pkg-bindings.spec        
2017-10-27 16:58:52.673477779 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-pkg-bindings
-Version:        4.0.1
+Version:        4.0.4
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-pkg-bindings-4.0.1/src/Callbacks.YCP.cc 
new/yast2-pkg-bindings-4.0.4/src/Callbacks.YCP.cc
--- old/yast2-pkg-bindings-4.0.1/src/Callbacks.YCP.cc   2017-09-27 
15:42:21.377200836 +0200
+++ new/yast2-pkg-bindings-4.0.4/src/Callbacks.YCP.cc   2017-10-27 
16:58:52.677477779 +0200
@@ -189,15 +189,21 @@
        const _cbdata_t::const_iterator tmp1 = _cbdata.find(id_r);
 
        if (tmp1 == _cbdata.end())
+       {
+           y2debug ("Callback %s not found", cbName(id_r).c_str());
            return NULL;
+       }
+       if (tmp1->second.empty())
+       {
+           y2debug ("Callback %s is empty", cbName(id_r).c_str());
+           return NULL;
+       }
 
        const YCPReference func(tmp1->second.top());
 
        if (func.isNull() || ! func->isReference())
        {
-           // TODO
-//         ycp2error ("Unexpected function pointer: %s"
-//             , ptr.isNull () ? "NULL" : ptr->toString ().c_str ());
+           y2debug ("Callback %s is not a func reference", 
cbName(id_r).c_str());
            return NULL;
        }
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-pkg-bindings-4.0.1/src/Resolvable_Properties.cc 
new/yast2-pkg-bindings-4.0.4/src/Resolvable_Properties.cc
--- old/yast2-pkg-bindings-4.0.1/src/Resolvable_Properties.cc   2017-09-27 
15:42:21.389200836 +0200
+++ new/yast2-pkg-bindings-4.0.4/src/Resolvable_Properties.cc   2017-10-27 
16:58:52.685477779 +0200
@@ -92,6 +92,8 @@
         + "smolt_urls"
        + "register_target"
        + "register_release"
+       + "register_flavor" -> string: kind of the product 
(module/extension/...)
+        or empty string if not defined
        + "flavor"
         + "replaces"
           + "name"
@@ -390,6 +392,7 @@
        // registration data
        info->add(YCPString("register_target"), 
YCPString(product->registerTarget()));
        info->add(YCPString("register_release"), 
YCPString(product->registerRelease()));
+       info->add(YCPString("register_flavor"), 
YCPString(product->registerFlavor()));
        info->add(YCPString("product_line"), YCPString(product->productLine()));
 
        // Live CD, FTP Edition...
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-pkg-bindings-4.0.1/src/Service.cc 
new/yast2-pkg-bindings-4.0.4/src/Service.cc
--- old/yast2-pkg-bindings-4.0.1/src/Service.cc 2017-09-27 15:42:21.389200836 
+0200
+++ new/yast2-pkg-bindings-4.0.4/src/Service.cc 2017-10-27 16:58:52.685477779 
+0200
@@ -433,7 +433,12 @@
           {
             y2milestone("Refreshing repository: %s", it->alias().c_str());
             // refresh the last added repository
-            SourceRefreshNow(repos.size() - 1);
+
+            YCPValue refreshed = SourceRefreshNow(repos.size() - 1);
+            // return false on refresh failure
+            if (!refreshed.isNull() && refreshed->isBoolean()
+                && !refreshed->asBoolean()->value())
+                return refreshed;
 
             // load resolvables
             PkgProgress pkgprogress(_callbackHandler);
@@ -441,7 +446,9 @@
             progress.sendTo(pkgprogress.Receiver());
             zypp::CombinedProgressData subprogrcv_ref(progress, 20);
 
-            LoadResolvablesFrom(new_repo, subprogrcv_ref);
+            bool loaded = LoadResolvablesFrom(new_repo, subprogrcv_ref);
+            // return false on resolvable load failure
+            if (!loaded) return YCPBoolean(false);
           }
         }
 


Reply via email to