Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 508712 - dev-lang/php[fpm] add configtest command to /etc/init.d/php-fpm
Summary: dev-lang/php[fpm] add configtest command to /etc/init.d/php-fpm
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: PHP Bugs
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2014-04-25 18:37 UTC by Garry Filakhtov
Modified: 2015-11-23 18:17 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch to add configtest functionality (php-fpm.patch,557 bytes, patch)
2014-04-25 18:39 UTC, Garry Filakhtov
Details | Diff
Patch to add configtest functionality (fixed) (php-fpm.patch,560 bytes, patch)
2014-04-25 18:46 UTC, Garry Filakhtov
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Garry Filakhtov 2014-04-25 18:37:58 UTC
dev-lang/php does not provide configtest functionality for testing FPM config files. Patch of init.d script attached.

Reproducible: Always

Steps to Reproduce:
1. Emerge php with fpm flag enabled
2. Modify php-fpm.conf file
3. Run 'rc-service php-fpm configtest' or '/etc/init.d/php-fpm configtest'
Actual Results:  
gentoo ~ # /etc/init.d/php-fpm configtest
 * php-fpm: unknown function `configtest'

Expected Results:  
gentoo ~ # /etc/init.d/php-fpm configtest
 * Testing PHP FastCGI Process Manager Configuration ...
[25-Apr-2014 21:36:51] WARNING: [pool www] pm.start_servers is not set. It's been set to 20.                                                                                                                                           [ ok ]
Comment 1 Garry Filakhtov 2014-04-25 18:39:18 UTC
Created attachment 375712 [details, diff]
Patch to add configtest functionality
Comment 2 Garry Filakhtov 2014-04-25 18:46:20 UTC
Created attachment 375714 [details, diff]
Patch to add configtest functionality (fixed)

Missed "-t" flag
Comment 3 Michael Orlitzky gentoo-dev 2015-11-23 18:17:49 UTC
Thanks! I've just committed the fix:

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=334ee76ee647bfeec1b0e81986eb94ea016358d8

Note that the affected package is app-eselect/eselect-php (not dev-lang/php), so you'll need to upgrade that to get the new init script.