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

missing parameter in find #41

Open
sigpet opened this issue Dec 21, 2021 · 3 comments
Open

missing parameter in find #41

sigpet opened this issue Dec 21, 2021 · 3 comments

Comments

@sigpet
Copy link

sigpet commented Dec 21, 2021

https://github.com/cloudera/cloudera-scripts-for-log4j/blob/main/cm_cdp_cdh_log4j_jndi_removal.sh#L63
it's missing the -name parameter after -o

line should read
for tarfile in $(find -L $targetdir -name ".tar.gz" -o -name ".tgz"); do

I fixed it locally and the find does not error any more

@isuller
Copy link

isuller commented Dec 22, 2021

Thank you, fixed.

@lhoss
Copy link

lhoss commented Dec 27, 2021

It seems fixed by commit f277913
why has the github issue not been closed ??
( need to keep the list of issues and PRs as clean as possible , it started to get messy )

@isuller
Copy link

isuller commented Dec 31, 2021

@lhoss I don't have the right to close an issue - don't know who can. Maybe @sunilgovind knows?

sdevineni pushed a commit that referenced this issue Apr 13, 2022
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

No branches or pull requests

3 participants