Bug 17017 - Dependency fixes for 16.05
Summary: Dependency fixes for 16.05
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Packaging (show other bugs)
Version: 16.05
Hardware: All All
: P5 - low normal
Assignee: Mirko Tietgen
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-02 12:22 UTC by Mirko Tietgen
Modified: 2016-08-13 11:51 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Bug 17017 Dependency fixes for 16.05 (3.89 KB, patch)
2016-08-02 13:25 UTC, Mirko Tietgen 		Details | Diff | Splinter Review
Bug 17017 Dependency fixes for 16.05 (4.03 KB, patch)
2016-08-02 15:19 UTC, Frédéric Demians 		Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.