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

Link to installation documentation and Windows downloads #1076

Merged
merged 2 commits into from
Sep 24, 2024

Conversation

jimwins
Copy link
Member

@jimwins jimwins commented Sep 18, 2024

No description provided.

Copy link

github-actions bot commented Sep 18, 2024

🚀 Regression report for commit f252981 is at https://web-php-regression-report-pr-1076.preview.thephp.foundation

Copy link

github-actions bot commented Sep 18, 2024

🚀 Preview for commit f252981 can be found at https://web-php-pr-1076.preview.thephp.foundation

Copy link
Member

@cmb69 cmb69 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! This makes generally sense to me.

downloads.php Outdated Show resolved Hide resolved
downloads.php Outdated
<a href="/manual/en/install.unix.php">Installation instructions for Unix systems</a>
</li>
<li>
<a href="/manual/en/install.macosx.php">Installation instructions for macOS</a>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we want to link to the install page for macOS and Unix instead of direct to the packages, just to not push users too deep into the weeds. Those pages in the documentation should be cleaned up to put information about downloading binaries/packages before instructions on building from source.

downloads.php Outdated

<ul>
<li>
<a href="/manual/en/install.unix.php">Installation instructions for Unix systems</a>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most of the installation instructions for Unix refer to manual builds. I think only https://www.php.net/manual/en/install.unix.debian.php is about packages. Besides that page needs an overhaul, we should at least have a respective page about Fedora/RHEL packages (yum). Maybe @remicollet is interested in this?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, we can at least have a unix.yum that corresponds to unix.debian without much controversy, I hope, and then actually link to the more comprehensive third-party repositories if we decide to start calling out the third-party things.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Debian page could mention Ubuntu too (and maybe mention i.e. Ondrej's repo since it covers more versions than the Debian/Ubuntu packages?).

<h2>Binaries</h2>

<p>
<a href="https://windows.php.net/download/">Binaries are available for
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should probably remove the Windows download links below if we link it up here.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not worried about the redundancy, and the later links go to the specific #id for the corresponding version.


<ul>
<li>
<a href="/manual/install.unix.php">Installation instructions for Unix systems</a>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is for building from source and should be put below, and probably made more obvious on the linked page itself that it's for building from source and not likely the user's first choice.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, see my response to @cmb69 earlier. This part of the documentation should be re-jiggered a bit to be more binary/package forward.

<a href="https://windows.php.net/download/">Binaries are available for
Microsoft Windows</a>. The PHP project does not currently release binary packages
for other platforms such as Linux or macOS, but they are packaged by distributions
and other providers. For more information, see:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should be mentioning them here perhaps?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure what you mean by "them".

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The distributions for PHP from third parties.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we want to link from this page to those, but I hope we can get agreement to start putting some of that sort of information in the documentation.

@NattyNarwhal
Copy link
Member

Forgot to check for other reviews before I hit send; I see Christoph has covered a lot of the same points I have at least.

@jimwins
Copy link
Member Author

jimwins commented Sep 18, 2024

Started working on the documentation side of this here: php/doc-en#3763

@jimwins
Copy link
Member Author

jimwins commented Sep 18, 2024

This was inspired by this GitHub issue and related discussion on the Foundation Slack channel.

@jimwins
Copy link
Member Author

jimwins commented Sep 24, 2024

Documentation changes landed, so merging this.

@jimwins jimwins merged commit 9a1fd55 into php:master Sep 24, 2024
5 checks passed
@jimwins jimwins deleted the add-downloads-introduction branch September 24, 2024 20:48
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