Merge pull request #145 from BlankOn/134

#134 #142 Take only *.deb and *.buildinfo (if any) files from pbuilder result dir.

Author: herpiko

cmd/builder/builder.go

@@ -150,7 +150,7 @@ func BuildPackage(payload string) (next string, err error) {
 
 	// Building the package
 	cmdStr = "docker run -v " + irgshConfig.Builder.Workdir + "/artifacts/" + raw["taskUUID"].(string)
-	cmdStr += ":/tmp/build --privileged=true -i pbocker bash -c /build.sh"
+	cmdStr += ":/tmp/build --privileged=true -i pbocker bash -c /build.sh" // See builder/init.go to modify this script
 	fmt.Println(cmdStr)
 	_, err = systemutil.CmdExec(
 		cmdStr,

cmd/builder/init.go

@@ -169,12 +169,14 @@ func InitBuilder() (err error) {
 		return
 	}
 
+	// build.sh script is written here.
+	// We're only taking the *.deb and *.buildinfo (if any) files from pbuilder result
 	cmdStr = `echo 'FROM debian:latest' > ` + irgshConfig.Builder.Workdir + `/pbocker/Dockerfile && \
     echo 'RUN apt-get update && apt-get -y install pbuilder' >> ` + irgshConfig.Builder.Workdir + `/pbocker/Dockerfile && \
     echo 'RUN echo "MIRRORSITE=` + irgshConfig.Builder.UpstreamDistUrl + `" > /root/.pbuilderrc' >> ` + irgshConfig.Builder.Workdir + `/pbocker/Dockerfile && \
     echo 'RUN echo "USENETWORK=yes"' >> ` + irgshConfig.Builder.Workdir + `/pbocker/Dockerfile && \
     echo 'COPY base.tgz /var/cache/pbuilder/' >> ` + irgshConfig.Builder.Workdir + `/pbocker/Dockerfile && \
-    echo 'RUN echo "pbuilder --build /tmp/build/*.dsc && cp -vR /var/cache/pbuilder/result/* /tmp/build/" > /build.sh && chmod a+x /build.sh' >> ` + irgshConfig.Builder.Workdir + `/pbocker/Dockerfile`
+    echo 'RUN echo "pbuilder --build /tmp/build/*.dsc \n cp -vR /var/cache/pbuilder/result/*.deb /tmp/build/ \n cp -vR /var/cache/pbuilder/result/*.buildinfo /tmp/build/ || true" > /build.sh && chmod a+x /build.sh' >> ` + irgshConfig.Builder.Workdir + `/pbocker/Dockerfile`
 	_, err = systemutil.CmdExec(
 		cmdStr,
 		"Preparing Dockerfile",