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

build: Remove updated composer patch for firebase/php-jwt #13874

Merged

Conversation

nickvergessen
Copy link
Member

Patch was updated by #13861

Before

$ composer i
[bamarni-bin] The setting "extra.bamarni-bin.bin-links" will be set to "false" from 2.x onwards. If you wish to keep it to "true", you need to set it explicitly.
[bamarni-bin] The setting "extra.bamarni-bin.forward-command" will be set to "true" from 2.x onwards. If you wish to keep it to "false", you need to set it explicitly.
Gathering patches from patch file.
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Package operations: 3 installs, 0 updates, 0 removals
Gathering patches from patch file.
Gathering patches for dependencies. This might take a minute.
  - Installing psr/simple-cache (3.0.0): Extracting archive
  - Installing cuyz/valinor (1.14.1): Extracting archive
  - Installing firebase/php-jwt (v6.10.2): Extracting archive
  - Applying patches for firebase/php-jwt
    .patches/firebase-php-jwt-572.diff (PHP 8.4 compatibility)
   Could not apply patch! Skipping. The error was: Cannot apply patch .patches/firebase-php-jwt-572.diff

Generating optimized autoload files
[bamarni-bin] The setting "extra.bamarni-bin.bin-links" will be set to "false" from 2.x onwards. If you wish to keep it to "true", you need to set it explicitly.
[bamarni-bin] The setting "extra.bamarni-bin.forward-command" will be set to "true" from 2.x onwards. If you wish to keep it to "false", you need to set it explicitly.

After

$ composer i
[bamarni-bin] The setting "extra.bamarni-bin.bin-links" will be set to "false" from 2.x onwards. If you wish to keep it to "true", you need to set it explicitly.
[bamarni-bin] The setting "extra.bamarni-bin.forward-command" will be set to "true" from 2.x onwards. If you wish to keep it to "false", you need to set it explicitly.
Gathering patches from patch file.
No patches supplied.
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Package operations: 3 installs, 0 updates, 0 removals
Gathering patches from patch file.
No patches supplied.
Gathering patches for dependencies. This might take a minute.
  - Installing psr/simple-cache (3.0.0): Extracting archive
  - Installing cuyz/valinor (1.14.1): Extracting archive
  - Installing firebase/php-jwt (v6.10.2): Extracting archive
Generating optimized autoload files
[bamarni-bin] The setting "extra.bamarni-bin.bin-links" will be set to "false" from 2.x onwards. If you wish to keep it to "true", you need to set it explicitly.
[bamarni-bin] The setting "extra.bamarni-bin.forward-command" will be set to "true" from 2.x onwards. If you wish to keep it to "false", you need to set it explicitly.

@nickvergessen nickvergessen added 3. to review bug feature: api 🛠️ OCS API for conversations, chats and participants labels Nov 26, 2024
@nickvergessen nickvergessen added this to the 🖤 Next Major (31) milestone Nov 26, 2024
@nickvergessen nickvergessen self-assigned this Nov 26, 2024
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the bugfix/noid/remove-shipped-firebase-php-jwt-patch branch from 2d23750 to 3fd7a41 Compare November 26, 2024 14:16
@nickvergessen nickvergessen merged commit f28fbe6 into main Nov 27, 2024
46 checks passed
@nickvergessen nickvergessen deleted the bugfix/noid/remove-shipped-firebase-php-jwt-patch branch November 27, 2024 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review bug feature: api 🛠️ OCS API for conversations, chats and participants
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants