CCAAS Builders

- adjust the makefile to have targets for linux-amd64/darwin-amd64/windows-amd64 - adjust the packaging as suggested in issue 3286 Signed-off-by: Matthew B White <whitemat@uk.ibm.com>
hyperledger · Jun 22, 2022 · edab0fa · edab0fa
1 parent 6e5e693
commit edab0fa
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 11 deletions.
diff --git a/Makefile b/Makefile
@@ -265,7 +265,7 @@ release-all: check-go-version $(RELEASE_PLATFORMS:%=release/%)
 .PHONY: $(RELEASE_PLATFORMS:%=release/%)
 $(RELEASE_PLATFORMS:%=release/%): GO_LDFLAGS = $(METADATA_VAR:%=-X $(PKGNAME)/common/metadata.%)
 $(RELEASE_PLATFORMS:%=release/%): release/%: $(foreach exe,$(RELEASE_EXES),release/%/bin/$(exe))
-$(RELEASE_PLATFORMS:%=release/%): ccaasbuilder
+$(RELEASE_PLATFORMS:%=release/%): release/%: ccaasbuilder/%
 
 # explicit targets for all platform executables
 $(foreach platform, $(RELEASE_PLATFORMS), $(RELEASE_EXES:%=release/$(platform)/bin/%)):
@@ -347,11 +347,17 @@ docs:
 	@docker run --rm -v $$(pwd):/docs n42org/tox:3.4.0 sh -c 'cd /docs && tox -e docs'
 
 .PHONY: ccaasbuilder-clean
-ccaasbuilder-clean:
-	rm -rf $(MARCH:%=release/%)/bin/ccaas_builder
+ccaasbuilder-clean/%:
+	$(eval platform = $(patsubst ccaasbuilder/%,%,$@) )
+	cd ccaas_builder &&	rm -rf $(strip $(platform))
 
 .PHONY: ccaasbuilder
-ccaasbuilder: ccaasbuilder-clean
-	cd ccaas_builder && go test -v ./cmd/detect && go build -o ../$(MARCH:%=release/%)/bin/ccaas_builder/bin/ -buildvcs=false ./cmd/detect/
-	cd ccaas_builder && go test -v ./cmd/build && go build -o ../$(MARCH:%=release/%)/bin/ccaas_builder/bin/ -buildvcs=false ./cmd/build/
-	cd ccaas_builder && go test -v ./cmd/release && go build -o ../$(MARCH:%=release/%)/bin/ccaas_builder/bin/ -buildvcs=false ./cmd/release/
+ccaasbuilder/%: ccaasbuilder-clean
+	$(eval platform = $(patsubst ccaasbuilder/%,%,$@) ) 
+	$(eval GOOS = $(word 1,$(subst -, ,$(platform))))
+	$(eval GOARCH = $(word 2,$(subst -, ,$(platform))))
+	cd ccaas_builder && go test -v ./cmd/detect && GOOS=$(GOOS) GOARCH=$(GOARCH) go build -o ./bin/$(strip $(platform))/ ./cmd/detect/
+	cd ccaas_builder && go test -v ./cmd/build && GOOS=$(GOOS) GOARCH=$(GOARCH) go build -o ./bin/$(strip $(platform))/ ./cmd/build/
+	cd ccaas_builder && go test -v ./cmd/release && GOOS=$(GOOS) GOARCH=$(GOARCH) go build -o ./bin/$(strip $(platform))/ ./cmd/release/
+
+ccaasbuilder: ccaasbuilder/$(MARCH)
diff --git a/ci/scripts/create_binary_package.sh b/ci/scripts/create_binary_package.sh
@@ -5,10 +5,15 @@
 set -euo pipefail
 
 make "release/${TARGET}"
-mkdir "release/${TARGET}/config"
-mv sampleconfig/*yaml "release/${TARGET}/config"
+mkdir -p "release/${TARGET}/config"
+mkdir -p "release/${TARGET}/builders/ccaas"
+
+# cp not move otherwise this breaks your source tree
+cp sampleconfig/*yaml "release/${TARGET}/config"
+cp ccaas_builder/bin/${TARGET}/* "release/${TARGET}/builders/ccaas"
+
 cd "release/${TARGET}"
 if [ "$TARGET" == "windows-amd64" ]; then
     for FILE in bin/*; do mv $FILE $FILE.exe; done
 fi
-tar -czvf "hyperledger-fabric-${TARGET}-${RELEASE}.tar.gz" bin config
+tar -czvf "hyperledger-fabric-${TARGET}-${RELEASE}.tar.gz" bin config builders
diff --git a/images/peer/Dockerfile b/images/peer/Dockerfile
@@ -35,6 +35,6 @@ VOLUME /var/hyperledger
 COPY --from=peer /go/src/github.com/hyperledger/fabric/build/bin /usr/local/bin
 COPY --from=peer /go/src/github.com/hyperledger/fabric/sampleconfig/msp ${FABRIC_CFG_PATH}/msp
 COPY --from=peer /go/src/github.com/hyperledger/fabric/sampleconfig/core.yaml ${FABRIC_CFG_PATH}/core.yaml
-COPY --from=peer /go/src/github.com/hyperledger/fabric/release/linux-amd64/bin/ccaas_builder/bin/ /opt/hyperledger/ccaas_builder/bin/
+COPY --from=peer /go/src/github.com/hyperledger/fabric/ccaas_builder/bin/ /opt/hyperledger/ccaas_builder/bin/
 EXPOSE 7051
 CMD ["peer","node","start"]