Skip to content

Conversation

@jahway603
Copy link

@jahway603 jahway603 commented Nov 21, 2022

I needed to add php for instructions about zip install for it to create the /etc/php.ini
OS: RockyLinux 8.7

Author checklist (Completed by original Author)

  • Good fit for the Rocky Linux project? Title and Author Metatags inserted ?
  • If applicable, steps and instructions have been tested to work
  • Initial self-review to fix basic typos and grammar completed

Rocky Documentation checklist (Completed by Rocky team)

  • 1st Pass (Document is good fit for project and author checklist completed)
  • 2nd Pass (Technical Review - check for technical correctness)
  • 3rd Pass (Detailed Editorial Review and Peer Review)
  • Final approval (Final Review)

I needed to add php for instructions about zip install for it to create the /etc/php.ini
@github-actions
Copy link

Test results for 2f62adf:

Number of broken URLs: 0

URL,RESULT,FILENAME

@sspencerwire
Copy link
Contributor

@jahway603 I apologize! I didn't see this and was doing the rewrite of the entire procedure this morning and just merged that (as promised from the conversation in your forum post). php should install with the php80-php from the REMI repository. Do you need PHP from Rocky as well?

@sspencerwire
Copy link
Contributor

@all-contributors please add @jahway603 for content

@allcontributors
Copy link
Contributor

@sspencerwire

I've put up a pull request to add @jahway603! 🎉

@sspencerwire
Copy link
Contributor

@jahway603 I'm guessing that you couldn't find the php.ini file in /etc and this is why you thought it necessary to install php... if that is the case, the path to the php.ini file is now corrected.

@jahway603
Copy link
Author

jahway603 commented Nov 21, 2022

@sspencerwire The rewrite looks great. That's correct that I couldn't find php.ini in /etc, as it was in the documentation.

@sspencerwire
Copy link
Contributor

@jahway603 since this should now be fixed, I'll close your PR, but thank you for being involved!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants