Hello community,

here is the log from the commit of package rubygem-cfa for openSUSE:Factory 
checked in at 2020-01-02 14:40:21
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/rubygem-cfa (Old)
 and      /work/SRC/openSUSE:Factory/.rubygem-cfa.new.6675 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "rubygem-cfa"

Thu Jan  2 14:40:21 2020 rev:13 rq:760046 version:1.0.2

Changes:
--------
--- /work/SRC/openSUSE:Factory/rubygem-cfa/rubygem-cfa.changes  2019-07-23 
22:27:42.499075588 +0200
+++ /work/SRC/openSUSE:Factory/.rubygem-cfa.new.6675/rubygem-cfa.changes        
2020-01-02 14:40:39.744895695 +0100
@@ -1,0 +2,7 @@
+Mon Dec 30 11:57:49 UTC 2019 - Josef Reidinger <[email protected]>
+
+- fix writting multiple new elements to previously single element
+  (found during fixing bsc#1156929)
+- 1.0.2
+
+-------------------------------------------------------------------

Old:
----
  cfa-1.0.1.gem

New:
----
  cfa-1.0.2.gem

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

Other differences:
------------------
++++++ rubygem-cfa.spec ++++++
--- /var/tmp/diff_new_pack.PS9agz/_old  2020-01-02 14:40:40.116895842 +0100
+++ /var/tmp/diff_new_pack.PS9agz/_new  2020-01-02 14:40:40.120895844 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           rubygem-cfa
-Version:        1.0.1
+Version:        1.0.2
 Release:        0
 %define mod_name cfa
 %define mod_full_name %{mod_name}-%{version}

++++++ cfa-1.0.1.gem -> cfa-1.0.2.gem ++++++
Binary files old/checksums.yaml.gz and new/checksums.yaml.gz differ
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/lib/cfa/augeas_parser/writer.rb 
new/lib/cfa/augeas_parser/writer.rb
--- old/lib/cfa/augeas_parser/writer.rb 2019-07-17 09:25:50.000000000 +0200
+++ new/lib/cfa/augeas_parser/writer.rb 2019-12-30 14:03:23.000000000 +0100
@@ -293,7 +293,15 @@
       # @param located_entry [LocatedEntry] entry to insert
       # @return [String] where value should be written.
       def insert_after(preceding, located_entry)
-        aug.insert(preceding.path, located_entry.key, false)
+        res = aug.insert(preceding.path, located_entry.key, false)
+        # if insert failed it means, that previous preceding entry was single
+        # element and now it is multiple ones, so try to first element as add
+        # is done in reverse order
+        if res == -1
+          # TODO: what about deep nesting of trees where upper level change
+          # from single to collection?
+          aug.insert(preceding.path + "[1]", located_entry.key, false)
+        end
         path_after(preceding)
       end
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/lib/cfa/base_model.rb new/lib/cfa/base_model.rb
--- old/lib/cfa/base_model.rb   2019-07-17 09:25:50.000000000 +0200
+++ new/lib/cfa/base_model.rb   2019-12-30 14:03:23.000000000 +0100
@@ -109,7 +109,7 @@
           tree_value_plain(generic_get(key))
         end
 
-        define_method(:"#{method_name.to_s}=") do |value|
+        define_method(:"#{method_name}=") do |value|
           tree_value_change(key, value)
         end
       end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/metadata new/metadata
--- old/metadata        2019-07-17 09:26:11.000000000 +0200
+++ new/metadata        2019-12-30 14:03:29.000000000 +0100
@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cfa
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Josef Reidinger
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-07-17 00:00:00.000000000 Z
+date: 2019-12-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-augeas
@@ -61,7 +61,7 @@
       version: 1.3.6
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.3
+rubygems_version: 2.7.6.2
 signing_key: 
 specification_version: 4
 summary: CFA (Config Files API) provides an easy way to create models on top 
of configuration


Reply via email to