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

index: update requirements and recommendations to current release #34

Merged
merged 10 commits into from
Mar 4, 2018

Conversation

Trim
Copy link

@Trim Trim commented Apr 25, 2017

Hello,

It's time to update our community website, since 1.2.0 release requirements have changed and recommendations too.

I have modified requirements according to the FreshRSS changelogs, so a review from a developer will be more than recommended ;)

I also have added a note about PHP versions supported with security fixes (only PHP 5.6 have security fixes, even if FreshRSS supports 5.3).

Thanks for your great work,
Adrien

pages/index.html Outdated
</tr>
</tbody>
</table>
<p>
*: only PHP 5.6+ <a href="http://php.net/supported-versions.php">currently has security fixes</a> by PHP community<br>
Copy link
Member

Choose a reason for hiding this comment

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

Actually, security updates are typically provided by the OS distributions much later than the upstream PHP end of support. It is still good to avoid using versions that are much too old, though :-)
An example of extremely slow distribution is CentOS http://distrowatch.com/table.php?distribution=CentOS

Copy link
Author

Choose a reason for hiding this comment

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

Yes indeed, but some distributions like Windows don't provide security updates for PHP ;)

PHP is a big piece of software and so on most case it's simpler to rely on the upstream support IMHO.

pages/index.html Outdated
<p>Required: <a href="http://php.net/xml">libxml</a>, <a href="http://php.net/curl">cURL</a>, <a href="http://php.net/pdo-mysql">PDO_MySQL</a> or <a href="http://php.net/pdo-sqlite">PDO_SQLite</a>, <a href="http://php.net/gmp">GMP</a></p>
<p>Recommended: <a href="http://fr2.php.net/json">JSON</a>, <a href="http://php.net/zlib">Zlib</a>, <a href="http://php.net/mbstring">mbstring</a>, <a href="http://php.net/iconv">iconv</a>, <a href="http://php.net/zip">Zip</a></p>
<p>Required: <a href="http://php.net/xml">libxml</a>, <a href="http://php.net/curl">cURL</a>, <a href="http://php.net/dom">DOM</a> <a href="http://php.net/pdo-mysql">PDO_MySQL</a></p>
<p>Recommended: <a href="http://fr2.php.net/json">JSON</a>, <a href="http://php.net/gmp">GMP</a>, <a href="http://php.net/zlib">Zlib</a>, <a href="http://php.net/intl.idn">IDN</a>, <a href="http://php.net/mbstring">mbstring</a>, <a href="http://php.net/iconv">iconv</a>, <a href="http://php.net/zip">Zip</a>, <a href="http://php.net/zlib">zilb</a></p>
Copy link
Member

Choose a reason for hiding this comment

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

Oups, there is a little typo at zilb ;-)

Copy link
Member

Choose a reason for hiding this comment

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

Could you please remove the fr2. prefix in the URLs?

Copy link
Author

Choose a reason for hiding this comment

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

Thanks, I've modified these typos.

pages/index.html Outdated
</td>
<td>

<p>Database: <a href="http://php.net/pdo-sqlite">PDO_SQLite</a> or <a href="http://php.net/pdo-pgsql">PDO_PGSQL</a></p>
Copy link
Member

Choose a reason for hiding this comment

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

There is also MySQL. (MySQL, SQLite, PostgreSQL)

Copy link
Author

@Trim Trim May 26, 2017

Choose a reason for hiding this comment

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

MySQL is in the "Recommened" first column. SQLite and PostgreSQL are in the "Works also with" column.

@Trim
Copy link
Author

Trim commented May 26, 2017

I've added some commits to be ready for the next 1.7 release and to simplify a bit the recommendation on older versions. I don't know if that's good to list all compatible versions on the home page of the project.

@Trim
Copy link
Author

Trim commented May 26, 2017

PS: we'll need to release 1.7 before publishing this to master. See FreshRSS/FreshRSS#1548 .

@marienfressinaud
Copy link
Member

Hi @Trim and @Alkarex, I'm sorry I totally forgot about this PR, but it seems very relevant. I'm not sure what versions we are supporting right now, is this PR still up-to-date? I can merge and publish it if it's ok 👍

pages/index.html Outdated
</tr>
</tbody>
</table>
<p>
*: be aware that open-source communities often don't support on a long term older versions.<br/>
Copy link
Member

Choose a reason for hiding this comment

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

don't offer (long-term) support for older versions
don't support older versions

pages/index.html Outdated
</tr>
</tbody>
</table>
<p>
*: be aware that open-source communities often don't support on a long term older versions.<br/>
Especially for PHP which will be in front of your server and have only support for 5.6+ and 7+ currently.</p>
Copy link
Member

Choose a reason for hiding this comment

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

How about something like

Especially for PHP, which only offers long-term support for 5.6+ and 7+.

pages/index.html Outdated
@@ -42,7 +42,7 @@ <h2>Self hostable</h2>
<p class="left">You are convinced by FreshRSS but you do not know what you need? Keep calm and have a look at the following table:</p>

<table class="alternate striped-vert">
<caption>Software basis for FreshRSS</caption>
<caption>Software basis for FreshRSS for 1.7 release</caption>
Copy link
Member

Choose a reason for hiding this comment

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

Maybe FreshRSS 1.10 requirements? I think a software basis is something slightly different than intended (e.g., GNU Core Utilities is a common software basis for most distros).

@Frenzie
Copy link
Member

Frenzie commented Feb 26, 2018

@marienfressinaud I added a couple of nitpicks. I don't think I ever even knew this PR existed. :-P

@Frenzie Frenzie merged commit d7d5b8f into FreshRSS:master Mar 4, 2018
@marienfressinaud
Copy link
Member

Thanks @Frenzie, I just pushed changes on the server :)

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.

4 participants