[MINOR] Only rename SparkR tar.gz if names mismatch

[shivaram]

What changes were proposed in this pull request?

For release builds the R_PACKAGE_VERSION and VERSION are the same (e.g., 2.1.0). Thus cp throws an error which causes the build to fail.

How was this patch tested?

Manually by executing the following script

set -o pipefail
set -e
set -x

touch a

R_PACKAGE_VERSION=2.1.0
VERSION=2.1.0

if [ "$R_PACKAGE_VERSION" != "$VERSION" ]; then
  cp a a
fi

[shivaram]

cc @rxin

[felixcheung]

can we mv instead?
https://github.com/apache/spark/pull/16249/files#diff-ba2c046d92a1d2b5b417788bfb5cb5f8R232

[felixcheung]

I found an issue with snapshot build just now - R CMD check doesn't handle the -SNAPSHOT part of the file name in SparkR_*tar.gz. Since it's only an issue with SNAPSHOT build I'll open a PR separately.

[shivaram]

Ah I see - yes - using mv is a simpler option - BTW when do we run R CMD check ? Is this for say when users download the tar.gz and run it ? (We can move this discussion to the JIRA)

[rxin]

Merging in master/branch-2.1.

[SparkQA]

Test build #70220 has finished for PR 16299 at commit fcc8265.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #70222 has finished for PR 16299 at commit 95ac227.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.