On Sat, Sep 29, 2018 at 11:18 PM <[email protected]> wrote: > > From: Changqing Li <[email protected]> > > fix below warning: > > do_package_qa: QA Issue: ELF binary 'xxx/usr/sbin/registry' > has relocations in .text [textrel] > > Add GOBUILDFLAGS which have -buildmode=pie to fix it
This is not universally true, so your configuration needs more explanation. We haven't needed these flags before, and I've never seen the warnings. I just build docker and dumped the environment, my GOBUILDFLAGS do not have buildmode=pie So clearly, your config is different. I'm not against adding the extra variable, but we are taking a the risk of injecting build configurations that we haven't needed before, and may not want. So I need to understand what configuration you are building that is making them required, and then figure out what else is being injected that wasn't being injected before. Same comment applies for all the QA fixes. Bruce > > Signed-off-by: Changqing Li <[email protected]> > --- > .../docker-distribution/docker-distribution_git.bb | 1 + > .../files/0001-fix-do_package_qa-error.patch | 32 > ++++++++++++++++++++++ > 2 files changed, 33 insertions(+) > create mode 100644 > recipes-containers/docker-distribution/files/0001-fix-do_package_qa-error.patch > > diff --git > a/recipes-containers/docker-distribution/docker-distribution_git.bb > b/recipes-containers/docker-distribution/docker-distribution_git.bb > index add5ce1..7c7bce2 100644 > --- a/recipes-containers/docker-distribution/docker-distribution_git.bb > +++ b/recipes-containers/docker-distribution/docker-distribution_git.bb > @@ -6,6 +6,7 @@ LIC_FILES_CHKSUM = > "file://LICENSE;md5=d2794c0df5b907fdace235a619d80314" > SRCREV_distribution="48294d928ced5dd9b378f7fd7c6f5da3ff3f2c89" > SRC_URI = > "git://github.com/docker/distribution.git;branch=release/2.6;name=distribution;destsuffix=git/src/github.com/docker/distribution > \ > file://docker-registry.service \ > + file://0001-fix-do_package_qa-error.patch \ > " > > PACKAGES =+ "docker-registry" > diff --git > a/recipes-containers/docker-distribution/files/0001-fix-do_package_qa-error.patch > > b/recipes-containers/docker-distribution/files/0001-fix-do_package_qa-error.patch > new file mode 100644 > index 0000000..269b0cd > --- /dev/null > +++ > b/recipes-containers/docker-distribution/files/0001-fix-do_package_qa-error.patch > @@ -0,0 +1,32 @@ > +From 4b9d701fabff8e7969db081406d00fa9fe46b3fd Mon Sep 17 00:00:00 2001 > +From: Changqing Li <[email protected]> > +Date: Thu, 27 Sep 2018 11:05:51 +0800 > +Subject: [PATCH] fix do_package_qa error > + > +fix below error: > +do_package_qa: QA Issue: ELF binary 'xxx/usr/sbin/registry' > +has relocations in .text [textrel] > + > +Upstream-Status: Inappropriate [oe-specific] > + > +Signed-off-by: Changqing Li <[email protected]> > +--- > + Makefile | 2 +- > + 1 file changed, 1 insertion(+), 1 deletion(-) > + > +diff --git a/Makefile b/Makefile > +index 47b8f1d..13b0f92 100644 > +--- a/Makefile > ++++ b/Makefile > +@@ -39,7 +39,7 @@ GODEP=$(shell which godep || echo '') > + > + ${PREFIX}/bin/registry: $(GOFILES) > + @echo "+ $@" > +- @go build -tags "${DOCKER_BUILDTAGS}" -o $@ ${GO_LDFLAGS} > ${GO_GCFLAGS} ./cmd/registry > ++ @go build -tags "${DOCKER_BUILDTAGS}" -o $@ ${GO_LDFLAGS} > ${GOBUILDFLAGS} ${GO_GCFLAGS} ./cmd/registry > + > + ${PREFIX}/bin/digest: $(GOFILES) > + @echo "+ $@" > +-- > +2.7.4 > + > -- > 2.7.4 > > -- > _______________________________________________ > meta-virtualization mailing list > [email protected] > https://lists.yoctoproject.org/listinfo/meta-virtualization -- "Thou shalt not follow the NULL pointer, for chaos and madness await thee at its end" -- _______________________________________________ meta-virtualization mailing list [email protected] https://lists.yoctoproject.org/listinfo/meta-virtualization
