Bug#758319: bird: Please ship birdcl in package

2014-08-18 Thread Ondřej Surý
On Mon, Aug 18, 2014, at 11:31, Ondrej Zajicek wrote:
> On Mon, Aug 18, 2014 at 10:58:13AM +0200, Ondřej Surý wrote:
> > Hi Micah,
> > 
> > why would we need to ship birdcl together with birdc in one package?
> > That makes no sense since birdcl is just no-readline version of birdc.
> 
> I think it makes sense. Generally birdc is more suited to interactive
> usage, while birdcl is more suited to non-interactive usage (although
> both could be used in the other way).
> 
> If you use birdc non-interactively using pipes, it cannot be ruled out
> that readline would somehow interfere undesirably with it in some
> unexpected way.

We have many other examples when the CLI is compiled with readline
and it doesn't interfere (there was a problem in libedit in the past
when
using CLI in non-interactive way, but it's gone...).

> Another possible reason is that some third-party scripts (like looking
> glass or scripting language bindings) use birdcl by default, so they
> would have to be modified to use birdc in case that birdcl is not
> available.

This could be solved by a symlink reducing the binary with duplicate
functionality.

Ondrej
-- 
Ondřej Surý 
Knot DNS (https://www.knot-dns.cz/) – a high-performance DNS server


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#758319: bird: Please ship birdcl in package

2014-08-18 Thread Ondrej Zajicek
On Mon, Aug 18, 2014 at 10:58:13AM +0200, Ondřej Surý wrote:
> Hi Micah,
> 
> why would we need to ship birdcl together with birdc in one package?
> That makes no sense since birdcl is just no-readline version of birdc.

I think it makes sense. Generally birdc is more suited to interactive
usage, while birdcl is more suited to non-interactive usage (although
both could be used in the other way).

If you use birdc non-interactively using pipes, it cannot be ruled out
that readline would somehow interfere undesirably with it in some
unexpected way.

Another possible reason is that some third-party scripts (like looking
glass or scripting language bindings) use birdcl by default, so they
would have to be modified to use birdc in case that birdcl is not
available.

-- 
Elen sila lumenn' omentielvo

Ondrej 'Santiago' Zajicek (email: santi...@crfreenet.org)
OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, wwwkeys.pgp.net)
"To err is human -- to blame it on a computer is even more so."


signature.asc
Description: Digital signature


Bug#758319: bird: Please ship birdcl in package

2014-08-18 Thread Ondřej Surý
Hi Micah,

why would we need to ship birdcl together with birdc in one package?
That makes no sense since birdcl is just no-readline version of birdc.

Cheers,
Ondřej

On Sat, Aug 16, 2014, at 21:50, Micah Anderson wrote:
> Package: bird
> Version: 1.4.4-1
> Severity: wishlist
> Tags: patch
> 
> Hello,
> 
> It seems that you are not shipping the birdcl binary in the package. The
> attached patch would add that to the package.
> 
> I'm happy to upload this as a NMU to help you.
> 
> micah
> 
> -- System Information:
> Debian Release: jessie/sid
>   APT prefers unstable
>   APT policy: (500, 'unstable')
> Architecture: amd64 (x86_64)
> Foreign Architectures: i386
> 
> Kernel: Linux 3.14-2-amd64 (SMP w/4 CPU cores)
> Locale: LANG=en_US.utf8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
> Shell: /bin/sh linked to /bin/dash
> 
> Versions of packages bird depends on:
> ii  adduser   3.113+nmu3
> ii  libc6 2.19-7
> ii  libreadline6  6.3-8
> ii  libtinfo5 5.9+20140712-2
> 
> bird recommends no packages.
> 
> Versions of packages bird suggests:
> ii  bird-doc  1.4.4-1
> Email had 1 attachment:
> + bird2.diff
>   1k (text/x-diff)


-- 
Ondřej Surý 
Knot DNS (https://www.knot-dns.cz/) – a high-performance DNS server


--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#758319: bird: Please ship birdcl in package

2014-08-16 Thread Micah Anderson
Package: bird
Version: 1.4.4-1
Severity: wishlist
Tags: patch

Hello,

It seems that you are not shipping the birdcl binary in the package. The
attached patch would add that to the package.

I'm happy to upload this as a NMU to help you.

micah

-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.14-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages bird depends on:
ii  adduser   3.113+nmu3
ii  libc6 2.19-7
ii  libreadline6  6.3-8
ii  libtinfo5 5.9+20140712-2

bird recommends no packages.

Versions of packages bird suggests:
ii  bird-doc  1.4.4-1
diff --git a/debian/bird.install b/debian/bird.install
index 2088dfc..c5ed65d 100644
--- a/debian/bird.install
+++ b/debian/bird.install
@@ -1,6 +1,7 @@
 etc/bird/bird.conf
 usr/sbin/bird
 usr/sbin/birdc
+usr/sbin/birdcl
 etc/bird/bird6.conf
 usr/sbin/bird6
 usr/sbin/birdc6
diff --git a/debian/changelog b/debian/changelog
index f8b69d0..42e8750 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+bird (1.4.4-1.1) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * Install birdcl
+
+ -- Micah Anderson   Sat, 16 Aug 2014 15:45:29 -0400
+
 bird (1.4.4-1) unstable; urgency=medium
 
   * New upstream version 1.4.4