Bug#818008: How to proceed

2019-09-01 Thread Lee Garrett
Closing due to inactivity. Feel free to re-open if you would like to
challenge the assessment below.

On Tue, 16 Oct 2018 15:46:25 +0200 Lee Garrett  wrote:
> Hi Geert, hi Raphael,
> 
> personally, I think that the Debian package should stop shipping
> /etc/ansible/{ansible.cfg, hosts} alltogether. The reason is that those files
> have a systemwide affect, and can only be edited by root. The former is
> problematic if you have several projects, in that case you'd just ship a
> ansible.cfg and hosts file within each project. Both those files can be edited
> by the local user. For that reason I think adding debconf questions for that
> is the wrong approach. But if you really want systemwide hosts and ansible
> config, you can still template those with ansible. :)
> 
> What are you thoughts on this?
> 
> Greets,
> Lee
> 
> 
> 
> On Sun, 14 Oct 2018 14:09:23 +0200 Geert Stappers  wrote:
> > 
> > Found today this bugreport ( 
> > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=818008 )
> > even with a patch.
> > 
> > To find out if it stills applies there is now 
> > https://salsa.debian.org/stappers/ansible
> > which has branch 'br818008patch'.
> > 
> > Difference with 
> > https://github.com/rmedaer/ansible-debian-pkg/commit/7fa153bdcc368fad2c3dc8701fe4a8aa73c80d08
> > is that updating debian/changelong is skipped. If all goes well,
> > is @rmedaer the author of the commit.
> > 
> > 
> > However, the question is what to do next?
> > 
> > Would it make sense to have the patch also here in the Debian BTS?
> > 
> > Should Ansible be configurable by debconf?
> > 
> > 
> > Groeten
> > Geert Stappers
> > -- 
> > Leven en laten leven
> > 
> > 
> 
> 



Bug#818008: How to proceed

2018-10-16 Thread Lee Garrett
Hi Geert, hi Raphael,

personally, I think that the Debian package should stop shipping
/etc/ansible/{ansible.cfg, hosts} alltogether. The reason is that those files
have a systemwide affect, and can only be edited by root. The former is
problematic if you have several projects, in that case you'd just ship a
ansible.cfg and hosts file within each project. Both those files can be edited
by the local user. For that reason I think adding debconf questions for that
is the wrong approach. But if you really want systemwide hosts and ansible
config, you can still template those with ansible. :)

What are you thoughts on this?

Greets,
Lee



On Sun, 14 Oct 2018 14:09:23 +0200 Geert Stappers  wrote:
> 
> Found today this bugreport ( 
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=818008 )
> even with a patch.
> 
> To find out if it stills applies there is now 
> https://salsa.debian.org/stappers/ansible
> which has branch 'br818008patch'.
> 
> Difference with 
> https://github.com/rmedaer/ansible-debian-pkg/commit/7fa153bdcc368fad2c3dc8701fe4a8aa73c80d08
> is that updating debian/changelong is skipped. If all goes well,
> is @rmedaer the author of the commit.
> 
> 
> However, the question is what to do next?
> 
> Would it make sense to have the patch also here in the Debian BTS?
> 
> Should Ansible be configurable by debconf?
> 
> 
> Groeten
> Geert Stappers
> -- 
> Leven en laten leven
> 
> 



Bug#818008: How to proceed

2018-10-14 Thread Geert Stappers


Found today this bugreport ( 
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=818008 )
even with a patch.

To find out if it stills applies there is now 
https://salsa.debian.org/stappers/ansible
which has branch 'br818008patch'.

Difference with 
https://github.com/rmedaer/ansible-debian-pkg/commit/7fa153bdcc368fad2c3dc8701fe4a8aa73c80d08
is that updating debian/changelong is skipped. If all goes well,
is @rmedaer the author of the commit.


However, the question is what to do next?

Would it make sense to have the patch also here in the Debian BTS?

Should Ansible be configurable by debconf?


Groeten
Geert Stappers
-- 
Leven en laten leven