Skip to content

Commit bcbc7b4

Browse files
authored
Merge pull request #269 from DannyBen/update/errors-go-to-stderr
Redirect error messages to stderr
2 parents b8a477c + 54008d2 commit bcbc7b4

16 files changed

+24
-24
lines changed

lib/bashly/views/argument/validations.gtx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ if validate
33
= view_marker
44

55
> if [[ -n $(validate_{{ validate }} "$1") ]]; then
6-
> printf "{{ strings[:validation_error] }}\n" "{{ name.upcase }}" "$(validate_{{ validate }} "$1")"
6+
> printf "{{ strings[:validation_error] }}\n" "{{ name.upcase }}" "$(validate_{{ validate }} "$1")" >&2
77
> exit 1
88
> fi
99
>

lib/bashly/views/command/catch_all_filter.gtx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ if catch_all.required?
22
= view_marker
33

44
> if [[ ${#other_args[@]} -eq 0 ]]; then
5-
> printf "{{ strings[:missing_required_argument] % { arg: catch_all.label, usage: usage_string } }}\n"
5+
> printf "{{ strings[:missing_required_argument] % { arg: catch_all.label, usage: usage_string } }}\n" >&2
66
> exit 1
77
> fi
88
>

lib/bashly/views/command/command_fallback.gtx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
= view_marker
22

33
> "" )
4-
> {{ function_name }}_usage
4+
> {{ function_name }}_usage >&2
55
> exit 1
66
> ;;
77
>
@@ -18,7 +18,7 @@ elsif extensible.is_a? String
1818
> if [[ -x "$(command -v "{{ extensible }}")" ]]; then
1919
> exec {{ extensible }} "$@"
2020
> else
21-
> printf "{{ strings[:invalid_command] }}\n" "$action"
21+
> printf "{{ strings[:invalid_command] }}\n" "$action" >&2
2222
> exit 1
2323
> fi
2424
>
@@ -28,13 +28,13 @@ elsif extensible
2828
> shift
2929
> exec "{{ function_name }}-$action" "$@"
3030
> else
31-
> printf "{{ strings[:invalid_command] }}\n" "$action"
31+
> printf "{{ strings[:invalid_command] }}\n" "$action" >&2
3232
> exit 1
3333
> fi
3434
>
3535

3636
else
37-
> printf "{{ strings[:invalid_command] }}\n" "$action"
37+
> printf "{{ strings[:invalid_command] }}\n" "$action" >&2
3838
> exit 1
3939
> ;;
4040
>

lib/bashly/views/command/dependencies_filter.gtx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ if dependencies
33

44
dependencies.each do |dependency|
55
> if ! [[ -x "$(command -v {{ dependency }})" ]]; then
6-
> printf "{{ strings[:missing_dependency] % { dependency: dependency } }}\n"
6+
> printf "{{ strings[:missing_dependency] % { dependency: dependency } }}\n" >&2
77
> exit 1
88
> fi
99
end

lib/bashly/views/command/environment_variables_filter.gtx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ if default_environment_variables.any? or required_environment_variables.any?
55
if required_environment_variables.any?
66
required_environment_variables.each do |env_var|
77
> if [[ -z "${<%= env_var.name.upcase %>:-}" ]]; then
8-
> printf "{{ strings[:missing_required_environment_variable] % { var: env_var.name.upcase } }}\n"
8+
> printf "{{ strings[:missing_required_environment_variable] % { var: env_var.name.upcase } }}\n" >&2
99
> exit 1
1010
> fi
1111
end

lib/bashly/views/command/parse_requirements_case_simple.gtx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,12 @@ if args.any?
1212
end
1313

1414
> else
15-
> printf "{{ strings[:invalid_argument] }}\n" "$key"
15+
> printf "{{ strings[:invalid_argument] }}\n" "$key" >&2
1616
> exit 1
1717
> fi
1818

1919
else
20-
> printf "{{ strings[:invalid_argument] }}\n" "$key"
20+
> printf "{{ strings[:invalid_argument] }}\n" "$key" >&2
2121
> exit 1
2222

2323
end

lib/bashly/views/command/parse_requirements_while.gtx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ if catch_all.enabled?
1717
> ;;
1818

1919
else
20-
> printf "<%= strings[:invalid_flag] %>\n" "$key"
20+
> printf "<%= strings[:invalid_flag] %>\n" "$key" >&2
2121
> exit 1
2222
> ;;
2323

lib/bashly/views/command/required_args_filter.gtx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ if required_args.any?
33

44
required_args.each do |arg|
55
> if [[ -z ${args[{{ arg.name }}]+x} ]]; then
6-
> printf "{{ strings[:missing_required_argument] % { arg: arg.name.upcase, usage: usage_string } }}\n"
6+
> printf "{{ strings[:missing_required_argument] % { arg: arg.name.upcase, usage: usage_string } }}\n" >&2
77
> exit 1
88
> fi
99
end

lib/bashly/views/command/required_flags_filter.gtx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ if required_flags.any?
33

44
required_flags.each do |flag|
55
> if [[ -z ${args[{{ flag.long }}]+x} ]]; then
6-
> printf "{{ strings[:missing_required_flag] % { usage: flag.usage_string } }}\n"
6+
> printf "{{ strings[:missing_required_flag] % { usage: flag.usage_string } }}\n" >&2
77
> exit 1
88
> fi
99
end

lib/bashly/views/command/user_filter.gtx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ if filters
55
filters.each do |filter|
66
> filter_error=$(filter_{{ filter }})
77
> if [[ -n $filter_error ]]; then
8-
> echo "$filter_error"
8+
> echo "$filter_error" >&2
99
> exit 1
1010
> fi
1111
>

0 commit comments

Comments
 (0)