Follow-up to #548482: remove ocis_5-bin and the NixOS module.
ocis_5-bin is far out of date (latest is 8), and oCIS requires a strict upgrade sequence, requiring every intermediate package.
My attempt in #409138 to bring those packages stalled out despite review requests.
I was the only active maintainer, and I’m migrating away from oCIS in general.
Since 5 no longer supported by upstream anyways, it only makes sense to remove the pkg and module from nixpkgs.
cc @bhankas @dotlambda
AI assistance: Pi with OpenAI gpt-6-astra.
Things done
- Built on platform:
- x86_64-linux
- aarch64-linux
- aarch64-darwin
- Tested, as applicable:
- NixOS tests in nixos/tests.
-
Package tests at
passthru.tests. - Tests in lib/tests or pkgs/test for functions and "core" functionality.
-
Ran
nixpkgs-reviewon this PR. See nixpkgs-review usage. -
Tested basic functionality of all binary files, usually in
./result/bin/. - Nixpkgs Release Notes
- Package update: when the change is major or breaking.
- NixOS Release Notes
- Module addition: when adding a new NixOS module.
- Module update: when the change is significant.
- Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.
- Follows the automation/AI policy.
- The commit messages follow the nixpkgs format, like
hello: 2.12 -> 2.13ornixos/nginx: fix the test. Guide metais complete:description,homepage,license,maintainers,mainProgram. Guide- Sources are fixed-output and pinned by a hash. No
lib.fakeHashis left. Guide - A command-line tool has
versionCheckHook, or another check that runs it. Guide meta.platformsmatches what upstream supports. Guide- The tests run:
passthru.tests, the NixOS test of a module, or nixpkgs-review. Guide - AI use is disclosed with an
Assisted-by:trailer. Guide
The title names what it changes. AI use: disclosed: pi.
Files changed 10
Reviews and comments
Thanks for contributing to the documentation
Make sure you follow the documentation styleguide, most notably:
- Show, don't tell: lead with a minimal working example; explanation follows the code.
- No meta-commentary: don't write "This section explains how to…", just do it.
- Imperative mood and active voice: "Run the command", not "The user should run the following command".
- Present tense: "This creates a folder", not "This will create a folder".
- Be confident: no hedging with "should", "might", "typically", "usually".
- Cut filler words: "simply", "just", "easily", "basically"; "to", not "in order to".
For larger changes, like adding or removing whole sections, ask the NixOS documentation team for a review.
Hi @Ramblurr , thank you for your work on this. Super sorry for not being responsive enough, life has been busy.
Not sure if I mentioned this before, but I have moved away from OCIS since quite a while and wouldn't be a good judge for its reviewing anything related to it. If its not too much trouble I'd like to request you to also remove my name from maintainers. If it is trouble, no problem all the same, I'll get around to doing it when life permits.
Apologies again for non-response earlier, even this message is being typed from a phone on pu lic transport because thats just where things are :)
On second note, you removed the package and any maintainer ship with it. Approved!
@bhankas wants to merge this PR.
Requirements to merge this PR with @NixOS/nixpkgs-merge-bot merge:
- :white_check_mark: PR targets a development branch.
- :x: PR touches only files of packages in
pkgs/by-name/. - :x: PR is at least one of:
- :white_check_mark: PR is not a draft
- :white_check_mark: PR is not blocked by a "changes requested" review from a committer.
- :white_check_mark: bhankas is a member of @NixOS/nixpkgs-maintainers.
:x: Pull Request could not be merged (#305350)