[PATCH v1 compass-ci 2/2] sched: result root add "." and replace "/" to "-"
Before: archlinuxaur-zzookeeper-HEAD After: archlinux-aur-z-zookeeper-HEAD Signed-off-by: Cao Xueliang <caoxl78320@163.com> --- src/scheduler/pp_params.cr | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/scheduler/pp_params.cr b/src/scheduler/pp_params.cr index 28ec6ef..da8d8f3 100644 --- a/src/scheduler/pp_params.cr +++ b/src/scheduler/pp_params.cr @@ -51,8 +51,8 @@ class Job private def format_string(original_str) temp = [] of String original_str.each_char do |char| - if "#{char}" =~ /-|\w/ - temp << "#{char}" + if "#{char}" =~ /-|\.|\/|\w/ + temp << "#{char}".gsub("\/", "-") end end -- 2.23.0
On Wed, Dec 02, 2020 at 10:33:39AM +0800, Cao Xueliang wrote:
Before: archlinuxaur-zzookeeper-HEAD After: archlinux-aur-z-zookeeper-HEAD
The content of the modification is not reflected.
Signed-off-by: Cao Xueliang <caoxl78320@163.com> --- src/scheduler/pp_params.cr | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/scheduler/pp_params.cr b/src/scheduler/pp_params.cr index 28ec6ef..da8d8f3 100644 --- a/src/scheduler/pp_params.cr +++ b/src/scheduler/pp_params.cr @@ -51,8 +51,8 @@ class Job private def format_string(original_str) temp = [] of String original_str.each_char do |char| - if "#{char}" =~ /-|\w/ - temp << "#{char}" + if "#{char}" =~ /-|\.|\/|\w/ + temp << "#{char}".gsub("\/", "-")
"/" doesn't need to escape. Why not replace the original_str directly? original_str.gsub('/', '-') Thanks, RenWen
end end
-- 2.23.0
On Wed, Dec 02, 2020 at 01:03:21PM +0800, Ren Wen wrote:
On Wed, Dec 02, 2020 at 10:33:39AM +0800, Cao Xueliang wrote:
Before: archlinuxaur-zzookeeper-HEAD After: archlinux-aur-z-zookeeper-HEAD
The content of the modification is not reflected.
I just show the result, "archlinux/aur-z/zookeeper" is the original string.
Signed-off-by: Cao Xueliang <caoxl78320@163.com> --- src/scheduler/pp_params.cr | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/scheduler/pp_params.cr b/src/scheduler/pp_params.cr index 28ec6ef..da8d8f3 100644 --- a/src/scheduler/pp_params.cr +++ b/src/scheduler/pp_params.cr @@ -51,8 +51,8 @@ class Job private def format_string(original_str) temp = [] of String original_str.each_char do |char| - if "#{char}" =~ /-|\w/ - temp << "#{char}" + if "#{char}" =~ /-|\.|\/|\w/ + temp << "#{char}".gsub("\/", "-")
"/" doesn't need to escape.
Why not replace the original_str directly? original_str.gsub('/', '-')
Good! Thanks, Xueliang
Thanks, RenWen
end end
-- 2.23.0
participants (2)
-
Cao Xueliang -
Ren Wen