Project

General

Profile

Actions

Feature #1847

closed

add -d option to spawn-fcgi

Added by raphaelh about 16 years ago. Updated over 15 years ago.

Status:
Fixed
Priority:
Normal
Target version:
-

Description

fixes problems with files not found


Files

100-spawn-fcgi_new_opt_directory.patch (2.42 KB) 100-spawn-fcgi_new_opt_directory.patch add -d option to spawn-fcgi raphaelh, 2008-12-15 16:54
Actions #1

Updated by icy about 16 years ago

  • Status changed from New to Need Feedback
  • Target version deleted (1.4.20)

Err, why don't you just cd to the directory before calling spawn-fcgi?

Actions #2

Updated by v about 16 years ago

i love this patch.

this is crap to cd in the directory before calling spawn-fcgi !!! A binary must be started anywhere and the -d option allows you to do that

Actions #3

Updated by raphaelh about 16 years ago

icy wrote:

Err, why don't you just cd to the directory before calling spawn-fcgi?

This is not intuitive, because it behaves differently from when the fastcgi daemon is automatically spawned by lighttpd:

  • before, I was telling lighttpd to launch the fastcgi daemon automatically, everything was working fine
  • then, I began using spawn-fcgi to launch the fastcgi daemon, but it was no longer finding the files it needed

That's why I made this quick patch to have a consistent behavior between the 2 methods of launching the fastcgi daemon.

Actions #4

Updated by Olaf-van-der-Spek about 16 years ago

raphaelh wrote:

This is not intuitive, because it behaves differently from when the fastcgi daemon is automatically spawned by lighttpd:

That's probably because relative paths are used somewhere. Why not use absolute paths?

Actions #5

Updated by raphaelh about 16 years ago

Olaf-van-der-Spek wrote:

raphaelh wrote:

This is not intuitive, because it behaves differently from when the fastcgi daemon is automatically spawned by lighttpd:

That's probably because relative paths are used somewhere. Why not use absolute paths?

Well yeah, I could just cd before spawning, or use absolute paths

and anyway I use the small patch on my lighttpd, so it's not a problem for me...

But I was just saying that it could save some time for other people when switching from lighttpd autospawn to spawn-fcgi, as this patch was helpful for me I wanted to share :)

Actions #6

Updated by Olaf-van-der-Spek about 16 years ago

raphaelh wrote:

and anyway I use the small patch on my lighttpd, so it's not a problem for me...

I still have no idea what the advantage of this patch over a 'manual' cd is...

Actions #7

Updated by icy about 16 years ago

  • Status changed from Need Feedback to Patch Pending
  • Pending changed from No to Yes

Well, at least it doesn't hurt and some people might find it convenient.

Actions #8

Updated by Olaf-van-der-Spek almost 16 years ago

icy wrote:

Well, at least it doesn't hurt and some people might find it convenient.

It does increase complexity of code and parameters. That's not a positive thing. ;)

Actions #9

Updated by stbuehler over 15 years ago

  • Project changed from Lighttpd to spawn-fcgi
  • Category deleted (spawn-fcgi)
Actions #10

Updated by stbuehler over 15 years ago

If you want to spawn an application in a chroot and want a specific current directory before spawning, a wrapper script is a little bit overkill (and don't forget you would have to add the interpreter to the chroot...).
And as this option looks easy i just added it :)

Actions #11

Updated by stbuehler over 15 years ago

  • Status changed from Patch Pending to Fixed
  • % Done changed from 0 to 100

Applied in changeset r34.

Actions

Also available in: Atom