co-log, chronos: unmarkBroken by disabling tests for chronos

[cideM]

Motivation for this change

The only reason co-log is broken because one of its dependencies chronos fails the doctests on GHC 8.10. I created an issue in the chronos repo but I think we can unbreak both packages until that is resolved

Will run nix review once PR is up

Strangely nix review says nothing to be built

Things done

• Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
• Built on platform(s)
  • NixOS
  • macOS
  • other Linux distributions
• Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
• Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
• Tested execution of all binary files (usually in ./result/bin/)
• Determined the impact on package closure size (by running nix path-info -S before and after)
• Ensured that relevant documentation is up to date
• Fits CONTRIBUTING.md.

[cdepillabout]

Instead of marking these unbroken here, can you just remove them from the broken-packages list in configuration-hackage2nix.yaml?

Here is a PR doing something similar: https://github.com/NixOS/nixpkgs/pull/108788/files#diff-3139f03f40f0c7740a4003f88522ac0af603af77f746f3e6bb8e26cccc1eaf88L8334

This this, you can remove the co-log = line, and just keep the chronos = dontCheck super.chronos; line.

Also, thanks for linking to the upstream issue for chronos!

[cideM]

Thanks @cdepillabout it's a bit embarrassing that I didn't check this myself 🙈 I just looked at the rest of the file and saw lots of unmarkBroken and just kind of went with it. A bit lazy in hindsight.

The diff is now down to removing co-log and chronos from broken packages and marking chronos as dontTest

[cdepillabout]

Looks good, thanks for this change!