Project

Profile

Help

HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. Read more...

Feature #891558

coding_styler.sh

Added by Marko Lindqvist almost 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Category:
Bootstrap
Sprint/Milestone:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

In preparation to give sdl2-client Coding Style overhaul I created a simple script to replace strings with another in a certain path.

We already do have scripts/replace, so one could argue we don't need another script for the same job. I just like mine more.
I can of course use my own script even if it's not put to git, so this review is more about if it should be made available for others too.

0013-Add-coding_styler.sh.patch (1.33 KB) 0013-Add-coding_styler.sh.patch Marko Lindqvist, 2020-10-18 07:33 PM
0004-Add-coding_styler.sh.patch (1.34 KB) 0004-Add-coding_styler.sh.patch Marko Lindqvist, 2020-10-23 02:16 PM
0035-Reimplement-scripts-replace.patch (1.57 KB) 0035-Reimplement-scripts-replace.patch Marko Lindqvist, 2020-10-25 01:12 PM

History

#1 Updated by Marko Lindqvist almost 2 years ago

#2 Updated by Marko Lindqvist almost 2 years ago

- Support files in entire subdirectory hierarchy, not just in the (root) directory given

#3 Updated by Marko Lindqvist almost 2 years ago

  • Status changed from Resolved to In Progress

Marko Lindqvist wrote:

We already do have scripts/replace, so one could argue we don't need another script for the same job. I just like mine more.

The solution I'm now preparing is that we get rid of old version of 'replace' by just overwriting it with the new script.

#4 Updated by Marko Lindqvist almost 2 years ago

Marko Lindqvist wrote:

The solution I'm now preparing is that we get rid of old version of 'replace' by just overwriting it with the new script.

As done in attached patch.

#5 Updated by Marko Lindqvist almost 2 years ago

  • Status changed from Resolved to Closed
  • Assignee set to Marko Lindqvist

Also available in: Atom PDF