Skip to content

Conversation

@myqi
Copy link

@myqi myqi commented Sep 26, 2018

Signed-off-by: Mingyuan Qi [email protected]

Copy link

@saulwold saulwold left a comment

Choose a reason for hiding this comment

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

I guess I missed this one, this is another one that should go upstream first.

Also, for the sed command, why not use subtituion so you don't hard code the "/usr/local/etc" and set can use alternative characters for the delimiter so you don't have use ''

sed -i -e 's:${sysconfdir}:<your rmp/shell variable here>:g ...

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.

2 participants