Hello community,

here is the log from the commit of package python-jenkins-job-builder for 
openSUSE:Factory checked in at 2017-04-12 17:33:17
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/python-jenkins-job-builder (Old)
 and      /work/SRC/openSUSE:Factory/.python-jenkins-job-builder.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "python-jenkins-job-builder"

Wed Apr 12 17:33:17 2017 rev:6 rq:482931 version:1.6.1

Changes:
--------
--- 
/work/SRC/openSUSE:Factory/python-jenkins-job-builder/python-jenkins-job-builder.changes
    2017-03-17 15:07:21.490973979 +0100
+++ 
/work/SRC/openSUSE:Factory/.python-jenkins-job-builder.new/python-jenkins-job-builder.changes
       2017-04-12 18:19:54.538626644 +0200
@@ -1,0 +2,5 @@
+Wed Mar 22 11:49:09 GMT 2017 - [email protected]
+
+- Fix backport of patches for log parser plugin
+
+-------------------------------------------------------------------

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

Other differences:
------------------
++++++ 0002-Update-logparser-plugin.patch ++++++
--- /var/tmp/diff_new_pack.RDKqTR/_old  2017-04-12 18:19:55.714460374 +0200
+++ /var/tmp/diff_new_pack.RDKqTR/_new  2017-04-12 18:19:55.718459808 +0200
@@ -1,22 +1,23 @@
-From a07c886c89f1afca70642ca5280716523dc8cef9 Mon Sep 17 00:00:00 2001
+From 1b40d4aec9a1311c4ffa2b86d157df761e6686c0 Mon Sep 17 00:00:00 2001
 From: Dong Ma <[email protected]>
 Date: Wed, 28 Sep 2016 23:50:31 -0700
-Subject: [PATCH] Update logparser plugin
+Subject: [PATCH 1/2] Update logparser plugin docstring
 
-- update logparser plugin to user convert xml
-- update logparser docstring
+This is a partial backport, in order to keep the backport of
+d3307d6938edfa45470a454b2092fe25ec6870f2 as clean as possible.
 
-Change-Id: I13b36024bcaa3cfca2c8f2d3530bbc31dc9f52a8
+(cherry picked from commit a07c886c89f1afca70642ca5280716523dc8cef9)
+
+Change-Id: I0f9f9f1248051cc6de855b803154175009781787
 ---
- jenkins_jobs/modules/publishers.py         | 18 ++++++++++--------
- tests/publishers/fixtures/logparser001.xml |  2 +-
- 2 files changed, 11 insertions(+), 9 deletions(-)
+ jenkins_jobs/modules/publishers.py | 5 +++--
+ 1 file changed, 3 insertions(+), 2 deletions(-)
 
 diff --git a/jenkins_jobs/modules/publishers.py 
b/jenkins_jobs/modules/publishers.py
-index 2a36c23..7ec9280 100644
+index ce2574d..47782da 100644
 --- a/jenkins_jobs/modules/publishers.py
 +++ b/jenkins_jobs/modules/publishers.py
