[aur-dev] [PATCH] Add check for if PATH_INFO exists
mark.weiman at markzz.com
Wed Feb 8 01:33:48 UTC 2017
On Tue, 2017-02-07 at 21:43 +0100, Lukas Fleischer wrote:
> I am not 100% convinced that this is a good idea. If somebody wants to
> setup a "proper" web server running aurweb, this workaround makes it
> harder to figure out what went wrong if PATH_INFO is unset due to some
> misconfiguration. On the other hand, being able to run a quick test
> using `php -S` is quite nice. Would you might adding a notice to the
> commit message that this is specifically for `php -S`?
If PATH_INFO is unset, then it will always return "/", so it would be fairly
obvious that something's wrong if you are only getting the root page when you
visit other pages.
> I also think that a note on how to test aurweb patches with `php -S`
> should be added to the documentation. Maybe in INSTALL, before the
> "real" setup instructions? Something like:
> For testing aurweb patches before submission, you can use [...]
> Note that you can only do limited testing using the PHP built-in web
> server. In particular, the SSH/Git interface will be unusable. For a
> detailed description on how to setup a full aurweb server, read the
> instructions below.
That second paragraph is not entirely true, you can still set up an SSH/Git
interface separate from the web interface. There's nothing too special about the
web interface that the SSH/Git interface depends on.
I think that while we are talking about testing installations, perhaps some
information on how to populate a database using sqlite instead of MySQL should
More information about the aur-dev