Package Details: php-cs-fixer 2.16.1-1

Git Clone URL: https://aur.archlinux.org/php-cs-fixer.git (read-only, click to copy)
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: sanduhrs
Votes: 24
Popularity: 0.000056
First Submitted: 2012-08-10 16:53
Last Updated: 2020-01-10 15:29

Dependencies (5)

Required by (0)

Sources (1)

Latest Comments

« First ‹ Previous 1 2 3 4 Next › Last »

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.

bgaleotti commented on 2015-08-15 22:43

@sanduhrs merged. I also added you as con-maintainer.

sanduhrs commented on 2015-08-14 21:49

@bgaleotti of course you are right.
I moved the check to a prepare method, how about that?
https://github.com/sanduhrs/arch-aur-php-cs-fixer

bgaleotti commented on 2015-08-14 18:36

@sanduhrs Added the removal of git-commit.

Isn't the warning at post_install too late? I mean, if you don't have configured:

extension=phar.so
phar.readonly = Off

it would fail at build becouse php-box won't have the build command enabled.

Stephen304 commented on 2015-08-14 01:04

I forgot to note but to enable the "build" command, the aforementioned edits need to go in the php.ini that resides at /etc/php/php.ini
(mine reset so I'm noting this so I can fix it later if I need to)

extension=phar.so
phar.readonly = Off

sanduhrs commented on 2015-08-12 17:14

@bgaleotti package is broken after you removed lin 19 from the pgkbuild.
See https://aur.archlinux.org/cgit/aur.git/diff/?h=php-cs-fixer&context=3&ignorews=0&dt=1#n19

And please have a look at https://github.com/sanduhrs/arch-aur-php-cs-fixer
which contains a working pkgbuild including a message about the required settings.

regards.