Skip to content

Conversation

@michag86
Copy link
Contributor

Fix fox #291

I have tested PHP 7 on CentOS 7.
The path informations for PHP 5.6 are copied from the rpms from
http://mirror.centos.org/centos/6/sclo/x86_64/rh/rh-php56/
http://mirror.centos.org/centos/7/sclo/x86_64/rh/rh-php56/

Can somebody check this?
@tflidd @lamixer

@MorrisJobke
Copy link
Member

@oparoz Weren't you interested in this lately?

@oparoz
Copy link
Member

oparoz commented Mar 15, 2017

Thanks for the heads up @MorrisJobke :)

@tflidd
Copy link
Contributor

tflidd commented Mar 15, 2017

I have no CentOS or RH setup. I quickly went through and didn't see obvious errors. Why do you mention ldap on the PHP 7.0 version and not for PHP 5.6?

@oparoz
Copy link
Member

oparoz commented Mar 21, 2017

I've tested the instructions and the biggest problem is that it's aimed at people already running PHP and Apache.
Those are upgrade instructions and not installation instructions.

2 ways forward:

  1. Rename to upgrade and explain how to get everything setup first so that people can later upgrade using this guide
  2. Remove the doc since it's outdated an pretty useless for people wanting to configure RHEL/CentOS 7 for Nextcloud

@michag86
Copy link
Contributor Author

michag86 commented Mar 21, 2017

@oparoz What do you think about changing "Disable loading the old PHP 5.3 Apache module" to "Disable loading the old PHP 5.3 Apache module (only required if php is already installed)"?

I think removing is no option because there are many ways to upgrade php on CentOS/RHEL. It's helpful to know what the Nextcloud experts recommend.

@MorrisJobke
Copy link
Member

@oparoz @karlitschek Would be nice to get some testing on this. At least we say to support this and should be happy about this nice writeup 😉

@tflidd
Copy link
Contributor

tflidd commented Feb 18, 2018

This topic is stuck, would be a pity to lose this contribution. I asked on the forum for CentOS users to help us reviewing this issue:
https://help.nextcloud.com/t/php-7-on-centos-install-instructions-are-updated-review-needed/27732

@michag86
Copy link
Contributor Author

I've tested and used 5.6.
I can also add the instructions for PHP 7.1.

@KB7777
Copy link

KB7777 commented Feb 22, 2018

Wouldn't be better to use remi repo for PHP7.x for CentOS7?
Installation is very easy -- https://help.nextcloud.com/t/what-is-the-supported-php7-installation-method-for-centos-7/9832/25?u=therion7777

Works fine at my NC instance from NC11-NC13 versions for few k users.

@oparoz
Copy link
Member

oparoz commented Feb 22, 2018

I agree that using Remi's repo would be the more universal solution.

@michag86
Copy link
Contributor Author

https://wiki.centos.org/de/AdditionalResources/Repositories/
Software Collections (SCL) is listes as Additional CentOS Provided Repositories and Remi is listed as Community Approved Repositories, so I decided to use SCL.

The manual for updating to newer php version for centos/rhel was also referencing to scl.
I think SCL is the "supported" source for newer php versions on rhel/centos.

@tflidd
Copy link
Contributor

tflidd commented Mar 21, 2018

It's now over a year. It's really missing in the documentation and there were no principal objections. And if it is really a matter of personal preference, we could perhaps link to a place with some comparison of the different ways to handle newer php versions e.g. https://www.hogarthuk.com/?q=node/15 (found on the page @michag86 linked)

@MorrisJobke MorrisJobke merged commit 4710f47 into master Mar 21, 2018
@MorrisJobke MorrisJobke deleted the redhat-centos-new-php branch March 21, 2018 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants