Rudder

This is a very high priority bug for us. It means that many of our Techniques won't work in Rudder 2.6 who's release is planned to start in a few weeks. Please start fixing, to use only ${}.

Jonathan CLARKE wrote:

This is a very high priority bug for us. It means that many of our Techniques won't work in Rudder 2.6 who's release is planned to start in a few weeks. Please start fixing, to use only ${}.

Ok, I'm starting to think about a sed comamnd to change each $() by ${}.

I've tried this

sed 's@^\(.*\)\$(\(\[^)\]*\))\(.*\)$@\1\$\{\2\}\3@g'

But it does not match end parenthesis related to "$(".

Pull Request URL added: https://github.com/Normation/rudder-techniques/pull/61

Nicolas, you forgot the initial-promises part, which also contains $(${})
could you modify them as well ?

Thanks

Nicolas CHARLES wrote:

Nicolas, you forgot the initial-promises part, which also contains $(${})
could you modify them as well ?

Thanks

Ok, I've updated the PR to modify the initial-promises too.
To resume, I've used theses commands:

• Change $() into ${}
  

  sed 's@\$(\([^\(\)]*\))@$\{\1\}@g'
  

• Change () into {} but was few used
  

  sed 's/\@(\(.*\))/\@{\1}/g'
  

Nico, could you review it, please ?

Applied in changeset commit:ff72beb0a3509df84af062da35712081c6b3be37.

This ticket has been addressed in version 2.6.0~beta1 of Rudder, which has just been released. Please see the changelog here: https://www.rudder-project.org/foswiki/System/Documentation:ChangeLog26.