home-manager icon indicating copy to clipboard operation
home-manager copied to clipboard

bug: order of `plugins` and `extraConfig` sections in generated `tmux.conf`

Open alex35mil opened this issue 2 years ago • 6 comments

Are you following the right branch?

  • [X] My Nixpkgs and Home Manager versions are in sync

Is there an existing issue for this?

  • [X] I have searched the existing issues

Issue description

I feel that it's not a bug, and I misuse something, b/c nobody ever reported it, but the case should be common. Anyway.

In generated tmux.conf, the plugins section goes before extraConfig. If I configure a status bar in the extraConfig and want to use values from the plugins, it doesn't work since the status bar must be configured before run-shell calls.

Should these sections be swapped in the output?

Maintainer CC

@rycee (mentioning b/c of git blame. couldn't find the maintainers list)

System information

- system: `"aarch64-darwin"`
 - host os: `Darwin 21.6.0, macOS 12.6`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.11.0`
 - channels(Alex): `"home-manager"`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

alex35mil avatar Jan 02 '23 19:01 alex35mil

What's the status on this issue? Is there anyone working on a fix?

I think adding a configBeforePlugins field makes sense as an escape hatch for me.

Pegasust avatar Mar 22 '23 16:03 Pegasust

I think that a new config option sounds like a good idea. We need to try and preserve backwards compatibility, and moving the extraConfig location would cause a breaking change.

sumnerevans avatar Mar 23 '23 15:03 sumnerevans

Thank you for your contribution! I marked this issue as stale due to inactivity. Please be considerate of people watching this issue and receiving notifications before commenting 'I have this issue too'. We welcome additional information that will help resolve this issue. Please read the relevant sections below before commenting.

If you are the original author of the issue

  • If this is resolved, please consider closing it so that the maintainers know not to focus on this.
  • If this might still be an issue, but you are not interested in promoting its resolution, please consider closing it while encouraging others to take over and reopen an issue if they care enough.
  • If you know how to solve the issue, please consider submitting a Pull Request that addresses this issue.
If you are not the original author of the issue

  • If you are also experiencing this issue, please add details of your situation to help with the debugging process.
  • If you know how to solve the issue, please consider submitting a Pull Request that addresses this issue.
Memorandum on closing issues

Don't be afraid to manually close an issue, even if it holds valuable information. Closed issues stay in the system for people to search, read, cross-reference, or even reopen – nothing is lost! Closing obsolete issues is an important way to help maintainers focus their time and effort.

stale[bot] avatar Jun 23 '23 04:06 stale[bot]

Any news?

D3vil0p3r avatar Jan 14 '24 01:01 D3vil0p3r

This is still needed. Some tmux plugins need to set some configurations before the plugin is loaded (i.e.: tmux-catppuccin)

diegodorado avatar Jan 24 '24 00:01 diegodorado

Thank you for your contribution! I marked this issue as stale due to inactivity. Please be considerate of people watching this issue and receiving notifications before commenting 'I have this issue too'. We welcome additional information that will help resolve this issue. Please read the relevant sections below before commenting.

If you are the original author of the issue

  • If this is resolved, please consider closing it so that the maintainers know not to focus on this.
  • If this might still be an issue, but you are not interested in promoting its resolution, please consider closing it while encouraging others to take over and reopen an issue if they care enough.
  • If you know how to solve the issue, please consider submitting a Pull Request that addresses this issue.
If you are not the original author of the issue

  • If you are also experiencing this issue, please add details of your situation to help with the debugging process.
  • If you know how to solve the issue, please consider submitting a Pull Request that addresses this issue.
Memorandum on closing issues

Don't be afraid to manually close an issue, even if it holds valuable information. Closed issues stay in the system for people to search, read, cross-reference, or even reopen – nothing is lost! Closing obsolete issues is an important way to help maintainers focus their time and effort.

stale[bot] avatar Apr 23 '24 02:04 stale[bot]