Skip to content

Remove SGI IRIX support#2404

Open
kinkie wants to merge 2 commits intosquid-cache:masterfrom
kinkie:deprecate-irix
Open

Remove SGI IRIX support#2404
kinkie wants to merge 2 commits intosquid-cache:masterfrom
kinkie:deprecate-irix

Conversation

@kinkie
Copy link
Copy Markdown
Contributor

@kinkie kinkie commented Apr 10, 2026

SGI IRIX was last released in 2006, and
support discontinued in 2013.

kinkie added 2 commits April 9, 2026 22:17
SCO OpenServer was EOL'd in 2023.
Users wishing to build for this OS can use build time options:
 ./configure CFLAGS="-lintl"
@yadij yadij added M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels feature maintainer needs documentation updates for merge S-could-use-an-approval An approval may speed this PR merger (but is not required) backport-to-v7 maintainer has approved these changes for v7 backporting labels Apr 11, 2026
@rousskov rousskov self-requested a review April 11, 2026 10:49
@rousskov rousskov changed the title Remove IRIX support Remove SGI IRIX support Apr 13, 2026
Copy link
Copy Markdown
Contributor

@rousskov rousskov left a comment

Choose a reason for hiding this comment

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

I support removing official SGI IRIX support and associated code hacks.

This PR leaves IRIX-specific code in tools/sysvinit/squid.rc. Please drop that code or adjust its description to become more general. This is not my area of expertise, but googling suggests that generalizing that comment may be more appropriate here, with an updated comment saying something like "Many older Linux distributions that use System V".

Please remove a mention of IRIX use case from compat/fdsetsize.h. For example, the updated comment can say "On some systems (e.g., NetBSD), FD_SETSIZE ...".

Please remove install-irix6.2 target from contrib/Makefile.am.

I added "SGI" to the PR title to provide more context for "IRIX".

@rousskov rousskov added S-waiting-for-author author action is expected (and usually required) and removed M-cleared-for-merge https://github.com/measurement-factory/anubis#pull-request-labels S-could-use-an-approval An approval may speed this PR merger (but is not required) labels Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-to-v7 maintainer has approved these changes for v7 backporting feature maintainer needs documentation updates for merge S-waiting-for-author author action is expected (and usually required)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants