FileCopyDetails.setMode not respected - CopySpec fileMode overrides

[aldendaley]

When configuring into-from CopySpec details for the ospackage/buildRpm, fileMode can be set for all files applicable to the the copy. There may be a need to set different file permissions on individual files. This can be accomplished through FileCopyDetails.setMode(), under a filesMatching(..) block. Gradle's native distribution plugin output from installDist respects the individually set modes, but the RPM doesn't.

ospackage {
	(..)
	into '/some/path/filePermissionTest'

	fileMode = 0664

	into('destDir1') {
		fileMode = 0644

		from ('srcDir1') {
			// srcDir1\file.txt
			// srcDir1\file.sh

			fileMode = 0640

			filesMatching('**/*.sh') {
				it.setMode(0700)
			}
		}
	}
}

installDist output and expected outcome - the .sh script is 0700

-rwx------. file.sh
-rw-r-----. file.txt

Installed RPM output - the .sh script gets the spec's fileMode 0640

-rw-r-----. file.sh
-rw-r-----. file.txt

https://github.com/nebula-plugins/gradle-ospackage-plugin/blob/ae9152c02ca066b0a3da99e6ccb7da46a083cb9d/src/main/groovy/com/netflix/gradle/plugins/rpm/RpmCopyAction.groovy#LL164C16-L164C16

The RpmCopyAction.visitFile looks like it will always apply the fileMode from the spec - would I be wrong in thinking visitFile(..) should not reference the spec's fileMode at all, and always rely on the fileDetails.mode?

int fileMode = lookup(specToLookAt, 'fileMode') ?: fileDetails.mode

becomes

int fileMode = fileDetails.mode