Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add documentation about lighttpd usage #126

Merged
merged 1 commit into from
Feb 9, 2017
Merged

Conversation

jronak
Copy link
Contributor

@jronak jronak commented Feb 8, 2017

Added instructions for installing and running cgimap with lighttpd

README Outdated

/usr/sbin/lighttpd -f lighttpd.conf

Run CGImap on the command line with a command like
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't we conclude that lighttpd handles starting cgimap and if you're using the config file you don't need to also start the binary?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, FastCGI handles starting the binary, but I thought we are starting the binary for debugging?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If lighttpd starts the binary then there's no purpose running a second copy

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: we still need to document how to start the binary, just make it clear you don't do that when running it through lighttpd with the supplied config file

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated PR, please review.

Added instructions and commands to install lighttpd and run lighttpd with CGImap/lighttpd.conf
@zerebubuth zerebubuth merged commit 4975a2c into zerebubuth:master Feb 9, 2017
@zerebubuth
Copy link
Owner

Thanks!

@jronak jronak deleted the doc branch February 10, 2017 04:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants