Bug #88088
closedcli binary is stored in wrong path
100%
Description
With the change in #86513, the cli binary is now automatically generated on composer install.
Unfortunately it is written to a wrong path.
Currently we distinguish 3 paths within TYPO3 in Composer mode.
- TYPO3_PATH_COMPOSER_ROOT This is the path the root composer.json of the project is located
- TYPO3_PATH_ROOT This is TYPO3 base path formerly exposed by the constant PATH_site, now accessible with Environment::getProjectPath()
- TYPO3_PATH_WEB This is the path to the publicly available document root, accessible with Environment::getPublicPath()
In default TYPO3 installations, even in composer mode the two latter are the same. However with the goal of at some point being able to have a directory with only public assets,
it is important to distinguish them in the code. It becomes important already now, when using helhum/typo3-secure-web, which exposes TYPO3_PATH_WEB to a different directory and
publishes all public assets to it.
TYPO3 currently writes the cli binary to the public folder then, which should be changed
Updated by Gerrit Code Review over 5 years ago
- Status changed from New to Under Review
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/60371
Updated by Helmut Hummel over 5 years ago
- Related to Bug #86513: typo3/sysext/core/bin/typo3 does not work in composer mode added
Updated by Helmut Hummel over 5 years ago
Side note:
Generating a file and storing it inside the source directory within composer install, will mark the package as dirty for composer (can be verified with composer status).
Rather than doing that, our code that shows this path (e.g. in scheduler) should present the path to the composer binary instead.
Updated by Gerrit Code Review over 5 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/60371
Updated by Gerrit Code Review over 5 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/60371
Updated by Gerrit Code Review over 5 years ago
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/60449
Updated by Helmut Hummel over 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 9487c117d226929dfb11d181ae428121b1b0b0b7.