-@@ -2146,9 +2146,10 @@ def logparser(registry, xml_parent, data):
+@@ -2076,9 +2076,10 @@ def logparser(parser, xml_parent, data):
      """yaml: logparser
      Requires the Jenkins :jenkins-wiki:`Log Parser Plugin 
<Log+Parser+Plugin>`.
  
@@ -29,39 +30,6 @@
  
      Example:
  
-@@ -2158,12 +2159,13 @@ def logparser(registry, xml_parent, data):
- 
-     clog = XML.SubElement(xml_parent,
-                           'hudson.plugins.logparser.LogParserPublisher')
--    XML.SubElement(clog, 'unstableOnWarning').text = \
--        str(data.get('unstable-on-warning', False)).lower()
--    XML.SubElement(clog, 'failBuildOnError').text = \
--        str(data.get('fail-on-error', False)).lower()
--    # v1.08: this must be the full path, the name of the rules is not enough
--    XML.SubElement(clog, 'parsingRulesPath').text = data.get('parse-rules', 
'')
-+    clog.set('plugin', 'log-parser')
-+    mappings = [
-+        ('unstable-on-warning', 'unstableOnWarning', False),
-+        ('fail-on-error', 'failBuildOnError', False),
-+        ('parse-rules', 'parsingRulesPath', '')
-+    ]
-+    helpers.convert_mapping_to_xml(clog, data, mappings, fail_required=True)
- 
- 
- def copy_to_master(parser, xml_parent, data):
-diff --git a/tests/publishers/fixtures/logparser001.xml 
b/tests/publishers/fixtures/logparser001.xml
-index 44881b7..15f3f8c 100644
---- a/tests/publishers/fixtures/logparser001.xml
-+++ b/tests/publishers/fixtures/logparser001.xml
-@@ -1,7 +1,7 @@
- <?xml version="1.0" encoding="utf-8"?>
- <project>
-   <publishers>
--    <hudson.plugins.logparser.LogParserPublisher>
-+    <hudson.plugins.logparser.LogParserPublisher plugin="log-parser">
-       <unstableOnWarning>true</unstableOnWarning>
-       <failBuildOnError>true</failBuildOnError>
-       <parsingRulesPath>/path/to/parserules</parsingRulesPath>
 -- 
 2.1.2.330.g565301e
 

++++++ 0003-add-two-new-options-to-Log-Parser-Plugin.patch ++++++
--- /var/tmp/diff_new_pack.RDKqTR/_old  2017-04-12 18:19:55.734457546 +0200
+++ /var/tmp/diff_new_pack.RDKqTR/_new  2017-04-12 18:19:55.734457546 +0200
@@ -1,7 +1,7 @@
-From d3307d6938edfa45470a454b2092fe25ec6870f2 Mon Sep 17 00:00:00 2001
+From 6fbcb2856e1dc67671a4cd38abbd502e193ee0cf Mon Sep 17 00:00:00 2001
 From: Adam Spiers <[email protected]>
 Date: Tue, 14 Mar 2017 13:51:08 +0000
-Subject: [PATCH] add two new options to Log Parser Plugin
+Subject: [PATCH 2/2] add two new options to Log Parser Plugin
 
 Support both global and per-project rules, and the showGraphs option,
 both of which became possible in the 2.0 release:
@@ -10,16 +10,18 @@
 
https://github.com/jenkinsci/log-parser-plugin/commit/ecde717b9bc618c2d4c8b2526caf10655272148c
 
https://github.com/jenkinsci/log-parser-plugin/commit/6e75ecc8f9f82a17ec151c879dc37d882909df17
 
-Change-Id: I38b0058467d16ca7e3c4d3dc4edb2c9294551f15
+(backported from commit d3307d6938edfa45470a454b2092fe25ec6870f2)
+
+Change-Id: I1f75bc37706b0d7ce8dc2a9d6158a35dbc685a10
 ---
- jenkins_jobs/modules/publishers.py               | 20 ++++++++++++++++++--
+ jenkins_jobs/modules/publishers.py               | 22 ++++++++++++++++++++--
  tests/publishers/fixtures/logparser-full.xml     | 12 ++++++++++++
  tests/publishers/fixtures/logparser-full.yaml    |  7 +++++++
  tests/publishers/fixtures/logparser-minimal.xml  | 12 ++++++++++++
  tests/publishers/fixtures/logparser-minimal.yaml |  3 +++
  tests/publishers/fixtures/logparser001.xml       | 10 ----------
  tests/publishers/fixtures/logparser001.yaml      |  5 -----
- 7 files changed, 52 insertions(+), 17 deletions(-)
+ 7 files changed, 54 insertions(+), 17 deletions(-)
  create mode 100644 tests/publishers/fixtures/logparser-full.xml
  create mode 100644 tests/publishers/fixtures/logparser-full.yaml
  create mode 100644 tests/publishers/fixtures/logparser-minimal.xml
@@ -28,10 +30,10 @@
  delete mode 100644 tests/publishers/fixtures/logparser001.yaml
 
 diff --git a/jenkins_jobs/modules/publishers.py 
b/jenkins_jobs/modules/publishers.py
-index 18e7e04..646007d 100644
+index 47782da..54f7c66 100644
 --- a/jenkins_jobs/modules/publishers.py
 +++ b/jenkins_jobs/modules/publishers.py
-@@ -2335,23 +2335,39 @@ def logparser(registry, xml_parent, data):
+@@ -2077,13 +2077,24 @@ def logparser(parser, xml_parent, data):
      Requires the Jenkins :jenkins-wiki:`Log Parser Plugin 
<Log+Parser+Plugin>`.
  
      :arg str parse-rules: full path to parse rules (default '')
@@ -57,32 +59,32 @@
         :language: yaml
      """
  
