huiqing / percept2

Concurrent profiling tool for Erlang
135 stars 34 forks source link

Error when starting web_server #17

Open SamuelCox opened 7 years ago

SamuelCox commented 7 years ago

Hi, I get the following trace when starting the web_server, any idea why this happens? As far as I can tell the install is correct, and I have the graphviz dependency.

{error, {inets, {{shutdown, {failed_to_start_child, {httpd_manager,any,8888,default}, {error, {invalid_option,{module_does_not_exist,mod_include}}}}}, {child,undefined, {httpd_instance_sup,any,8888,default}, {httpd_instance_sup,start_link, [[{server_root, "/home/cst/sc630/erlang-lib/percept2-1.0/priv/server_root"}, {document_root, "/home/cst/sc630/erlang-lib/percept2-1.0/priv/server_root/htdocs"}, {eval_script_alias,{"/eval",[io]}}, {erl_script_alias, {"/cgi-bin",[percept2_graph,percept2_html,io]}}, {script_alias, {"/cgi-bin/", "/home/cst/sc630/erlang-lib/percept2-1.0/priv/server_root/cgi-bin"}}, {alias, {"/javascript/", "/home/cst/sc630/erlang-lib/percept2-1.0/priv/server_root/scripts/"}}, {alias, {"/images/", "/home/cst/sc630/erlang-lib/percept2-1.0/priv/server_root/images/"}}, {alias, {"/svgs/", "/home/cst/sc630/erlang-lib/percept2-1.0/priv/server_root/svgs/"}}, {alias, {"/css/", "/home/cst/sc630/erlang-lib/percept2-1.0/priv/server_root/css/"}}, {alias, {"/tree/", "/home/cst/sc630/erlang-lib/percept2-1.0/priv/server_root/tree/"}}, {default_type,"text/plain"}, {directory_index,[[...]]}, {mime_types,[...]}, {modules,...}, {...}|...], 15000,[]]}, permanent,infinity,supervisor, [httpd_instance_sup]}}}}

=ERROR REPORT==== 9-Jan-2017::16:52:54 === Failed initiating web server: undefined {invalid_option,{module_does_not_exist,mod_include}}

SamuelCox commented 7 years ago

Hi, I've figured out that the problem is this specific line: https://github.com/huiqing/percept2/blob/master/src/percept2.erl#L376

inets:start(httpd, Config) is failing, presumably due to the mod_include part of the config. Does the app need this module or can I comment this line out?

SamuelCox commented 7 years ago

Hi again. For what it's worth, superficially my suggested fix has fixed it. Not sure if it will cause me to have problems down the line. I can send in a pull request or you can just do it yourself if you'd like, since its such a trivial change.

huiqing commented 7 years ago

Thanks Samuel. Submit a pull request please , then I can check and merge.

Regards, Huiqing From: Samuel [mailto:notifications@github.com] Sent: 09 January 2017 22:40 To: huiqing/percept2 Cc: Subscribed Subject: Re: [huiqing/percept2] Error when starting web_server (#17)

Hi again. For what it's worth, superficially my suggested fix has fixed it. Not sure if it will cause me to have problems down the line. I can send in a pull request or you can just do it yourself if you'd like, since its such a trivial change.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/huiqing/percept2/issues/17#issuecomment-271430845, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAk6XxVnNeTHyT4_dfLuXpu_cqWM2OQjks5rQrcqgaJpZM4LefCl.

NOTICE AND DISCLAIMER This email contains BT information, which may be privileged or confidential. It's meant only for the individual(s) or entity named above. If you're not the intended recipient, note that disclosing, copying, distributing or using this information is prohibited. If you've received this email in error, please let me know immediately on the email address above. Thank you.

We monitor our email system, and may record your emails.

EE Limited Registered office:Trident Place, Mosquito Way, Hatfield, Hertfordshire, AL10 9BW Registered in England no: 02382161

EE Limited is a wholly owned subsidiary of:

British Telecommunications plc Registered office: 81 Newgate Street London EC1A 7AJ Registered in England no: 1800000

SamuelCox commented 7 years ago

Submitted #18