X-Original-To: alpine-devel@lists.alpinelinux.org Delivered-To: alpine-devel@mail.alpinelinux.org Received: from ncopa-laptop.res.nor.wtbts.net (3.203.202.84.customer.cdi.no [84.202.203.3]) (using SSLv3 with cipher ECDHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) (Authenticated sender: nc@alpinelinux.org) by mail.alpinelinux.org (Postfix) with ESMTPSA id B8274DC0096; Wed, 3 Jul 2013 18:20:13 +0000 (UTC) Date: Wed, 3 Jul 2013 20:20:09 +0200 From: Natanael Copa To: Dubiousjim Cc: alpine-devel@lists.alpinelinux.org Subject: Re: [alpine-devel] [PATCH 3/4] update-conf: accept long options Message-ID: <20130703202009.29625ede@ncopa-laptop.res.nor.wtbts.net> In-Reply-To: <20130703140910.GE1550@zen> References: <6d0295ddc265976445cb2d46c6194db304e92386.1372713126.git.dubiousjim@gmail.com> <20130703153033.5230023e@ncopa-desktop.alpinelinux.org> <20130703140910.GE1550@zen> X-Mailer: Claws Mail 3.9.2 (GTK+ 2.24.17; i686-pc-linux-gnu) X-Mailinglist: alpine-devel Precedence: list List-Id: Alpine Development List-Unsubscribe: List-Post: List-Help: List-Subscribe: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Wed, 3 Jul 2013 10:09:10 -0400 Dubiousjim wrote: ... > I've got a bunch of patches for the abuild package too that I'm cleaning up, > including long options for many of the small scripts. > > I'm pretty sure you're right that POSIX doesn't require any getopt shell > command or utility. So this is extra-POSIX. BusyBox (as we configure it) > provides getopt as an external utility, so in our case we do go beyond > Posix but we're not thereby requiring an extra-POSIX *shell*. hm 'getops' is in posix. > Perhaps Bash implements getopt as an internal shell command, I don't > remember. I think we can use getopt with long opts. it will save us for some work. > > I also made a posix shell implementation for parsing options for lxc > > templates which we could use instead. > > Where is that? https://github.com/lxc/lxc/commit/e60a8164c12d565f70071ff6b32b823dd495df9e Basically all the src/lxc/*.in scripts in there. -nc PS. I'm really happy for your review of the code, questioning the old code and patches for improvements. Thanks! --- Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org Help: alpine-devel+help@lists.alpinelinux.org ---