> On Sat, 16 Apr 2022, Sam James wrote:
> +# @FUNCTION: edo
Just a remark: A similar command existed a long time ago under the
name "try". [1]
It was executed under "env" [2], should we also do that?
> +# @USAGE: command [arg1 [arg2 ...]]
should be in angle brackets, if we follow the usua
On 16/04/2022 10.38, Ulrich Mueller wrote:
On Sat, 16 Apr 2022, Sam James wrote:
+# @FUNCTION: edo
Just a remark: A similar command existed a long time ago under the
name "try". [1]
It was executed under "env" [2], should we also do that?
+# @USAGE: command [arg1 [arg2 ...]]
should be
On Sat, Apr 16, 2022 at 12:23:18PM +0200, Florian Schmaus wrote:
> > ebegin "Running $@"
> > "$@"
> > eend $? || die -n "$@ failed" > return $?
>
> I think this return statement can be omitted since it will always be
> invoked with 0 as argument, and this is the default behavi
On Sat, Apr 16, 2022 at 06:48:56AM -0400, Ionen Wolkens wrote:
> On Sat, Apr 16, 2022 at 12:23:18PM +0200, Florian Schmaus wrote:
> > > ebegin "Running $@"
> > > "$@"
> > > eend $? || die -n "$@ failed" > return $?
> >
> > I think this return statement can be omitted since it w
> On Sat, 16 Apr 2022, Florian Schmaus wrote:
>> edobe() {
> nit: I'd personally would use 'edob', as shorter is sometimes better
> plus every begin needs an end, so no need to explicitly state that
> there is an end.
It's even more obscure as a name however. :)
>> ebegin "Running $@"
>
Bug: https://bugs.gentoo.org/744880
Signed-off-by: Sam James
---
eclass/edo.eclass | 46 ++
1 file changed, 46 insertions(+)
create mode 100644 eclass/edo.eclass
diff --git a/eclass/edo.eclass b/eclass/edo.eclass
new file mode 100644
index 000
Changes since v1:
- Add EAPI 7 support (useful for e.g. base-system@ ebuilds)
- Add 'edob' (edo with ebegin/eend for better logs log-running commands, UX)
Sam James (1):
edo.eclass: add new eclass
eclass/edo.eclass | 46 ++
1 file changed, 46 inserti
On Sat, 2022-04-16 at 19:14 +0100, Sam James wrote:
> Bug: https://bugs.gentoo.org/744880
> Signed-off-by: Sam James
> ---
> eclass/edo.eclass | 46 ++
> 1 file changed, 46 insertions(+)
> create mode 100644 eclass/edo.eclass
>
> diff --git a/eclass/e
On Sat, Apr 16, 2022 at 2:28 AM Sam James wrote:
>
> eend should be preceded by an begin call.
Looks ok.
On Sat, Apr 16, 2022 at 08:21:34PM +0200, Michał Górny wrote:
> > +# @USAGE: [...]
> > +# @DESCRIPTION:
> > +# Executes 'command' with any given arguments and exits on failure unless
> > +# called under 'nonfatal'.
> > +edo() {
> > + elog "$@"
>
> einfo?
Way I see it, this could've even been j
On Sat, Apr 16, 2022 at 08:50:46PM -0400, Ionen Wolkens wrote:
> e.g. if using edo to replace stuff like
> # ./configure is not real autoconf!
> local cmd=(
> ./configure
> --args
> ${EXTRA_ECONF}
> )
> echo ${cmd[*]}"
> "${cmd[@]}" || die "cmd failed: ${cm
11 matches
Mail list logo