Bug 17017

Summary: Dependency fixes for 16.05
Product: Koha Reporter: Mirko Tietgen <mirko>
Component: PackagingAssignee: Mirko Tietgen <mirko>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: frederic, mirko
Version: 16.05   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16588
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 17017 Dependency fixes for 16.05
Bug 17017 Dependency fixes for 16.05

Description Mirko Tietgen 2016-08-02 12:22:20 UTC
Things I did for 16.05.02. Should go in 16.05.03.
Comment 1 Mirko Tietgen 2016-08-02 13:25:58 UTC Comment hidden (obsolete)
Comment 2 Frédéric Demians 2016-08-02 15:13:29 UTC
(In reply to Mirko Tietgen from comment #0)
> Things I did for 16.05.02. Should go in 16.05.03.

Isn't it also required for master?
Comment 3 Frédéric Demians 2016-08-02 15:19:56 UTC
Created attachment 53919 [details] [review]
Bug 17017 Dependency fixes for 16.05

Things I did for 16.05.02. Should go in 16.05.03.

Adds to PerlDependencies.pm:

IO::Socket::IP
Readonly
Readonly::XS

Adds overrides to debian/list-deps:

IO::Socket::IP => 'perl-modules (>= 5.20.0) | libio-socket-ip-perl'
Swagger2 => 'libswagger2-perl (>= 0.59)'

and an up to date debian/control based on that.

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Comment 4 Frédéric Demians 2016-08-02 15:21:57 UTC
Since this patch is directly related to the stable release process, including Debian packaging, I'm really tempted to push it directly into 16.05.x branch without QAing...
Comment 5 Mirko Tietgen 2016-08-02 15:32:34 UTC
(In reply to Frédéric Demians from comment #2)
> Isn't it also required for master?

Readonly will be removed from master, see bug 16588.
Comment 6 Mirko Tietgen 2016-08-02 15:42:20 UTC
(In reply to Frédéric Demians from comment #4)

> Since this patch is directly related to the stable release process,
> including Debian packaging, I'm really tempted to push it directly into
> 16.05.x branch without QAing...


Katrin says it's ok

> (17:39:15) drojf: cait: bug 17017
> (17:39:34) drojf: i used it in the packages without even a signoff. any objections to pushing it?
> (17:40:14) cait: nope - trusting you there
Comment 7 Frédéric Demians 2016-08-02 15:54:19 UTC
Done. Thanks.