onlyoffice-documentserver: use icu78

[raboof]

This is needed so that doctrenderer uses the same version of icu as the nodejs.libv8 . This is a little scary because it's a different version of icu than boost is built with, but seems to work out in practice. After #520553 this should ideally be in sync again.

Fixes #536824

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-review on 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.

[timhae]

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 537160
Commit: f9f60a4608798c2bf37208562dbdcf2ddfa59e61

---

x86_64-linux

✅ 1 package built:

• onlyoffice-documentserver

---

aarch64-linux

✅ 1 package built:

• onlyoffice-documentserver

[vcunat]

Beware that on the current staging-next #535855 nodejs uses icu 77 again 🤕 So after this merges, I expect we revert it on staging-next.

[raboof]

Beware that on the current staging-next #535855 nodejs uses icu 77 again 🤕 So after this merges, I expect we revert it on staging-next.

Ah, I hadn't seen that, thanks ;) . So after merging this to master, I'd wait for master to be merged into staging-next again, and then PR the revert to that? (probably I'd revert only the first two changes and keep the last one in case we want to override icu again in the future)