Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1 v2] mongoose: startup script must use short options
@ 2013-11-25  9:03 Davide Viti
  2013-11-25  9:43 ` Peter Korsgaard
  0 siblings, 1 reply; 2+ messages in thread
From: Davide Viti @ 2013-11-25  9:03 UTC (permalink / raw)
  To: buildroot

From: Davide Viti <zinosat@tiscali.it>

mongoose fails to start because long options are not supported (as
shown by "mongoose -h")

Signed-off-by: Davide Viti <d.viti@infosolution.it>
---
 package/mongoose/S85mongoose | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/mongoose/S85mongoose b/package/mongoose/S85mongoose
index e9e8524..ecf896a 100755
--- a/package/mongoose/S85mongoose
+++ b/package/mongoose/S85mongoose
@@ -10,7 +10,7 @@ NAME=mongoose
 DESC="Mongoose HTTP server"
 
 DAEMON=`which mongoose`
-OPTIONS="-max_threads 3 -root /var/www  -ports 80"
+OPTIONS="-t 3 -r /var/www -p 80"
 
 [ -e /etc/default/mongoose ] && . /etc/default/mongoose
 
-- 
1.8.1.2

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH 1/1 v2] mongoose: startup script must use short options
  2013-11-25  9:03 [Buildroot] [PATCH 1/1 v2] mongoose: startup script must use short options Davide Viti
@ 2013-11-25  9:43 ` Peter Korsgaard
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Korsgaard @ 2013-11-25  9:43 UTC (permalink / raw)
  To: buildroot

>>>>> "Davide" == Davide Viti <zinosat@tiscali.it> writes:

 > From: Davide Viti <zinosat@tiscali.it>
 > mongoose fails to start because long options are not supported (as
 > shown by "mongoose -h")

That's not really true:

./mongoose -h
Mongoose version 3.7 (c) Sergey Lyubka, built Nov 25 2013
Usage:
  mongoose -A <htpasswd_file> <realm> <user> <passwd>
  mongoose <config_file>
  mongoose [-option value ...]

OPTIONS:
  -C cgi_pattern (default: "**.cgi$|**.pl$|**.php$")
  -E cgi_environment (default: "")
  -G put_delete_auth_file (default: "")
  -I cgi_interpreter (default: "")
  -P protect_uri (default: "")
  -R authentication_domain (default: "mydomain.com")
  -S ssi_pattern (default: "**.shtml$|**.shtm$")
  -T throttle (default: "")
  -a access_log_file (default: "")
  -d enable_directory_listing (default: "yes")
  -e error_log_file (default: "")
  -g global_auth_file (default: "")
  -i index_files (default: "index.html,index.htm,index.cgi,index.shtml,index.php")
  -k enable_keep_alive (default: "no")
  -l access_control_list (default: "")
  -m extra_mime_types (default: "")
  -p listening_ports (default: "8080")
  -r document_root (default: ".")
  -s ssl_certificate (default: "")
  -t num_threads (default: "20")
  -u run_as_user (default: "")
  -w url_rewrite_patterns (default: "")
  -x hide_files_patterns (default: "")
  -z request_timeout_ms (default: "30000")

Both forms are supported (E.G. -t or -num_threads), but for some reason
our init script used invalid arguments, that as far as I can see have
never been supported by mongoose.

I also prefer to use the (correct) long options, as that makes it more
clear what they do.

I've reworded the commit message to make this clear and committed,
thanks.

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-11-25  9:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-25  9:03 [Buildroot] [PATCH 1/1 v2] mongoose: startup script must use short options Davide Viti
2013-11-25  9:43 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox