Skip to content

Update composer.json files #6632

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

Closed
wants to merge 1 commit into from

Conversation

stloyd
Copy link
Contributor

@stloyd stloyd commented Jan 9, 2013

No description provided.

@fabpot
Copy link
Member

fabpot commented Jan 9, 2013

When will Doctrine 2.4 be released? If not in the next couple of weeks, this PR cannot be merged as we need to ship Symfony 2.2 with a stable version of Doctrine.

@stof
Copy link
Member

stof commented Jan 9, 2013

@fabpot This allows to use Doctrine 2.4. It does not require to use 2.4. Actually, as Doctrine releases are BC, I would even vote for using ~2.2 as requirement

@fabpot
Copy link
Member

fabpot commented Jan 10, 2013

If the Doctrine is fine with ~2.2, let's do that. @beberlei Can you confirm?

@stloyd
Copy link
Contributor Author

stloyd commented Jan 10, 2013

As Doctrine latest release is 2.3.2 maybe for Symfony 2.2 we should increase to ~2.3 not ~2.2 ?

@stof
Copy link
Member

stof commented Jan 10, 2013

@stloyd Why should we force using 2.3 when we don't require any of the 2.3 features ? Composer will use the most recent package matching the constraint anyway, so 2.2 will not be used unless someone forbids 2.3 in another package.

@@ -19,7 +19,7 @@
"php": ">=5.3.3"
},
"suggest": {
"ext-intl": ">=5.3.3"
"ext-intl": "*"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i.e. because such version not even exists ? =)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest is not taking a version constraint anyway, ideally it should be a description as to why you need the package, then specifying a version might be good to hint at what is compatible (but for extensions it's kind of pointless)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, let's just use an empty string then.

- to allow versions ~2.2 (>=2.2,<3.0) of Doctrine DBAL, ORM & Common
- fixed Propel1 versions difference between main and bridge files
- fixed Twig versions difference between main and bridge files
- to allow versions ~1.11 (>=1.11,<2.0) of Twig
- fixed Locale ext-intl version to accept all, not non-existing version
@stloyd
Copy link
Contributor Author

stloyd commented Jan 29, 2013

@fabpot ping

fabpot added a commit that referenced this pull request Feb 1, 2013
This PR was submitted for the master branch but it was merged into the 2.2 branch instead (closes #6632).

Commits
-------

a1c0ff5 Update `composer.json` files: - to allow versions ~2.2 (>=2.2,<3.0) of Doctrine DBAL, ORM & Common - fixed Propel1 versions difference between main and bridge files - fixed Twig versions difference between main and bridge files - to allow versions ~1.11 (>=1.11,<2.0) of Twig - fixed Locale ext-intl version to accept all, not non-existing version

Discussion
----------

Update `composer.json` files

---------------------------------------------------------------------------

by fabpot at 2013-01-09T17:55:54Z

When will Doctrine 2.4 be released? If not in the next couple of weeks, this PR cannot be merged as we need to ship Symfony 2.2 with a stable version of Doctrine.

---------------------------------------------------------------------------

by stof at 2013-01-09T18:13:59Z

@fabpot This allows to use Doctrine 2.4. It does not require to use 2.4. Actually, as Doctrine releases are BC, I would even vote for using ``~2.2`` as requirement

---------------------------------------------------------------------------

by fabpot at 2013-01-10T08:18:52Z

If the Doctrine is fine with `~2.2`, let's do that. @beberlei Can you confirm?

---------------------------------------------------------------------------

by stloyd at 2013-01-10T08:30:19Z

As Doctrine latest release is `2.3.2` maybe for Symfony `2.2` we should increase to `~2.3` not `~2.2` ?

---------------------------------------------------------------------------

by stof at 2013-01-10T09:15:00Z

@stloyd Why should we force using 2.3 when we don't require any of the 2.3 features ? Composer will use the most recent package matching the constraint anyway, so 2.2 will not be used unless someone forbids 2.3 in another package.

---------------------------------------------------------------------------

by stloyd at 2013-01-29T19:24:09Z

@fabpot ping
@fabpot fabpot closed this Feb 1, 2013
@stloyd stloyd deleted the feature/doctrine_versions branch February 1, 2013 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants