@dgrammatiko Shouldn't the testing instructions tell to use the public folder option --public-folder? As far as I understood the issue, the CLI installation broke only with using that option. Or was that wrong and it's broken also when being called without that opzion?

Shouldn't the testing instructions tell to use the public folder option --public-folder?

Good call, fixed

Shouldn't the testing instructions tell to use the public folder option --public-folder?

Good call, fixed

@dgrammatiko Where? I see no change in the description. Now it's there.

@richard67 fwiw php joomla.php install without any options kicks an interactive set of questions, so the last question is about the public folder path. Leaving it empty skips the ceremony for the public folder...

@dgrammatiko
This is what I see on my local machine at the end of cd installation;php joomla.php install before applying the PR.
I do not have a DB error.

Relative or absolute path to the public folder []:
 > 

OK
Validating DB connection...OK
Creating and populating the database...OK
Writing configuration.php and additional setup ...OK

                                                                                                                        
 [OK] Joomla has been installed                                                                                         
                                                                                                                        

hans2103@192 installation % 

Thank for the PR! Confirm is work now! ✅

Hope this includes in 5.0 at next Tuesday. I will use this + Ansible. Make build server easy for me.

this pr is scheduled for 5.0.1 thanks @rabidgrowth for the test

Clarification please: I unpacked the J5rc2 package and started from its installation folder. First try I answered all of the questions and entered the root folder:

/Users/ceford/Sites/j5rc2test

That ended in an error

OK
Validating DB connection...OK
Creating and populating the database...OK
Writing configuration.php and additional setup ...OK
Creating the public folder...
In PublicFolderGeneratorHelper.php line 112:
                                                                
  Unable to create the given folder, index.php already exists.                                                        

I did it again and left the public folder at the default [] and the installation went to completion with no errors. This is before applying the patch.

 Relative or absolute path to the public folder []:
 > 

OK
Validating DB connection...OK
Creating and populating the database...OK
Writing configuration.php and additional setup ...OK
                                                                                                                        
 [OK] Joomla has been installed

It has and works fine. So what error should I look for in the installed site?

@ceford You put all joomla files inside the folder /Users/ceford/Sites/j5rc2test, yes? Can not use same folder /Users/ceford/Sites/j5rc2test for public. Must use different folder. Example: /Users/ceford/Sites/rabidgrowth. After install ends the site is servered from /Users/ceford/Sites/rabidgrowth.

See my issue #42114 . I used folder outside from /var/www for joomla files, then folder inside /var/www for public.

Hope this is help for you!

@hans2103 @ceford the CLI installation is broken only if the user provided a path for the public folder (and maybe the path needs to be outside of the root path) otherwise nothing is broken

I see now, the cli installer is used for creating a site within a site or multiple sites on the same server. That did not occur to me. Thanks for the clarification.

I have tested this item ✅ successfully on ec5c8d2

To test I unpacked the download in ~/tmp/joomla5 and followed the instructions. Without the patch I got the Database error and with the patch I got a working site in ~/Sites/j5testcli (I keep all my test sites in ~/Sites). I had not twigged this is a way to keep most of the php code outside the public folder.

Thank for the PR! Confirm is work now! ✅

@rabidgrowth Could you mark your test result right? It's not enough just to add a comment with a green check mark. It needs to submit your test result in the issue tracker. For doing this, go to https://issues.joomla.org/tracker/joomla-cms/42116 , then use the blue "Test this" at the top left corner, then select your test result and finally submit. Thanks in advance.

I have tested this item ✅ successfully on ec5c8d2

Sorry, didn't know. I now marked test inside tracker.

RTC

@ceford @rabidgrowth Thanks for testing.

thanks, there is another problem I will create a PR for

@HLeithner #42120 or another one?

I talk about #42168 no fix for #42120 yet

Nice but I think the error msg needs to be translatable