Bug#348314: busybox-udeb: please enable getopt

2011-03-10 Thread Michael Tokarev
16.01.2006 12:39, Colin Watson wrote:
 Package: busybox-udeb
 Version: 1:1.01-4
 Severity: wishlist
 
 Please enable CONFIG_GETOPT in config-udeb and config-udeb-linux. I need
 this in order to merge Kickstart support; it's not feasible to do this
 properly in shell without the help of getopt (I tried), and a C
 implementation would be significantly bigger.

Isn't it sufficient to use getopts ash built-in?  It does
approx. the same thing (with different usage), and it's
already enabled in busybox.

Thanks!

/mjt



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



Bug#348314: busybox-udeb: please enable getopt

2011-03-10 Thread Colin Watson
On Thu, Mar 10, 2011 at 02:37:20PM +0300, Michael Tokarev wrote:
 16.01.2006 12:39, Colin Watson wrote:
  Please enable CONFIG_GETOPT in config-udeb and config-udeb-linux. I need
  this in order to merge Kickstart support; it's not feasible to do this
  properly in shell without the help of getopt (I tried), and a C
  implementation would be significantly bigger.
 
 Isn't it sufficient to use getopts ash built-in?  It does
 approx. the same thing (with different usage), and it's
 already enabled in busybox.

No, getopts can't deal with long options, which Kickstart requires.

-- 
Colin Watson   [cjwat...@debian.org]



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



Bug#348314: busybox-udeb: please enable getopt

2011-03-10 Thread Otavio Salvador
On Thu, Mar 10, 2011 at 12:12, Colin Watson cjwat...@debian.org wrote:
 On Thu, Mar 10, 2011 at 02:37:20PM +0300, Michael Tokarev wrote:
 16.01.2006 12:39, Colin Watson wrote:
  Please enable CONFIG_GETOPT in config-udeb and config-udeb-linux. I need
  this in order to merge Kickstart support; it's not feasible to do this
  properly in shell without the help of getopt (I tried), and a C
  implementation would be significantly bigger.

 Isn't it sufficient to use getopts ash built-in?  It does
 approx. the same thing (with different usage), and it's
 already enabled in busybox.

 No, getopts can't deal with long options, which Kickstart requires.

What is the binary size change enabling it?

-- 
Otavio Salvador                             O.S. Systems
E-mail: ota...@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br



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



Bug#348314: busybox-udeb: please enable getopt

2011-03-10 Thread Michael Tokarev
10.03.2011 17:09, Otavio Salvador wrote:
 On Thu, Mar 10, 2011 at 12:12, Colin Watson cjwat...@debian.org wrote:
 On Thu, Mar 10, 2011 at 02:37:20PM +0300, Michael Tokarev wrote:

 Isn't it sufficient to use getopts ash built-in?

 No, getopts can't deal with long options, which Kickstart requires.

 What is the binary size change enabling it?

The size is relatively small:

   textdata bss dec hex filename
 22273014988684  232912   38dd0 current
 22435415028684  234540   3942c +long_getopt +1624 bytes for long_getopt
 22465615068684  234846   3955e +long_getopt+mktemp  +302 bytes for 
mktemp

(+long_getopt - CONFIG_GETOPT + CONFIG_GETOPT_LONG_OPTIONS)

Note that we already have chown with long options and
cp with long options in busybox udeb.

   textdata bss dec hex filename
 22248014948680  232654   38cce current -longopts +getopt
 22146214948680  231636   388d4 current -longopts -1268 bytes for 
getopt_long

(-longopts - i disabled long options for cp and chown).

/mjt



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



Bug#348314: busybox-udeb: please enable getopt

2006-01-16 Thread Colin Watson
Package: busybox-udeb
Version: 1:1.01-4
Severity: wishlist

Please enable CONFIG_GETOPT in config-udeb and config-udeb-linux. I need
this in order to merge Kickstart support; it's not feasible to do this
properly in shell without the help of getopt (I tried), and a C
implementation would be significantly bigger.

This adds 1176 bytes to the busybox binary on powerpc.

Thanks,

-- 
Colin Watson   [EMAIL PROTECTED]


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]