[Buildroot] [PATCH v2] mksh: add new package MirOS Korn Shell

Waldemar Brodkorb wbx at openadk.org
Fri Sep 23 06:37:55 UTC 2016


Hi Thorsten,
Thorsten Glaser wrote,

> Waldemar Brodkorb dixit:
> 
> >The MirOS Korn Shell is a quite complete posix shell implementation,
> 
> POSIX compatible, not 100% but closer than most others…
> 
> >+MKSH_LICENSE = MirOS, BSD-3c, ISC
> 
> Only MirOS and ISC unless you add printf.c (which I don’t see
> in the patch).
> 
> >+MKSH_LICENSE_FILES = COPYING
> 
> Huh?

Okay, seems to be a problem.
Is the License file for mksh (MirOS License) available in 
a file in the source tree?
Buildroot has a function to collect all license files of
a created root filesystem.

> >+define MKSH_CONFIGURE_CMDS
> >+	cd $(@D) && $(TARGET_MAKE_ENV) \
> >+		CPPFLAGS="-DMKSH_S_NOVI=0 $(TARGET_CPPFLAGS)" \
> 
> You only need -DMKSH_S_NOVI=0 if you also add -DMKSH_SMALL
> and want the vi editing mode.
> 
> >+		sh ./Build.sh -M
> >+endef
> >+
> >+define MKSH_BUILD_CMDS
> >+	cd $(@D) && $(TARGET_MAKE_ENV) \
> >+		sh ./Rebuild.sh
> >+endef
> 
> Arguable, but just call Build.sh without -M to configure *and*
> build it in *one* step.
> 
> >+define MKSH_INSTALL_TARGET_CMDS
> >+	$(INSTALL) -m 0755 $(@D)/mksh $(TARGET_DIR)/bin/mksh
> >+endef
> 
> Please also ship dot.mkshrc or some other sort of ~/.mkshrc
> so that users get a slightly more comfortable/usable environment,
> if you can afford it, byte-wise (it’s 15K), or at least make it
> configurable. (Also patch it, if needed, e.g. for an ls(1) without
> the -o option.)
> 
> Thanks,
> //mirabilos (mksh upstream)

thx,
 Waldemar


More information about the buildroot mailing list