[Buildroot] [PATCH 1/1] dropbear: add extra build customization options
thomas.petazzoni at free-electrons.com
Tue Feb 3 19:30:19 UTC 2015
Dear Floris Bos,
On Tue, 03 Feb 2015 18:53:42 +0100, Floris Bos wrote:
> > We'd however be open to merge the option to install or not the clients,
> > but we do have some comments/questions below.
> >> +ifeq ($(BR2_PACKAGE_DROPBEAR_CLIENT),y)
> >> DROPBEAR_TARGET_BINS = dbclient dropbearkey dropbearconvert scp ssh
> >> DROPBEAR_MAKE = $(MAKE) MULTI=1 SCPPROGRESS=1 \
> >> PROGRAMS="dropbear dbclient dropbearkey dropbearconvert scp"
> >> -
> >> -DROPBEAR_LICENSE = MIT, BSD-2c-like, BSD-2c
> >> -DROPBEAR_LICENSE_FILES = LICENSE
> >> +else
> >> +DROPBEAR_TARGET_BINS = dropbearkey dropbearconvert scp
> >> +DROPBEAR_MAKE = $(MAKE) MULTI=1 SCPPROGRESS=1 \
> >> + PROGRAMS="dropbear dropbearkey dropbearconvert scp"
> >> +endif
> > Why is scp part of the server-only installation?
> Because scp is both a client and server program, similar to other
> programs that can tunnel data over SSH like rsync.
> When using scp on the client, it simply calls the ssh client program to
> connect to the SSH server and executes scp server-side there with a flag
> to tell it to play server and read further instructions/data from stdin,
> send data to stdout.
Ah, ok. Can you indicate that in the commit log, or maybe better, as a
comment in the code above the place where we defined what gets built
for the server-only case vs. server+client case?
> Do could make building scp a seperate Config.in option.
I don't think that's needed.
So, can you cook an updated patch?
Thanks a lot!
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
More information about the buildroot