Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix cp -n #3483

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

michacassola
Copy link

Running the wizard script for installation gives me the following output a few times:

cp: warning: behavior of -n is non-portable and may change in future; use --update=none instead

So I updated the commands and comments regarding that.

Copy link

@michacassola: There are no 'kind' label on this PR. You need a 'kind' label to generate the release automatically.

  • /kind feature
  • /kind enhancement
  • /kind refactoring
  • /kind fix
  • /kind chore
  • /kind dependencies
Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

Copy link

@michacassola: There are no area labels on this PR. You can add as many areas as you see fit.

  • /area agent
  • /area local-api
  • /area cscli
  • /area appsec
  • /area security
  • /area configuration
Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

@michacassola
Copy link
Author

michacassola commented Feb 24, 2025

/kind fix
/area configuration

@LaurenceJJones
Copy link
Contributor

@mmetc should we use rsync instead of cp or since it outside of the container we should stick to cp?

@LaurenceJJones LaurenceJJones added this to the 1.6.6 milestone Feb 24, 2025
@mmetc
Copy link
Contributor

mmetc commented Feb 25, 2025

@mmetc should we use rsync instead of cp or since it outside of the container we should stick to cp?

-1, you can't have a dependency on rsync just to install something

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

Successfully merging this pull request may close these issues.

3 participants