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

Screen configuration in /etc/screenrc #280

Merged
merged 1 commit into from
Jan 27, 2013
Merged

Screen configuration in /etc/screenrc #280

merged 1 commit into from
Jan 27, 2013

Conversation

aristidb
Copy link
Contributor

It is in general possible to add a global screen configuration, but the nixpkgs screen package does not generate one. Most people only use a per-user configuration, of course.

I saw two options:

  1. Parametrise the screen package with the configuration file. While pure, this would require rebuilding screen for every change and would make it impossible to use binary packages.
  2. Set screen to use the global /etc/screenrc. Use this in conjunction with the nixos "etc" configuration option, and it should be possible to deal with.

While not entirely happy, as it is slightly impure, I chose option (2): I think it is a strict improvement over the status quo and should be workable.

(Oh and yes, I am absolutely militant about Ctrl-a being a stupid escape sequence for screen.)

@shlevy
Copy link
Member

shlevy commented Jan 27, 2013

System-wide configuration is more of a task for nixos, especially if you're putting files in /etc.

@viric
Copy link
Member

viric commented Jan 27, 2013

Well, nixpkgs already makes, for many programs, them to read from /etc. Nothing new.

Who should put that /etc/screenrc file there, it's up to nixos/sysadmin.

@aristidb
Copy link
Contributor Author

@shlevy I'm not doing any system-wide configuration here, it's just that the package accepts a configuration which I've placed in etc. via environment.etc from NixOS.

@aristidb
Copy link
Contributor Author

I'm open for suggestions for how to implement the same goal differently though, no matter if it requires changing nixos or nixpkgs.

viric added a commit that referenced this pull request Jan 27, 2013
Make Screen read the configuration from /etc/screenrc
@viric viric merged commit 336d929 into NixOS:master Jan 27, 2013
@viric
Copy link
Member

viric commented Jan 27, 2013

I saw screen comes without any $out/etc/screenrc, and we make many nixpkgs programs read from /etc, so I think it's all fine.

infinixbot pushed a commit to infinixbot/nixpkgs that referenced this pull request Feb 17, 2025
Format using the base commit from nixfmt PR 283: NixOS/nixfmt@e825e95
infinixbot pushed a commit to infinixbot/nixpkgs that referenced this pull request Feb 18, 2025
Format using the base commit from nixfmt PR 284: NixOS/nixfmt@e825e95
infinixbot pushed a commit to infinixbot/nixpkgs that referenced this pull request Feb 18, 2025
Format using the base commit from nixfmt PR 285: NixOS/nixfmt@e825e95
infinixbot pushed a commit to infinixbot/nixpkgs that referenced this pull request Feb 21, 2025
Format using the base commit from nixfmt PR 283: NixOS/nixfmt@e825e95
infinixbot pushed a commit to infinixbot/nixpkgs that referenced this pull request Feb 24, 2025
Format using the base commit from nixfmt PR 286: NixOS/nixfmt@e825e95
infinixbot pushed a commit to infinixbot/nixpkgs that referenced this pull request Feb 24, 2025
Format using the base commit from nixfmt PR 283: NixOS/nixfmt@e825e95
CodeRadu pushed a commit to CodeRadu/nixpkgs that referenced this pull request Feb 25, 2025
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.

3 participants