Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[MonologBridge] Uninstallable together with symfony/http-kernel in 3.0.6 #18745

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

Merged
merged 1 commit into from
May 11, 2016

Conversation

ymc-dabe
Copy link
Contributor

@ymc-dabe ymc-dabe commented May 11, 2016

Q A
Branch? 3.0
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

Caused by #18705, it is impossible to install v3.0.6 of symfony/monolog-bridge
together with v3.0.6 of symfony/http-kernel.

The intention of #18705 "added a conflict between Monolog bridge 2.8 and
HTTP Kernel 3.0+" was to prevent installing symfony/monolog-bridge from the
3.0 series with http-kernel from the 2.8 series of symfony. While this now
works correctly in v2.8.6, it breaks installing symfony/monolog-bridge v3.0.6
with symfony/http-kernel v3.0.6.

This PR resolves this issue.

How to reproduce

  • Create a test directory and change into it - e.g. with mkdir /tmp/reproduce-symfony-18745 && cd /tmp/reproduce-symfony-18745
  • Add the following composer.json to this test directory
{
   "require": {
     "symfony/monolog-bridge": "3.0.6",
     "symfony/http-kernel": "3.0.6"
  }
}
  • Run composer install from the test directory

Expected behavior

Composer installs symfony/monolog-bridge and symfony/http-kernel (together with their dependencies).

Actual behavior

Composer fails with the following error messages:

#:/tmp/reproduce-symfony-18745$ composer install
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for symfony/monolog-bridge 3.0.6 -> satisfiable by symfony/monolog-bridge[v3.0.6].
    - symfony/http-kernel v3.0.6 conflicts with symfony/monolog-bridge[v3.0.6].
    - Installation request for symfony/http-kernel 3.0.6 -> satisfiable by symfony/http-kernel[v3.0.6].

@ymc-dabe ymc-dabe changed the title [WIP] [MonologBridge] Uninstallable together with symfony/http-kernel in 3.0.6 [MonologBridge] Uninstallable together with symfony/http-kernel in 3.0.6 May 11, 2016
@ymc-dabe
Copy link
Contributor Author

Note that I'm actually not sure if it is right to prevent installing symfony/monolog-bridge v3.0.6 together with symfony/http-kernel v2.8.6, since PR #18705 only mentions a problem the other way around (monolog-bridge v2.8.x conflicts with http-kernel v3.0+).
If it is okay to install a 3.0.x monolog-bridge with an older 2.8.x http-kernel, the conflict in composer.json should be removed at all in branch 3.0. In that case, please let me know, so I can update this PR accordingly.

@stof
Copy link
Member

stof commented May 11, 2016

yeah, the conflict rule should be removed entirely in the 3.0 branch, which was forgotten when merging 2.8 into 3.0

Caused by symfony#18705, it is impossible to install v3.0.6 of symfony/monolog-bridge
together with v3.0.6 of symfony/http-kernel.

The intention of symfony#18705 "added a conflict between Monolog bridge 2.8 and
HTTP Kernel 3.0+" was to prevent installing symfony/monolog-bridge from the
3.0 series with http-kernel from the 2.8 series of symfony. While this now
works correctly in v2.8.6, it breaks installing symfony/monolog-bridge v3.0.6
with symfony/http-kernel v3.0.6.

This commit resolves this issue.
@ymc-dabe ymc-dabe force-pushed the fix-broken-dependecy-in-monolog-bridge branch from 35fd3f2 to 72c44c2 Compare May 11, 2016 08:46
@ymc-dabe
Copy link
Contributor Author

Thanks for the feedback @stof. I updated this PR to remove the conflict with symfony/http-kernel in src/Symfony/Bridge/Monolog/composer.json.

@Tobion
Copy link
Contributor

Tobion commented May 11, 2016

👍
Status: Reviewed

@xabbuh
Copy link
Member

xabbuh commented May 11, 2016

👍

@fabpot
Copy link
Member

fabpot commented May 11, 2016

Thank you @ymc-dabe.

@fabpot fabpot merged commit 72c44c2 into symfony:3.0 May 11, 2016
fabpot added a commit that referenced this pull request May 11, 2016
…ernel in 3.0.6 (ymc-dabe)

This PR was merged into the 3.0 branch.

Discussion
----------

[MonologBridge] Uninstallable together with symfony/http-kernel in 3.0.6

| Q             | A
| ------------- | ---
| Branch?       | 3.0
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

Caused by #18705, it is impossible to install v3.0.6 of symfony/monolog-bridge
together with v3.0.6 of symfony/http-kernel.

The intention of #18705 "added a conflict between Monolog bridge 2.8 and
HTTP Kernel 3.0+" was to prevent installing symfony/monolog-bridge from the
3.0 series with http-kernel from the 2.8 series of symfony. While this now
works correctly in v2.8.6, it breaks installing symfony/monolog-bridge v3.0.6
with symfony/http-kernel v3.0.6.

This PR resolves this issue.

# How to reproduce
- Create a test directory and change into it - e.g. with `mkdir /tmp/reproduce-symfony-18745 && cd /tmp/reproduce-symfony-18745`
- Add the following composer.json to this test directory
```
{
   "require": {
     "symfony/monolog-bridge": "3.0.6",
     "symfony/http-kernel": "3.0.6"
  }
}
```
- Run `composer install` from the test directory

## Expected behavior
Composer installs symfony/monolog-bridge and symfony/http-kernel (together with their dependencies).

## Actual behavior
Composer fails with the following error messages:
```
#:/tmp/reproduce-symfony-18745$ composer install
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for symfony/monolog-bridge 3.0.6 -> satisfiable by symfony/monolog-bridge[v3.0.6].
    - symfony/http-kernel v3.0.6 conflicts with symfony/monolog-bridge[v3.0.6].
    - Installation request for symfony/http-kernel 3.0.6 -> satisfiable by symfony/http-kernel[v3.0.6].
```

Commits
-------

72c44c2 [MonologBridge] Uninstallable together with symfony/http-kernel in 3.0.6
@fabpot fabpot mentioned this pull request May 13, 2016
@derrabus
Copy link
Member

So, the workaround for Silex projects (and other projects using the symfony/monolog-bridge package directly) would be to mark the 3.0.6 release of that package as conflicting. There are no other changes between 3.0.5 and 3.0.6 anyway and composer will otherwise downgrade HttpKernel to 2.8.

"conflict": {
    "symfony/monolog-bridge": "3.0.6"
}

@fabpot fabpot mentioned this pull request Jun 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.