Bug 26702

Summary: Remove explicit libnet-stomp-perl from debian/control.in
Product: Koha Reporter: David Cook <dcook>
Component: PackagingAssignee: David Cook <dcook>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: jonathan.druart, lucas, martin.renvoize, mtj
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.11.00
Bug Depends on: 22417    
Bug Blocks:    
Attachments: Bug 26702: Remove explicit libnet-stomp-perl from debian/control.in
Bug 26702: Remove explicit libnet-stomp-perl from debian/control.in
Bug 26702: Remove explicit libnet-stomp-perl from debian/control.in

Description David Cook 2020-10-16 00:57:53 UTC
libnet-stomp-perl is added to debian/control via __AUTODEPENDS__, so we don't need to add it manually into debian/control.in
Comment 1 David Cook 2020-10-16 01:00:45 UTC
Created attachment 111824 [details] [review]
Bug 26702: Remove explicit libnet-stomp-perl from debian/control.in

libnet-stomp-perl is added to debian/control via __AUTODEPENDS__,
so we don't need to add it manually into debian/control.in
Comment 2 Mason James 2020-10-16 09:51:48 UTC
Created attachment 111836 [details] [review]
Bug 26702: Remove explicit libnet-stomp-perl from debian/control.in

libnet-stomp-perl is added to debian/control via __AUTODEPENDS__,
so we don't need to add it manually into debian/control.in

Signed-off-by: Mason James <mtj@kohaaloha.com>
Comment 3 Martin Renvoize 2020-10-16 13:49:37 UTC
Created attachment 111876 [details] [review]
Bug 26702: Remove explicit libnet-stomp-perl from debian/control.in

libnet-stomp-perl is added to debian/control via __AUTODEPENDS__,
so we don't need to add it manually into debian/control.in

Signed-off-by: Mason James <mtj@kohaaloha.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize 2020-10-16 13:50:16 UTC
This all makes perfect sense.. Passing QA
Comment 5 Jonathan Druart 2020-10-25 22:55:33 UTC
(In reply to David Cook from comment #0)
> libnet-stomp-perl is added to debian/control via __AUTODEPENDS__, so we
> don't need to add it manually into debian/control.in

Mason, are you sure this is correct? Isn't control auto-generated from control.in?
Comment 6 David Cook 2020-10-26 00:09:44 UTC
(In reply to Jonathan Druart from comment #5)
> (In reply to David Cook from comment #0)
> > libnet-stomp-perl is added to debian/control via __AUTODEPENDS__, so we
> > don't need to add it manually into debian/control.in
> 
> Mason, are you sure this is correct? Isn't control auto-generated from
> control.in?

control is auto-generated from control.in, but control.in is also updated by replacing __AUTODEPENDS__ with the output of debian/list-deps, which sources the dependencies from the "cpanfile", which contains Net::Stomp.
Comment 7 Jonathan Druart 2020-11-02 09:59:17 UTC
Mason, not directly related to this patch but, I wanted to push the changes to debian/control along with this patch and I have:

- libreadonly-xs-perl,
and
+ libtest-harness-perl,

Should we deal with them on a separate bug report? Is that new or something you are aware of already?
Comment 8 Jonathan Druart 2020-11-02 10:05:40 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 9 Lucas Gass 2020-11-16 18:42:12 UTC
missing dependencies for 20.05.x, no backport