Package Details: php-cs-fixer 1.12.0-1

Git Clone URL: https://aur.archlinux.org/php-cs-fixer.git (read-only)
Package Base: php-cs-fixer
Description: Analyzes some PHP source code and tries to fix coding standards issues (PSR-1 and PSR-2 compatible).
Upstream URL: http://cs.sensiolabs.org/
Keywords: php
Licenses: MIT
Submitter: bgaleotti
Maintainer: bgaleotti (sanduhrs)
Last Packager: bgaleotti
Votes: 16
Popularity: 0.647923
First Submitted: 2012-08-10 16:53
Last Updated: 2016-08-17 13:27

Dependencies (4)

Required by (0)

Sources (1)

Latest Comments

supermario commented on 2016-02-13 19:20

license needs to be an array now. Can you please update the PKGBUILD?

license=("MIT")

sanduhrs commented on 2016-01-15 11:33

Fixed the problems, current package should work fine.

soyel commented on 2016-01-04 04:55

@sanduhrs PHP in extra is now PHP 7. It causes several problems with the installation of this package with a default PHP install.

* Phar is now bundled inside PHP (See Arch PHP maintainer blog : https://pierre-schmitz.com/php-7-on-arch-linux/).
* "open_basedir" is now commented by default (that means no open_basedir in effect).

Building...
==> Starting check()...
Warning: You must add ':/usr/' to 'open_basedir' setting in your php.ini to use php-cs-fixer
==> ERROR: Makepkg was unable to build php-cs-fixer.

Got rid of the whole "check()" function and few "-d extension=phar.so" inside the "build()" function made the package installation successful.

sanduhrs commented on 2015-12-21 15:02

Updated the package to inform the user of the runtime requirements.

lakiluki commented on 2015-12-19 14:03

PHP Fatal error: Class 'Phar' not found in /usr/bin/composer on line 23

Getting this error when installing the package.


EDIT: Got it to work by adding the phar.so extension to php.ini and adding /usr/bin to open_basedir.

jgmdev commented on 2015-12-03 22:55

@sanduhrs phar.so comes bundled with the php package so there's no need for the install file, since you already indicated php is a dependency.

Edit:
Never mind I understand what you mean, maybe it should warn about the requirement of enabling phar.so for php-cs-fixer to work after install but not exit with error message. Ahh it could also add a .ini file into /etc/php/conf.d to enable phar.so

sanduhrs commented on 2015-12-03 17:58

@jgmdev we still have a runtime dependency on phar.so to be enabled. So the install file's still needed.

jgmdev commented on 2015-12-03 17:34

@sanduhrs you forgot to remove php-cs-fixer.install

sanduhrs commented on 2015-12-02 09:29

@jgmdev good point! Changed the package accordingly, thanks.
Updated to latest upstream, too.

jgmdev commented on 2015-12-02 02:41

Hi, prompting the user to enable phar.so and disable phar.readonly isn't necessary if you replace 'php-box build' with:

php -d extension=phar.so -d phar.readonly=0 /usr/bin/php-box build

So the prepare() function and php-cs-fixer.install can be removed.

All comments