make.sh: Clarify options need to come before the actual command

Message ID 31f0870d-f1c9-9f29-48d6-eaa9040aab7e@ipfire.org
State Accepted
Commit a55f90e9cd6ca3fcd1ac5caa89df830d5326920e
Headers
Series make.sh: Clarify options need to come before the actual command |

Commit Message

Peter Müller Jan. 9, 2022, 12:54 p.m. UTC
  This might prevent some misunderstandings, as the help of make.sh does
not precisely state where the options (target architecture) needs to be
specified.

See: https://lists.ipfire.org/pipermail/development/2022-January/011911.html

Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
---
 make.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Michael Tremer Jan. 9, 2022, 3:20 p.m. UTC | #1
Reviewed-by: Michael Tremer <michael.tremer@ipfire.org>

> On 9 Jan 2022, at 12:54, Peter Müller <peter.mueller@ipfire.org> wrote:
> 
> This might prevent some misunderstandings, as the help of make.sh does
> not precisely state where the options (target architecture) needs to be
> specified.
> 
> See: https://lists.ipfire.org/pipermail/development/2022-January/011911.html
> 
> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
> ---
> make.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/make.sh b/make.sh
> index 7568ee232..3f45e1953 100755
> --- a/make.sh
> +++ b/make.sh
> @@ -1957,7 +1957,7 @@ check-manualpages)
> 	fi
> 	;;
> *)
> -	echo "Usage: $0 {build|changelog|clean|gettoolchain|downloadsrc|shell|sync|toolchain|update-contributors|find-dependencies|check-manualpages}"
> +	echo "Usage: $0 [OPTIONS] {build|changelog|clean|gettoolchain|downloadsrc|shell|sync|toolchain|update-contributors|find-dependencies|check-manualpages}"
> 	cat doc/make.sh-usage
> 	;;
> esac
> -- 
> 2.26.2
  

Patch

diff --git a/make.sh b/make.sh
index 7568ee232..3f45e1953 100755
--- a/make.sh
+++ b/make.sh
@@ -1957,7 +1957,7 @@  check-manualpages)
 	fi
 	;;
 *)
-	echo "Usage: $0 {build|changelog|clean|gettoolchain|downloadsrc|shell|sync|toolchain|update-contributors|find-dependencies|check-manualpages}"
+	echo "Usage: $0 [OPTIONS] {build|changelog|clean|gettoolchain|downloadsrc|shell|sync|toolchain|update-contributors|find-dependencies|check-manualpages}"
 	cat doc/make.sh-usage
 	;;
 esac