Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecated error when using make build #41

Closed
llaville opened this issue Jan 25, 2024 · 3 comments
Closed

Deprecated error when using make build #41

llaville opened this issue Jan 25, 2024 · 3 comments

Comments

@llaville
Copy link

Hello

When running make build command, I got a deprecated error when using BOX v3 to build the PHAR version on a PHP 8.2 platform.

Please consider to upgrade to BOX v4 !

>>>> >>>> >>>> >>>> >>>> >>>>  Install/Update all 3rd party dependencies
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Nothing to install, update or remove
Generating optimized autoload files
64 packages you are using are looking for funding.
Use the `composer fund` command to find out more!
make[1]: Entering directory '/shared/backups/forks/CI-Report-Converter'
File ./vendor/bin/box.phar not found. Downloading.
Box version 3.16.0@adb282a 2022-02-13 23:54:13 UTC

Box Requirements Checker
========================

> Using PHP 8.2.15
> PHP is using the following php.ini file:
  WARNING: No configuration file (php.ini) used by PHP!

> Checking Box requirements:
  ✔ The application requires the version "^7.4 || ^8.0" or greater.
  ✔ The application requires the extension "zlib".
  ✔ The application requires the extension "phar".
  ✔ The package "nikic/php-parser" requires the extension "tokenizer".
  ✔ The package "phpdocumentor/reflection-docblock" requires the extension "filter".


 [OK] Your system is ready to run the application.



In DumpCompletionCommand.php line 48:

  [ErrorException]
  Using ${var} in strings is deprecated, use {$var} instead


Exception trace:
  at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Command/DumpCompletionCommand.php:48
 KevinGH\Box\{closure}() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/.box/vendor/composer/ClassLoader.php:571
 include() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/.box/vendor/composer/ClassLoader.php:571
 Composer\Autoload\includeFile() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/.box/vendor/composer/ClassLoader.php:428
 Composer\Autoload\ClassLoader->loadClass() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Application.php:1092
 Symfony\Component\Console\Application->getDefaultCommands() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/src/Console/Application.php:83
 KevinGH\Box\Console\Application->getDefaultCommands() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Application.php:1272
 Symfony\Component\Console\Application->init() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Application.php:662
 Symfony\Component\Console\Application->find() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Application.php:259
 Symfony\Component\Console\Application->doRun() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Application.php:171
 Symfony\Component\Console\Application->run() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/bin/box:52
 require() at /shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar:18


Box Requirements Checker
========================

> Using PHP 8.2.15
> PHP is using the following php.ini file:
  WARNING: No configuration file (php.ini) used by PHP!

> Checking Box requirements:
  .....


 [OK] Your system is ready to run the application.



In DumpCompletionCommand.php line 48:

  [ErrorException]
  Using ${var} in strings is deprecated, use {$var} instead


Exception trace:
  at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Command/DumpCompletionCommand.php:48
 KevinGH\Box\{closure}() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/.box/vendor/composer/ClassLoader.php:571
 include() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/.box/vendor/composer/ClassLoader.php:571
 Composer\Autoload\includeFile() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/.box/vendor/composer/ClassLoader.php:428
 Composer\Autoload\ClassLoader->loadClass() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Application.php:1092
 Symfony\Component\Console\Application->getDefaultCommands() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/src/Console/Application.php:83
 KevinGH\Box\Console\Application->getDefaultCommands() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Application.php:1272
 Symfony\Component\Console\Application->init() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Application.php:662
 Symfony\Component\Console\Application->find() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Application.php:259
 Symfony\Component\Console\Application->doRun() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/vendor/symfony/console/Application.php:171
 Symfony\Component\Console\Application->run() at phar:///shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar/bin/box:52
 require() at /shared/backups/forks/CI-Report-Converter/vendor/bin/box.phar:18

make[1]: *** [/shared/backups/forks/CI-Report-Converter/vendor/jbzoo/codestyle/src/Makefiles/02_project.Makefile:38: build-phar] Error 1
make[1]: Leaving directory '/shared/backups/forks/CI-Report-Converter'
make: *** [Makefile:22: build] Error 2
@llaville
Copy link
Author

If you build the PHAR version of your project with BOX v4, there are no regression !


    ____
   / __ )____  _  __
  / __  / __ \| |/_/
 / /_/ / /_/ />  <
/_____/\____/_/|_|


Box version 4.6.0@95b0d8e 2023-12-09 11:06:49 UTC

 // Loading the configuration file "/shared/backups/forks/CI-Report-Converter/box.json.dist".

🔨  Building the PHAR "/shared/backups/forks/CI-Report-Converter/build/ci-report-converter.phar"

? Checking Composer compatibility
    > Supported version detected
? Setting replacement values
  + @git-version@: 4a0888e
? No compactor to register
? Adding main file: /shared/backups/forks/CI-Report-Converter/ci-report-converter
? Adding requirements checker
? Adding binary files
    > No file found
? Auto-discover files? No
? Exclude dev files? Yes
? Adding files
    > 587 file(s)
? Generating new stub
  - Using shebang line: #!/usr/bin/env php
  - Using banner:
    > JBZoo Toolbox - CI-Report-Converter
    >
    > This file is part of the JBZoo Toolbox project.
    > For the full copyright and license information, please view the LICENSE
    > file that was distributed with this source code.
    >
    > @package    CI-Report-Converter
    > @license    MIT
    > @copyright  Copyright (C) JBZoo.com, All rights reserved.
    > @link       https://github.com/JBZoo/CI-Report-Converter
? Dumping the Composer autoloader
? Removing the Composer dump artefacts
? No compression
? Setting file permissions to 0755
* Done.

No recommendation found.
No warning found.

 // PHAR: 620 files (35.09MB)
 // You can inspect the generated PHAR with the "info" command.

 // Memory usage: 59.20MB (peak: 59.41MB), time: 3secs

Test phar command :

./build/ci-report-converter.phar --version
JBZoo/CI-Report-Converter 4a0888e

@SmetDenis
Copy link
Member

Thank you @llaville

Yeah, I think it's time to update the dependencies.

I was held hostage to this error a while back - box-project/box#1004

It was fixed a few months ago.
Right now I don't feel like anything should be blocking me.

@SmetDenis
Copy link
Member

Hello.

I updated the Box to 4.5.1 because of it's the latest version for PHP 8.1+

Related PR #47
See version https://github.com/JBZoo/CI-Report-Converter/releases/tag/7.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants