[Buildroot] [BALENA PATCH] balena : New Package
Refik Tuzaklı
refik.tuzakli at savronik.com.tr
Tue Jun 5 08:20:57 UTC 2018
Hello Thomas,
Thanks for feedback.
On 04-06-2018 17:55, Thomas Petazzoni wrote:
> Hello Refik,
>
> Thanks a lot for your contribution! I believe it's your first Buildroot
> contribution, so congratulations :-)
>
> The first comment is that you should sent your patch using "git
> send-email". Because you didn't do that, your patch has been horribly
> rewrapped by your e-mail client, and it cannot be applied.
I tried to send this patch via git send-email but I have problem with my
ssl certificate as below;
STARTTLS failed! SSL connect attempt failed with unknown error
error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate
verify failed at /usr/lib/git-core/git-send-email line 1241.
When I solve problem, I will use this tool instead of sending via direct
e-mail.
>
> The second thing, less important, is that the [BALENA PATCH] subject
> prefix is not necessary, just [PATCH] is sufficient or more precisely
> [PATCHv2] for your next iteration.
>
> The title should be:
>
> balena: new package
>
> which is almost the same as what you did, but without the space
> before :, and lower case.
OK
>
> On Mon, 4 Jun 2018 17:22:37 +0300, Refik Tuzaklı wrote:
>> package/Config.in | 1 +
>> package/balena/Config.in | 16 +++++++++++
>> package/balena/balena.hash | 2 ++
>> package/balena/balena.mk | 68
>> ++++++++++++++++++++++++++++++++++++++++++++++
> Please add an entry in the DEVELOPERS file for this package.
OK
>
>> + source "package/balena/Config.in"
>> source "package/bootutils/Config.in"
> Indentation is wrong here, but maybe it's an artefact of not sending
> with git send-email.
At my local patch file, I can not see this typo but I will take care of it.
>
>> diff --git a/package/balena/Config.in b/package/balena/Config.in
>> new file mode 100644
>> index 0000000..2904d2a
>> --- /dev/null
>> +++ b/package/balena/Config.in
>> @@ -0,0 +1,16 @@
>> +config BR2_PACKAGE_BALENA
>> + bool "balena"
>> + depends on BR2_PACKAGE_HOST_GO_ARCH_SUPPORTS
>> + depends on BR2_PACKAGE_HOST_GO_CGO_LINKING_SUPPORTS
>> + depends on BR2_TOOLCHAIN_HAS_THREADS
>> + depends on BR2_INIT_SYSTEMD
>> + select BR2_PACKAGE_LVM2
>> + select BR2_PACKAGE_LVM2_STANDARD_INSTALL
> You need to propagate the dependencies of lvm2 here:
>
> depends on BR2_TOOLCHAIN_HAS_THREADS
> depends on BR2_USE_MMU # needs fork()
> depends on !BR2_STATIC_LIBS # It fails to build statically
OK
>
>> + help
>> + Moby-based Container Engine for Embedded, IoT, and Edge uses
>> +
>> + https://github.com/resin-os/balena
>> +
>> +
> Only one empty line here.
OK
>
>> +comment "balena needs systemd"
>> + depends on !BR2_INIT_SYSTEMD
> Also, please run check-package to make sure the indentation is correct.
OK
>
>> +BALENA_LICENSE = Apache-2.0
>> +BALENA_LICENSE_FILES = LICENSE
>> +
>> +BALENA_DEPENDENCIES = host-go host-pkgconf systemd lvm2
> Drop the host-go dependency, it's not needed because you're using the
> golang-package infrastructure.
OK
>
>> +BALENA_GOPATH = "$(@D)/.gopath"
> I believe this is not needed, it's taken care of by the golang-package
> infrastructure.
OK
>
>> +BALENA_VERSION_DIR = 17.06.0-dev
> Ditto, should not be needed.
OK
>
>> +BALENA_BUILD_TARGETS = balena
> This one is OK.
>
>> +BALENA_INSTALL_BINS = $(TARGET_DIR)/usr/bin/$(target)
> This one doesn't make sense because $(target) is empty. Instead list
> the binaries to be installed.
OK
>
>> +BALENA_CONFIGURE_ENV = GOPATH=${BALENA_GOPATH} \
>> + $(TARGET_MAKE_ENV) \
>> + $(HOST_GO_TARGET_ENV) \
>> + DOCKER_GITCOMMIT=${BALENA_COMMIT} \
>> + DOCKER_BUILDTAGS='exclude_graphdriver_btrfs exclude_graphdirver_zfs
>> exclude_graphdriver_devicemapper'
> None of that should be needed with the golang-package infrastructure.
> Look at the docker-engine package for an example.
I thought so but when I remove one of them, it crashes while
configuring. Even it is balena package, it needs some docker parameters.
>
>> +
>> +BALENA_MAKE_ENV = GOPATH=$(BALENA_GOPATH) \
>> + $(TARGET_MAKE_ENV) \
>> + $(HOST_GO_TARGET_ENV)
> Same.
>
>> +
>> +BALENA_GLDFLAGS += -extldflags '-static'
> Same.
>
>> +
>> +define BALENA_CONFIGURE_CMDS
>> + mkdir -p $(BALENA_GOPATH)/src/github.com/docker
>> + ln -fs $(@D) $(BALENA_GOPATH)/src/github.com/docker/docker
> Not needed with the golang-package infrastructure.
golang-package does ;
mkdir -p $(@D)/_gopath/src/github.com/resin-os/
ln -sf $(@D) $(@D)/_gopath/src/github.com/resin-os/balena
but balena needs ;
mkdir -p $(@D)/.gopath/src/github.com/docker
ln -fs $(@D) $(@D)/.gopath/src/github.com/docker/docker
I think this caused by package itself. I am not a developer of balena, I
can ask this issue to balena developers
>
>> + cd $(@D) && \
>> + $(BALENA_CONFIGURE_ENV) \
>> + bash ./hack/make.sh dynbinary-balena
> Do this in a POST_CONFIGURE_HOOKS.
OK
>
>> +define BALENA_BUILD_CMDS
>> + $(foreach target,$(BALENA_BUILD_TARGETS), \
>> + cd $(BALENA_GOPATH)/src/github.com/docker/docker/cmd/mobynit; \
>> + $(BALENA_MAKE_ENV) \
>> + $(HOST_DIR)/bin/go build -v\
>> + -ldflags "$(BALENA_GLDFLAGS)"
>> + )
>> +endef
> Not needed, it should be done by the golang-package infrastructure.
build directory is
$(BALENA_GOPATH)/src/github.com/docker/docker/cmd/mobynit
I guess buildroot looks $(BALENA_GOPATH)/src/github.com/resin-io/balena/cmd/balena
Am I right?
>
>> +
>> +define BALENA_INSTALL_TARGET_CMDS
>> + $(foreach target,$(BALENA_BUILD_TARGETS), \
>> + $(INSTALL) -D -m 0755
>> $(@D)/bundles/17.06.0-dev/dynbinary-balena/$(target) $(BALENA_INSTALL_BINS)
>> + ln -sf balena $(TARGET_DIR)/usr/bin/balenad
>> + ln -sf balena $(TARGET_DIR)/usr/bin/balena-containerd
>> + ln -sf balena $(TARGET_DIR)/usr/bin/balena-containerd-shim
>> + ln -sf balena $(TARGET_DIR)/usr/bin/balena-containerd-ctr
>> + ln -sf balena $(TARGET_DIR)/usr/bin/balena-runc
>> + ln -sf balena $(TARGET_DIR)/usr/bin/balena-proxy
>> + )
> Same.
build step creates
$(BALENA_GOPATH)/src/github.com/docker/docker/cmd/mobynit/mobynit
but we need
$(@D)/bundles/17.06.0-dev/dynbinary-balena/balena
>
> Basically, I have the feeling that you created this package as a
> generic-package originally, and then quickly converted it to
> golang-package. But the golang-package infrastructure greatly
> simplifies Go packages, so you should really leverage the features of
> the golang-package infrastructure.
>
> Could you rework this package to use golang-package more effectively ?
>
> Thanks a lot!
>
> Thomas
I check docker-engine before converting to golang package. Also I took
balena_git.bb file on meta-resin as a reference for cross-compile
procedure from Yocto Projects. If I can I want to rework on this package
with your help.
Cheers,
Refik Tuzaklı
Embedded Software Engineer
IT and Simulation Group
Savronik A.Ş.
Organize Sanayi Bölgesi
20. Cadde No: 19 26110
Eskişehir Turkey
Tel : (+90 222) 236 1590
Faks: (+90 222) 236 1589
www.savronik.com.tr
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180605/f9c9d606/attachment-0002.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: refik_tuzakli.vcf
Type: text/x-vcard
Size: 4 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180605/f9c9d606/attachment-0002.vcf>
More information about the buildroot
mailing list