Skip to content

Change to forcing adding php alias. #200

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

Closed
wants to merge 1 commit into from
Closed

Conversation

Romeoy
Copy link

@Romeoy Romeoy commented Apr 29, 2025

There is already a default php alias for some php version. The code RUN ln -s /usr/bin/php84 /usr/bin/php will be failed, so just adding -f arg to make it work well.

@TrafeX
Copy link
Owner

TrafeX commented May 18, 2025

Hi @Romeoy,

I'm not sure why this would be needed. There should only be one PHP version installed.
Can you clarify this a bit more?

@Romeoy
Copy link
Author

Romeoy commented May 19, 2025

Hi @Romeoy,

I'm not sure why this would be needed. There should only be one PHP version installed. Can you clarify this a bit more?

For my case, I just modify and change the php and extensions' version to 8.3. When instaling PHP 8.3, system will automacially create a link file /usr/bin/php, then the command RUN ln -s /usr/bin/php83 /usr/bin/php will be failed.

@TrafeX
Copy link
Owner

TrafeX commented Jun 29, 2025

Can you show me how your Dockerfile looks like so I can better understand your use-case?

@Romeoy
Copy link
Author

Romeoy commented Jun 30, 2025

Can you show me how your Dockerfile looks like so I can better understand your use-case?

Please check. Thanks. https://github.com/Romeoy/docker-php-nginx/blob/develop/Dockerfile

@TrafeX
Copy link
Owner

TrafeX commented Jun 30, 2025

I get it, but this seems more applicable on your variant of this Docker image than on this version.
I explicitly keep things simple and not dynamic so it's easy to understand for everyone. When the symlink already exists, I want to see that error because then I don't need to add the symlink, or I might made a mistake.
So forcing it doesn't add any value in my opinion.

@TrafeX TrafeX closed this Jun 30, 2025
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.

3 participants