Skip to content

Commit

Permalink
Remove recursive dependency to TYPO3
Browse files Browse the repository at this point in the history 
To avoid composer issues, we remove
the backwards dependency here again.
  • Loading branch information
@helhum
helhum authored Sep 6, 2017
1 parent 713666d commit c27c358
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions composer.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@
"license": ["GPL-2.0+"],

"require": {
"php" : "^7.0",
"typo3/cms-core": "^8.7 || ^9.0@dev"
"php" : "^7.0"
},
"autoload": {
"psr-4": {
Expand Down

3 comments on commit c27c358

@LeoniePhiline
Copy link

Choose a reason for hiding this comment

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

Thanks @helhum !

Could you also update the typo3-cms composer.lock?
https://github.com/TYPO3/TYPO3.CMS/blob/TYPO3_8-7/composer.lock#L1307

It still references cms-cli 1.0.1, which makes our composer fail:

typo3_src]$ composer install --no-dev
We get the following error:

Loading composer repositories with package information
Installing dependencies from lock file
Your requirements could not be resolved to an installable set of packages.

Problem 1

  • Installation request for typo3/cms-cli 1.0.1 -> satisfiable by typo3/cms-cli[1.0.1].
  • typo3/cms-cli 1.0.1 requires typo3/cms-core ^8.7 || ^9.0@dev -> no matching package found.

Potential causes:

Read https://getcomposer.org/doc/articles/troubleshooting.md for further common problems.

@helhum
Copy link
Contributor Author

@helhum helhum commented on c27c358 Sep 7, 2017

Choose a reason for hiding this comment

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

Will do. Meanwhile @LeoniePhiline you can fix this by calling composer like so: COMPOSER_ROOT_VERSION=8.7.0 composer install --no-dev

@helhum
Copy link
Contributor Author

@helhum helhum commented on c27c358 Sep 7, 2017

Choose a reason for hiding this comment

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

Please vote: https://review.typo3.org/53941

Please sign in to comment.