Skip to content
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

php install script help only available after installation #779

Closed
nyov opened this issue Dec 4, 2014 · 5 comments
Closed

php install script help only available after installation #779

nyov opened this issue Dec 4, 2014 · 5 comments

Comments

@nyov
Copy link

nyov commented Dec 4, 2014

The README (Running the Magento 2 installer) says

Before you begin, you can run the following commands to find values for some required options:
php -f index.php help languages [...]

These actually only produce results after an installation, aside from failing without any helpful error messages (Reason for failure: Invalid arguments or no arguments provided), which almost had me thinking the installer was broken there for a while.

What made matters worse for me, the script will always "silently" fail, if run from inside the setup directory, while the README explicitely states

You must install Magento from its setup subdirectory.

The above mentioned README lines should really read something like php -f setup/index.php help languages, and fix the notice to say that the script is supposed to be run from the magento root directory.
Unless this is a temporary hiccup and supposed to work differently later.
(version tested 0.1.0-alpha106 / e05a60c)

Thanks

@alankent
Copy link

alankent commented Dec 4, 2014

Hi. Why do you believe the script always silently fails if run from within the setup directly? If you have a look at https://github.com/alankent/docker-magento2-demo-apache/tree/master/scripts you will see the scripts I use to create a Docker image. This does a cd into the setup directory, and after I created the image the installation worked. So I was wondering if you could provide a bit more detail about what behavior you observed. Thanks!

Thanks for the other feedback on the error message and sequence of commands in documentation that might not be quite right.

@nyov
Copy link
Author

nyov commented Dec 4, 2014

Well, this is typical. Trying to retrace the steps... Now it works.
I can only guess this must have been a permission problem previously where I set them too strict/wrong and the script didn't find some includes. Can't figure it out anymore.

And the other issue actually seems to be a mis-spelling thing I tripped over: php -f index.php help languages|timezones|currencies in the README should be

php -f index.php help language
php -f index.php help currency
php -f index.php help timezone

That's the syntax the tool's help actually mentions, so I must have copy-pasted it from the readme :/

Very sorry, all my fault!

@nyov
Copy link
Author

nyov commented Dec 4, 2014

I'll close this, as fixing the readme is probably not quite an important priority at this time.

@nyov nyov closed this as completed Dec 4, 2014
@alankent
Copy link

alankent commented Dec 4, 2014

Thanks. We will review the README as well. It is important (to me anyway). It should also be easy! ;-)

@ghost
Copy link

ghost commented Jan 4, 2015

mmansoor-magento pushed a commit that referenced this issue Jan 27, 2017
…ge-gallery-look-ups

MAGETWO-63062: Product image gallery look-ups use incorrect ID
fe-lix- pushed a commit to fe-lix-/magento2 that referenced this issue Apr 6, 2018
MSI-690: Unskip and Fix Integration Test for Quantity Increment Messa…
magento-engcom-team added a commit that referenced this issue Jul 30, 2019
…e Products #779

 - Merge Pull Request magento/graphql-ce#779 from vovsky/graphql-ce:702-Test-coverage-Customer-Downloadable-Products
 - Merged commits:
   1. ea0a500
   2. f3c387f
   3. 153a236
   4. 70f40dc
   5. b23231c
   6. 373b56b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants