Task #68700

Improve composer integration

Added by Alexander Stehlik almost 5 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
composer
Target version:
-
Start date:
2015-08-03
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Since the composer handling in TYPO3 6.2 is not perfect and you quite easy run into troubles when it comes to dependency handling, I have a suggestion to improve the current situation.

My suggestion is to ignore the Composer dependencies in the TYPO3 Extension Manager completely and only respect the ext_emconf.php files.

This would makes things a lot easier:

In the composer.json files you could use real Composer Package names for dependency handling (e.g. typo3-ter/my-dependency or even my-vendor/mypackage for packages from Packagist).

In the ext_emconf.php files you use Extension keys.

No need for "replace" configuration in the composer.json files any more. No more hassle with underscore / dash naming.

I'll push my proposed solution to Gerrit.


Related issues

Duplicated by TYPO3 Core - Bug #63545: Cannot download extension because buildDependencyGraphForPackages failed Closed 2014-12-03

Associated revisions

Revision d07e333a (diff)
Added by Helmut Hummel over 4 years ago

[TASK] Ignore composer.json dependencies for TYPO3 Extensions

This resolves problems with composer dependencies and Extension name /
Composer package name differences.

Resolves: #68700
Releases: master
Change-Id: I07f57f24ef7e9337e7ef8f6ee69538c038f2c0ff
Reviewed-on: http://review.typo3.org/42615
Reviewed-by: Alexander Opitz <>
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>
Reviewed-by: Markus Sommer <>
Tested-by: Markus Sommer <>
Reviewed-by: Helmut Hummel <>
Tested-by: Helmut Hummel <>

History

#1 Updated by Gerrit Code Review almost 5 years ago

  • Status changed from New to Under Review

Patch set 2 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42214

#2 Updated by Alexander Stehlik almost 5 years ago

@Helmut: I assigned this issue to you as discussed on Slack some days ago. Please ping me there if you would like to discuss my suggestion.

#3 Updated by Helmut Hummel almost 5 years ago

  • Description updated (diff)
  • Category changed from composer to 1594
  • Status changed from Under Review to Accepted
  • TYPO3 Version changed from 6.2 to 7

We will tackle this for master, but not for 6.2

#4 Updated by Helmut Hummel almost 5 years ago

  • Subject changed from Improve composer integration for TYPO3 6.2 to Improve composer integration
  • Description updated (diff)

#5 Updated by Gerrit Code Review almost 5 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42615

#6 Updated by Gerrit Code Review almost 5 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42615

#7 Updated by Gerrit Code Review over 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42615

#8 Updated by Gerrit Code Review over 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42615

#9 Updated by Gerrit Code Review over 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42615

#10 Updated by Helmut Hummel over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#11 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF