Skip to content

Conversation

@barracuda156
Copy link
Contributor

Use standard cp syntax to avoid failures

@G-Ragghianti
Copy link
Contributor

What failures does this avoid? cp -a is almost functionally equivalent to cp -pPR.

@barracuda156
Copy link
Contributor Author

What failures does this avoid? cp -a is almost functionally equivalent to cp -pPR.

Yes, but it is not supported with some versions of cp. And then the build fails with invalid option.

@G-Ragghianti
Copy link
Contributor

OK, so the change to to convert the GNU -a option to POSIX options. I wasn't sure how this would affect symlinks, but my tests look fine. Thanks.

@barracuda156
Copy link
Contributor Author

From experience, it works as expected as a replacement for -a.

@G-Ragghianti
Copy link
Contributor

The checks were not running due to an oversight in the self-hosted meta runner. I've fixed this, and the jobs are running now.

@mgates3 mgates3 merged commit 6ce688f into icl-utk-edu:master Feb 17, 2025
4 checks passed
@barracuda156 barracuda156 deleted the cp branch February 17, 2025 14:59
@barracuda156
Copy link
Contributor Author

Thanks for merging!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants