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

🔗 Fix install URLs #376

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

Jamim
Copy link
Contributor

@Jamim Jamim commented Mar 7, 2024

Hello everyone,

I hope you'll find these changes useful.

Best regards!

Copy link

linux-foundation-easycla bot commented Mar 7, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: Jamim / name: Aliaksei Urbanski (63fd6b7)

`https://pkgs.org/download/Imath
<https://pkgs.org/download/Imath>`_ for a complete list. The common
ones that generally provide current releases include:
`Imath's page on Repology
Copy link
Member

Choose a reason for hiding this comment

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

Thank you for the fixes! You might have seen that we recently added a similar reference to openexr, including an inline badge from repology listing the current versions. I was going to make a similar addition here but you've done it already, thanks!

Could I ask you to mirror that here, with some slight modifications? I like the badge/table better than the list of Fedora, Gentoo, and Ubuntu, and the comment about out-of-date distributions applies to OpenEXR than to Imath, although it does warrant a reference to ilmbase. And homebrew is up to date now, too, so that comment should be struck.

Suggested change
`Imath's page on Repology
Refer to the current version of Imath on various major Linux distros at
`repology.org <https://repology.org/project/imath/versions>`_:
.. image:: https://repology.org/badge/vertical-allrepos/imath.svg?exclude_unsupported=1&columns=4&header=Imath%20Packaging%20Status&minversion=3.0
:target: https://repology.org/project/imath/versions
Older versions of Imath were distributed as a component of OpenEXR
called ``ilmbase``. We do not recommend using these outdated
versions.
To install via ``yum`` on RHEL/CentOS:
.. code-block::
% sudo yum makecache
% sudo yum install imath
To install via ``apt-get`` on Ubuntu:
.. code-block::
% sudo apt-get update
% sudo apt-get install imath
macOS
-----
On macOS, install via `Homebrew <https://formulae.brew.sh/formula/imath>`_:
.. code-block::
% brew install imath
Alternatively, you can install on macOS via `MacPorts
<https://ports.macports.org/port/imath>`_:
.. code-block::
% port install imath
Windows
-------
Install via `vcpkg <https://vcpkg.io/en/packages>`_:
.. code-block::
% .\vcpkg install imath

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hello @cary-ilm,

Thank you for the suggestion! 🙇🏼
I've applied it with minor adjustments.

You might have seen that we recently added a similar reference to openexr, including an inline badge from repology listing the current versions.

No, I haven't 😄
I just noticed that the link to the Gentoo package is broken while I was working on gentoo/gentoo#35188. So I assumed it's worth checking other links. And I also decided to replace pkgs.org with Repology since it has a much broader coverage.

Co-authored-by: Cary Phillips <cary@ilm.com>
Signed-off-by: Aliaksei Urbanski <aliaksei.urbanski@gmail.com>
Copy link
Member

@cary-ilm cary-ilm left a comment

Choose a reason for hiding this comment

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

Perfect, thanks!

@cary-ilm cary-ilm merged commit 9613093 into AcademySoftwareFoundation:main Mar 8, 2024
25 checks passed
@Jamim Jamim deleted the fix/install-urls branch March 8, 2024 15:23
cary-ilm added a commit that referenced this pull request Sep 4, 2024
Signed-off-by: Aliaksei Urbanski <aliaksei.urbanski@gmail.com>
Co-authored-by: Cary Phillips <cary@ilm.com>
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