From bbc09cae5601a691fdb3121469ac8f2d1f570584 Mon Sep 17 00:00:00 2001 From: Petr Písař <ppi...@redhat.com> Date: Oct 04 2017 14:56:14 +0000 Subject: Adapt to changes in JSON-PP-0.92
--- diff --git a/SQL-Translator-0.11021-Adapt-tests-to-JSON-PP-2.92.patch b/SQL-Translator-0.11021-Adapt-tests-to-JSON-PP-2.92.patch new file mode 100644 index 0000000..9e21636 --- /dev/null +++ b/SQL-Translator-0.11021-Adapt-tests-to-JSON-PP-2.92.patch @@ -0,0 +1,178 @@ +From 5dd8305efe56f37f8718e0a9cd08e9ed5aa83cbf Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?= <ppi...@redhat.com> +Date: Fri, 26 May 2017 10:43:05 +0200 +Subject: [PATCH] Adapt tests to JSON-PP-2.92 +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +JSON-PP-2.91_01 fixed numeric value detection and that cused +t/23json.t test failure. There was a change in "order" value quoting. +Previously, the value was encoded as a string, now it's encoded as +number. + +CPAN RT#121901 + +Ported to 0.11021. + +Signed-off-by: Petr Písař <ppi...@redhat.com> +--- + t/23json.t | 37 +++++++++++++++++++++---------------- + 1 file changed, 21 insertions(+), 16 deletions(-) + +diff --git a/t/23json.t b/t/23json.t +index 0b063e2..9ef4e86 100644 +--- a/t/23json.t ++++ b/t/23json.t +@@ -15,8 +15,7 @@ BEGIN { + } + + my $sqlt_version = $SQL::Translator::VERSION; +-use JSON; +-my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); ++my $json_string = <<JSON; + { + "schema" : { + "procedures" : {}, +@@ -62,7 +61,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "is_primary_key" : 0, + "is_unique" : 0, + "name" : "age", +- "order" : "3", ++ "order" : 3, + "size" : [ + "0" + ] +@@ -74,7 +73,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "is_primary_key" : 0, + "is_unique" : 0, + "name" : "description", +- "order" : "6", ++ "order" : 6, + "size" : [ + "0" + ] +@@ -86,7 +85,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "is_primary_key" : 0, + "is_unique" : 0, + "name" : "iq", +- "order" : "5", ++ "order" : 5, + "size" : [ + "0" + ] +@@ -98,7 +97,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "is_primary_key" : 0, + "is_unique" : 1, + "name" : "name", +- "order" : "2", ++ "order" : 2, + "size" : [ + "20" + ] +@@ -111,7 +110,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "is_primary_key" : 1, + "is_unique" : 0, + "name" : "person_id", +- "order" : "1", ++ "order" : 1, + "size" : [ + "0" + ] +@@ -123,7 +122,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "is_primary_key" : 0, + "is_unique" : 0, + "name" : "weight", +- "order" : "4", ++ "order" : 4, + "size" : [ + "11", + "2" +@@ -133,7 +132,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "indices" : [], + "name" : "person", + "options" : [], +- "order" : "1" ++ "order" : 1 + }, + "pet" : { + "constraints" : [ +@@ -192,7 +191,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "is_primary_key" : 0, + "is_unique" : 0, + "name" : "age", +- "order" : "4", ++ "order" : 4, + "size" : [ + "0" + ] +@@ -204,7 +203,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "is_primary_key" : 0, + "is_unique" : 0, + "name" : "name", +- "order" : "3", ++ "order" : 3, + "size" : [ + "30" + ] +@@ -216,7 +215,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "is_primary_key" : 1, + "is_unique" : 0, + "name" : "person_id", +- "order" : "2", ++ "order" : 2, + "size" : [ + "0" + ] +@@ -228,7 +227,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "is_primary_key" : 1, + "is_unique" : 0, + "name" : "pet_id", +- "order" : "1", ++ "order" : 1, + "size" : [ + "0" + ] +@@ -237,7 +236,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "indices" : [], + "name" : "pet", + "options" : [], +- "order" : "2" ++ "order" : 2 + } + }, + "triggers" : { +@@ -255,7 +254,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "fields" : null, + "name" : "pet_trig", + "on_table" : "pet", +- "order" : "1", ++ "order" : 1, + "perform_action_when" : "after" + } + }, +@@ -263,7 +262,7 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + "person_pet" : { + "fields" : [], + "name" : "person_pet", +- "order" : "1", ++ "order" : 1, + "sql" : "select pr.person_id, pr.name as person_name, pt.name as pet_name\\n from person pr, pet pt\\n where person.person_id=pet.pet_id\\n" + } + } +@@ -285,6 +284,12 @@ my $json = to_json(from_json(<<JSON), { canonical => 1, pretty => 1 }); + } + } + JSON ++use JSON; ++if (JSON->is_pp && JSON->VERSION < 2.91_01) { ++ # JSON-PP-2.91_01 fixed numeric value detection, CPAN RT#121901 ++ $json_string =~ s/("order" : )(\d+)(,?)/$1"$2"$3/g; ++} ++my $json = to_json(from_json($json_string), { canonical => 1, pretty => 1 }); + + my $file = "$Bin/data/sqlite/create.sql"; + open my $fh, '<', $file or die "Can't read '$file': $!\n"; +-- +2.9.4 + diff --git a/perl-SQL-Translator.spec b/perl-SQL-Translator.spec index 08f4633..eda742d 100644 --- a/perl-SQL-Translator.spec +++ b/perl-SQL-Translator.spec @@ -1,10 +1,12 @@ Name: perl-SQL-Translator Summary: Manipulate structured data definitions (SQL and more) Version: 0.11021 -Release: 7%{?dist} +Release: 8%{?dist} License: GPL+ or Artistic Group: Development/Libraries Source0: http://search.cpan.org/CPAN/authors/id/I/IL/ILMARI/SQL-Translator-%{version}.tar.gz +# Adapt to changes in JSON-PP-0.92, bug #1455782, CPAN RT#121901 +Patch0: SQL-Translator-0.11021-Adapt-tests-to-JSON-PP-2.92.patch URL: http://search.cpan.org/dist/SQL-Translator/ BuildArch: noarch BuildRequires: perl @@ -136,6 +138,7 @@ ER diagram producer for SQL::Translator. %prep %setup -q -n SQL-Translator-%{version} +%patch0 -p1 # Remove bundled modules rm -r inc sed -i -e '/^inc\//d' MANIFEST @@ -169,6 +172,9 @@ make test %{_mandir}/man3/SQL::Translator::Producer::Diagram.* %changelog +* Fri May 26 2017 Petr Pisar <ppi...@redhat.com> - 0.11021-8 +- Adapt to changes in JSON-PP-0.92 (bug #1455782) + * Sat Feb 11 2017 Fedora Release Engineering <rel...@fedoraproject.org> - 0.11021-7 - Rebuilt for https://fedoraproject.org/wiki/Fedora_26_Mass_Rebuild https://src.fedoraproject.org/rpms/perl-SQL-Translator/c/bbc09cae5601a691fdb3121469ac8f2d1f570584?branch=f26 _______________________________________________ perl-devel mailing list -- perl-devel@lists.fedoraproject.org To unsubscribe send an email to perl-devel-le...@lists.fedoraproject.org