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

Add prototype dead method script #4

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

Conversation

ZitaNemeckova
Copy link
Collaborator

@ZitaNemeckova ZitaNemeckova commented May 6, 2019

  • Run in manageiq
  • you have to have manageiq and manageiq-ui-classic folders at same place
  • without params just runs debride on current branch
  • with one param (name of a branch) it runs debride on that branch
  • with two params (names of branch) it runs debride on both branches and compares differences between the two

TODO:

  • move ruby part to sh

git checkout $0
bin/update
fi
ruby /Users/zita/Desktop/all.rb > "$ALLOWEDMETHODS"
Copy link
Owner

@himdel himdel May 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/Users/zita/Desktop/ 🙀

dirname "$0"

should work :) ($0 is the current script)

bin/update
fi
ruby /Users/zita/Desktop/all.rb > "$ALLOWEDMETHODS"
sed -i "" '/^\*/ d' "$ALLOWEDMETHODS"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to me, this always fails with

sed: can't read /^\*/ d: No such file or directory

I must be doing something wrong

Copy link
Owner

@himdel himdel May 14, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mac vs Linux:

in macs, -i takes a required parameter (which extension to add to the backup file)
on linux, -i takes an optional parameter, and space means it's not there (so the "" becomes the regex, and the next param a filename)

=> sed -i" " ... should work on both

@romanblanco
Copy link
Contributor

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