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

staging: 2024-12-14 #27

Merged
merged 9 commits into from
Dec 15, 2024
Merged

staging: 2024-12-14 #27

merged 9 commits into from
Dec 15, 2024

Conversation

Frontear
Copy link
Owner

Things Done

  1. (81f7850) Added LICENSE to repository. Honestly though, I really discourage any usage of my code.
  2. (ecdfd1c) Update nixpkgs to the latest revision and fix the module problems (nerdfonts migration mainly).
  3. (caed403) Fix the indentation in lib extensions for mkModules
  4. (e9b1839) Fix the ZSH history bug with persistence bind mounting. This is not the best fix but it works well enough that it's not worth changing. The ZSH module does need a cleanup though in the future.
  5. (a312edd) Drop the WSL desktop host, as it's been unused for a couple of months and was quite broken even while it was being used.
  6. (06c6c95) Migrate GitHub actions CI to use flake check, and setup flake check to attempt thorough builds of flake outputs to ensure correctness
  7. (39c9fda) Remove editorconfig manipulations from neovim configuration, they weren't done right and I don't think they're supposed to be set that way anyways
  8. (79dce9f) Remove stupid aliases from ZSH, specifically find and grep as they diverged too much from GNU tooling to be drop-in replacements
  9. (ca59329) Remove lib.mkMerge from XDG modules. There are many more that needs to be removed though, as these slow evaluation by a LOT.

While I honestly don't expect nor encourage anyone to use my code, I
figure it's still a good idea to add some licensing.
…usage

Nixpkgs recently split `pkgs.nerdfonts` into properly named attributes (thankfully). This update is accompanied by a small fix to use said attributes
zsh would try to merge history files by "renaming" `$HISTFILE.new` onto
`$HISTFILE`. Since the file was a bind mount, this operation would fail.

Although it never caused much problems given that originally this merge
seemed to happen if exiting an ephemeral shell, it's started happening
significantly more frequently, which prompted a desire to fix it.
I haven't used it in weeks now and I honestly am finding it annoying
with how clunky its gotten. It would consistently fail to setup the
`PATH` environment variable, which would fail to link the setuid
binaries, and occasionally would fail to startup the daemon, citing
a missing socket.

I simply don't use it enough for it to be worth my time to fix these
issues. Eventually once I am able to install NixOS on the direct
hardware I'll return this host, obviously without the WSL bindings.
Until then, it shall stay gone.
This will attempt to link together all of the relevant dependencies and force a build on them when `nix flake check` is invoked. This is useful to ensure our outputs are still building as expected, and sets the base for further checks down the line, such as run-time (with nixosModules being the first instance of this whenever we get to it).
This was incompatible and I misunderstood it's purpose. Instead, I've configured the vim specific knobs that corresponded to said settings. It's all kind of messy but it should work as expected.

On a different note, it may be a good idea to cleanup my neovim stuff. I'm just not nearly as concerned about it as I am with my NixOS configuration, but I really should be.
`fd` is not a good replacement for GNU `find`, and has been very annoying. `rg` is a bit similar, it has some surprising differences. In both cases there are benefits to keep both accessible, hence why I'm removing the aliases for these.
@Frontear Frontear merged commit ca59329 into main Dec 15, 2024
1 check passed
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.

1 participant