stable filemode for zip distributions

andyb-elastic · andyb-elastic · commit 4247d38955fa · 2018-05-24T17:18:19.000-07:00
Applies default file and directory permissions to zip distributions similar to how they're set for the tar distributions. Previously zip distributions would retain permissions they had on the build host's working tree, which could vary depending on its umask For elastic#30799
diff --git a/distribution/archives/build.gradle b/distribution/archives/build.gradle
@@ -104,15 +104,23 @@ tasks.withType(AbstractArchiveTask) {
   baseName = "elasticsearch${ subdir.contains('oss') ? '-oss' : ''}"
 }
 
+Closure commonZipConfig = {
+  dirMode 0755
+  fileMode 0644
+}
+
 task buildIntegTestZip(type: Zip) {
+  configure(commonZipConfig)
   with archiveFiles(transportModulesFiles, 'zip', false)
 }
 
 task buildZip(type: Zip) {
+  configure(commonZipConfig)
   with archiveFiles(modulesFiles(false), 'zip', false)
 }
 
 task buildOssZip(type: Zip) {
+  configure(commonZipConfig)
   with archiveFiles(modulesFiles(true), 'zip', true)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -104,15 +104,23 @@ tasks.withType(AbstractArchiveTask) {`
`104`	`104`	`baseName = "elasticsearch${ subdir.contains('oss') ? '-oss' : ''}"`
`105`	`105`	`}`
`106`	`106`
	`107`	`+Closure commonZipConfig = {`
	`108`	`+ dirMode 0755`
	`109`	`+ fileMode 0644`
	`110`	`+}`
	`111`	`+`
`107`	`112`	`task buildIntegTestZip(type: Zip) {`
	`113`	`+ configure(commonZipConfig)`
`108`	`114`	`with archiveFiles(transportModulesFiles, 'zip', false)`
`109`	`115`	`}`
`110`	`116`
`111`	`117`	`task buildZip(type: Zip) {`
	`118`	`+ configure(commonZipConfig)`
`112`	`119`	`with archiveFiles(modulesFiles(false), 'zip', false)`
`113`	`120`	`}`
`114`	`121`
`115`	`122`	`task buildOssZip(type: Zip) {`
	`123`	`+ configure(commonZipConfig)`
`116`	`124`	`with archiveFiles(modulesFiles(true), 'zip', true)`
`117`	`125`	`}`
`118`	`126`