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

BOX info command did not display built with box version #1236

Open
llaville opened this issue Dec 1, 2023 · 0 comments
Open

BOX info command did not display built with box version #1236

llaville opened this issue Dec 1, 2023 · 0 comments
Labels

Comments

@llaville
Copy link
Contributor

llaville commented Dec 1, 2023

Bug report

Question Answer
Box version 4.5.1@1e10a1e
PHP version 8.3.0
Platform with version Ubuntu 22.04 LTS on WSL2 with Docker
Github Repo - https://github.com/overtrue/phplint main branch commit overtrue/phplint@d7aa2cc

Don't care about metadata setting, it will be removed in future commits. This is the reason why I've fixed latest PHPLint repo commit as base of this report !

box.json.dist
{
 "chmod": "0700",
 "banner": [
   "This file is part of the overtrue/phplint package",
   "",
   "(c) overtrue",
   "",
   "This source file is subject to the MIT license that is bundled",
   "with this source code in the file LICENSE."
 ],
 "compactors": [
   "KevinGH\\Box\\Compactor\\PhpScoper"
 ],
 "metadata": "Bartlett\\BoxManifest\\Composer\\ManifestFactory::toHighlight"
}

Compile command :

Output
$ box compile
> output
   ____
  / __ )____  _  __
 / __  / __ \| |/_/
/ /_/ / /_/ />  <
/_____/\____/_/|_|


Box version 4.5.1@1e10a1e 2023-11-04 17:57:48 UTC

// Loading the configuration file "/shared/backups/github/phplint/box.json".

🔨  Building the PHAR "/shared/backups/github/phplint/bin/phplint.phar"

? Checking Composer compatibility
   > Supported version detected
? Registering compactors
 + KevinGH\Box\Compactor\PhpScoper
? Adding main file: /shared/backups/github/phplint/bin/phplint
? Adding requirements checker
? Adding binary files
   > No file found
? Auto-discover files? Yes
? Exclude dev files? Yes
? Adding files
   > 416 file(s)
? Generating new stub
 - Using shebang line: #!/usr/bin/env php
 - Using banner:
   > This file is part of the overtrue/phplint package
   >
   > (c) overtrue
   >
   > This source file is subject to the MIT license that is bundled
   > with this source code in the file LICENSE.
? Setting metadata
 - Bartlett\BoxManifest\Composer\ManifestFactory::toHighlight
? Dumping the Composer autoloader
? Removing the Composer dump artefacts
? No compression
? Setting file permissions to 0700
* Done.

No recommendation found.
No warning found.

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

// Memory usage: 44.28MB (peak: 77.56MB), time: 13secs

Info command :

Output
$ box compile
> output
API Version: 1.1.0

Archive Compression: None
Files Compression: None

Signature: SHA-1
Signature Hash: 0809DA48FD90341C05CD1967173431D920CE1FD5

Metadata:
'Bartlett\\BoxManifest\\Composer\\ManifestFactory::toHighlight'

Timestamp: 1701410470 (2023-12-01T06:01:10+00:00)

RequirementChecker:
 Required:
 - PHP ^8.0 (root)
 - ext-json (root)
 Conflict:
 - ext-psr (symfony/service-contracts)

Contents: 459 files (2.36MB)

// Use the --list|-l option to list the content of the PHAR.

BOX info command is supposed to display BOX built version => see https://github.com/box-project/box/blob/4.5.1/src/Console/PharInfoRenderer.php#L96-L111

i.e: I got Built with Box: 3.16.0@adb282a when I look for Composer Unused 0.8.11

@theofidry theofidry added the bug label Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants