Bug 1278707 - perl-SQL-SplitStatement: Failed test t/95-script.t
Summary: perl-SQL-SplitStatement: Failed test t/95-script.t
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: perl-SQL-SplitStatement
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Emmanuel Seyman
QA Contact: Fedora Extras Quality Assurance
URL: https://apps.fedoraproject.org/kosche...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-11-06 09:00 UTC by Jitka Plesnikova
Modified: 2015-11-22 02:24 UTC (History)
3 users (show)

Fixed In Version: perl-SQL-SplitStatement-1.00020-4.fc22 perl-SQL-SplitStatement-1.00020-4.fc23
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-11-20 15:24:37 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
CPAN 108595 0 None None None Never

Description Jitka Plesnikova 2015-11-06 09:00:23 UTC
Description of problem:
Package perl-SQL-SplitStatement fails to build from source in rawhide.

Version-Release number of selected component (if applicable):
1.00020-3.fc24

Steps to Reproduce:
koji build --scratch f24 perl-SQL-SplitStatement-1.00020-3.fc24.src.rpm

Additional info:
This package is tracked by Koschei. See:
https://apps.fedoraproject.org/koschei/package/perl-SQL-SplitStatement

Test failed with new perl-Getopt-Long 3.48

The similar command is called in the test 
$ perl -Ilib bin/sql-split -s --# -f --### -E t/data/create_table.sql 

Current result:
Unknown option: E
Usage:
        sql-split [ OPTIONS ] [ FILE(S) ]
        sql-split --man

Expected result:
perl -Ilib bin/sql-split -s --# -f --### -E t/data/create_table.sql
CREATE TABLE foo (
    foo_field_1 VARCHAR,
    foo_field_2 VARCHAR
)--#CREATE TABLE bar (
    bar_field_1 VARCHAR,
    bar_field_2 VARCHAR
)--#

Comment 1 Jitka Plesnikova 2015-11-06 09:13:16 UTC
The temporary fix for the test is replacing using "-E" by "--empty"

Comment 2 Petr Pisar 2015-11-06 09:31:34 UTC
Getopt-Long 3.48 fixed gnu_getopt compatibility mode. In that mode, it consider "--fo bar" as an option and a positional argument because gnut_getopt mode would requires "--foo=bar" syntax to make the "bar" a value of "foo" option.

If sql-split uses that mode, you have to call it as -E=t/data/create_table.sql. If not, it's probably a regressions in the Getopt::Long.

Comment 3 Jitka Plesnikova 2015-11-06 14:53:40 UTC
If you add 'no_ignore_case' to Getopt::Long::Configure in bin/sql-split, it will fix the issue.

Due to the last changes in Getopt::Long 2.48, the bundling is disabled when gnu_compat is enabled. And it causes that single character options are not treated case-sensitive.

See:
https://metacpan.org/source/JV/Getopt-Long-2.48//lib/Getopt/Long.pm#L897

Comment 4 Emmanuel Seyman 2015-11-08 21:30:21 UTC
Fixed in perl-SQL-Splitstatement 1.00020-4. I'll release this as an update once perl-Getopt-Long 2.48 hits stable in the other branches.

Comment 5 Jitka Plesnikova 2015-11-09 08:14:40 UTC
Please add the fix also to rawhide. Currently, it is only in F23 branch.

Comment 6 Emmanuel Seyman 2015-11-09 08:48:26 UTC
(In reply to Jitka Plesnikova from comment #5)
>
> Please add the fix also to rawhide. Currently, it is only in F23 branch.

My bad. I've cherry-picked the commit to master and the build is underway.

Comment 7 Petr Pisar 2015-11-09 08:55:38 UTC
You can put to perl-Getopt-Long into built root overrides, build the perl-SQL-Splitstatment and then I can append the new perl-SQL-Splitstatment builds to perl-Getopt-Long errata so they will be released together.

Comment 8 Fedora Update System 2015-11-10 08:33:12 UTC
perl-SQL-SplitStatement-1.00020-4.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-743dc4e9b5

Comment 9 Emmanuel Seyman 2015-11-10 08:42:27 UTC
(In reply to Petr Pisar from comment #7)
> You can put to perl-Getopt-Long into built root overrides

After a good night's sleep, I realized this was unnecessary. Adding 'no_ignore_case' doesn't require the new Getopt-Long package.

I've built the updates and released them.

Comment 10 Fedora Update System 2015-11-11 00:50:46 UTC
perl-SQL-SplitStatement-1.00020-4.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update perl-SQL-SplitStatement'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-743dc4e9b5

Comment 11 Fedora Update System 2015-11-11 02:23:15 UTC
perl-SQL-SplitStatement-1.00020-4.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'dnf --enablerepo=updates-testing update perl-SQL-SplitStatement'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-75ef0d88a0

Comment 12 Fedora Update System 2015-11-20 15:24:35 UTC
perl-SQL-SplitStatement-1.00020-4.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.

Comment 13 Fedora Update System 2015-11-22 02:24:20 UTC
perl-SQL-SplitStatement-1.00020-4.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.