Arch Linux User Repository

Hello, I am sharing little patch, which makes final phar always executing system installed php. In case one is using something like nix-shell to get older php versions running dependency on environment breaks cs-fixer...

diff --git a/PKGBUILD b/PKGBUILD
index 035ce1f..34ae0e9 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -14,6 +14,11 @@ makedepends=("php-humbug-box-bin" "composer" "git")
 source=("${_pkgname}"::"git+https://github.com/FriendsOfPHP/PHP-CS-Fixer#tag=v${pkgver}")
 md5sums=('SKIP')

+prepare() {
+    cd "${srcdir}/${_pkgname}"
+    sed -i '1s|.*|#!/usr/bin/php|' php-cs-fixer
+}
+
 build() {
   cd "${srcdir}/${_pkgname}"
   php /usr/bin/composer install --prefer-dist --no-dev

There is a problem w/ update and install too.

❯ yay -S php-cs-fixer
 -> status 200: Too many package results.
status 200: Too many package results.
status 200: Too many package results.
status 200: Too many package results.
status 200: Too many package results.
status 200: Too many package results.

Due to the latest php81 update I needed to modify the PKGBUILD such that version 3.5.0 is installed (currently latest). Otherwise php-cs-fixer does not work without an additional environment variable, and recognizes some incorrect formats, which are actually correct.

I do not experience any bugs, so I think it should be save to update. At least it works for me ¯_(ツ)_/¯

@Krisque: if php-humbug-box-bin is failing to build then your comment belongs on that package rather than this one ;)

It doesn't work for me: "box.phar ... Failed" "LICENSE ... Passed" "Failed to compile php-humbug-box-bin"

If this is built in clean chroot it does not work.

To fix the issue add dependency on php-intl and run box in build function this way: php -n -d phar.readonly=Off -d extension=intl /usr/bin/box compile

-n is necessary for people who have xdebug enabled and intl disabled. If it is not present box restart itself wih modified php.ini to disable xdebug but intl disabled again.

You have to increase the PHP version on depends array. Now it should be:

depends=("php>=7.2")

Any chance of an update to 2.18 please? 2.17 doesn't have PHP 8 support, which is now stable and used by the php package.

The php-composer dependency doesn't exist any more. Replacing with composer seems to work.

diff --git i/PKGBUILD w/PKGBUILD
index d332769..742da57 100644
--- i/PKGBUILD
+++ w/PKGBUILD
@@ -3,14 +3,14 @@

 _pkgname=php-cs-fixer
 pkgname=${_pkgname}
-pkgver=2.16.4
+pkgver=2.16.6
 pkgrel=1
 pkgdesc="Analyzes some PHP source code and tries to fix coding standards issues (PSR-1 and PSR-2 compatible)."
 url="http://cs.sensiolabs.org/"
 license=("MIT")
 arch=("any")
 depends=("php>=5.6" "php>=7")
-makedepends=("php-humbug-box-bin" "php-composer" "git")
+makedepends=("php-humbug-box-bin" "composer" "git")
 source=("${_pkgname}"::"git+https://github.com/FriendsOfPHP/PHP-CS-Fixer#tag=v${pkgver}")
 md5sums=('SKIP')

This is not necessarily a package bug, as it may well be a pikaur bug, but does anybody know why, when I build this package in pikaur, it tries to use sudo right after the "Updating dependencies" step, as follows?:

==> Validating source files with md5sums...
    php-cs-fixer ... Skipped
==> Extracting sources...
  -> Creating working copy of php-cs-fixer git repo...
Cloning into 'php-cs-fixer'...
done.
Switched to a new branch 'makepkg'
==> Starting build()...
Loading composer repositories with package information
Updating dependencies
[sudo] password for ...:

This does not happen if I freshly clone it and use makepkg.

Edit: It also does not happen in "yay". I guess I'll figure it out and give pikaur a report.

Edit: Hmm. Now when I try it again in pikaur, I can't reproduce it. Weird. This happens to me every time this package needs to update.

Thanks for the diff @danger89

Yes, Finally we have v2.14.0 Sunrise, supporting 7.3 now officially. Diff for pkgbuild:

diff --git a/PKGBUILD b/PKGBUILD
index 05acdbe..042304d 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -3,13 +3,13 @@

 _pkgname=php-cs-fixer
 pkgname=${_pkgname}
-pkgver=2.13.1
+pkgver=2.14.0
 pkgrel=1
 pkgdesc="Analyzes some PHP source code and tries to fix coding standards issues (PSR-1 and PSR-2 compatible)."
 url="http://cs.sensiolabs.org/"
 license=("MIT")
 arch=("any")
-depends=("php>=5.6" "php<7.3")
+depends=("php>=5.6" "php>=7")
 makedepends=("php-box" "php-composer" "git")
 source=("${_pkgname}"::"git+https://github.com/FriendsOfPHP/PHP-CS-Fixer#tag=v${pkgver}")
 md5sums=('SKIP')

Thanks for your quick update

@macfly that went in by mistake, should have been <7.3 as seen in the composer.json file: https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/v2.10.3/composer.json Thanks for reporting!

Why do you require php under version 7.2 ? I've just remove it from the PKGBUILD and it seems ok.

Also on sensiolabs site they say: "PHP needs to be a minimum version of PHP 5.6.0." But not incompatibility with php version 7.2