-     clog = XML.SubElement(xml_parent,
-                           'hudson.plugins.logparser.LogParserPublisher')
-     clog.set('plugin', 'log-parser')
-+    rules_path_element = ("projectRulePath"
-+                          if data.get("use-project-rules", True)
+@@ -2094,7 +2105,14 @@ def logparser(parser, xml_parent, data):
+     XML.SubElement(clog, 'failBuildOnError').text = \
+         str(data.get('fail-on-error', False)).lower()
+     # v1.08: this must be the full path, the name of the rules is not enough
+-    XML.SubElement(clog, 'parsingRulesPath').text = data.get('parse-rules', 
'')
++    use_project_rules = data.get("use-project-rules", True)
++    XML.SubElement(clog, 'showGraphs').text = \
++        str(data.get('show-graphs', True)).lower()
++    XML.SubElement(clog, 'useProjectRule').text = \
++        str(use_project_rules).lower()
++    rules_path_element = ("projectRulePath" if use_project_rules
 +                          else "parsingRulesPath")
-     mappings = [
-         ('unstable-on-warning', 'unstableOnWarning', False),
-         ('fail-on-error', 'failBuildOnError', False),
--        ('parse-rules', 'parsingRulesPath', '')
-+        ('show-graphs', 'showGraphs', True),
-+        ('use-project-rules', 'useProjectRule', True),
-+        ('parse-rules', rules_path_element, ''),
-     ]
-     helpers.convert_mapping_to_xml(clog, data, mappings, fail_required=True)
++    XML.SubElement(clog, rules_path_element).text = data.get('parse-rules', 
'')
+ 
  
+ def copy_to_master(parser, xml_parent, data):
 diff --git a/tests/publishers/fixtures/logparser-full.xml 
b/tests/publishers/fixtures/logparser-full.xml
 new file mode 100644
-index 0000000..caac716
+index 0000000..652b478
 --- /dev/null
 +++ b/tests/publishers/fixtures/logparser-full.xml
 @@ -0,0 +1,12 @@
 +<?xml version="1.0" encoding="utf-8"?>
 +<project>
 +  <publishers>
-+    <hudson.plugins.logparser.LogParserPublisher plugin="log-parser">
++    <hudson.plugins.logparser.LogParserPublisher>
 +      <unstableOnWarning>true</unstableOnWarning>
 +      <failBuildOnError>true</failBuildOnError>
 +      <showGraphs>false</showGraphs>
@@ -106,14 +108,14 @@
 +      show-graphs: false
 diff --git a/tests/publishers/fixtures/logparser-minimal.xml 
b/tests/publishers/fixtures/logparser-minimal.xml
 new file mode 100644
-index 0000000..681d63f
+index 0000000..67bc7bd
 --- /dev/null
 +++ b/tests/publishers/fixtures/logparser-minimal.xml
 @@ -0,0 +1,12 @@
 +<?xml version="1.0" encoding="utf-8"?>
 +<project>
 +  <publishers>
-+    <hudson.plugins.logparser.LogParserPublisher plugin="log-parser">
++    <hudson.plugins.logparser.LogParserPublisher>
 +      <unstableOnWarning>false</unstableOnWarning>
 +      <failBuildOnError>false</failBuildOnError>
 +      <showGraphs>true</showGraphs>
@@ -133,14 +135,14 @@
 +      parse-rules: "project-log-parser-rules.txt"
 diff --git a/tests/publishers/fixtures/logparser001.xml 
b/tests/publishers/fixtures/logparser001.xml
 deleted file mode 100644
-index 15f3f8c..0000000
+index 44881b7..0000000
 --- a/tests/publishers/fixtures/logparser001.xml
 +++ /dev/null
 @@ -1,10 +0,0 @@
 -<?xml version="1.0" encoding="utf-8"?>
 -<project>
 -  <publishers>
--    <hudson.plugins.logparser.LogParserPublisher plugin="log-parser">
+-    <hudson.plugins.logparser.LogParserPublisher>
 -      <unstableOnWarning>true</unstableOnWarning>
 -      <failBuildOnError>true</failBuildOnError>
 -      <parsingRulesPath>/path/to/parserules</parsingRulesPath>


Reply via email to