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

codeowners: Add myself to autoPatchelfHook #108839

Merged
merged 1 commit into from Jan 12, 2021

Conversation

aszlig
Copy link
Member

@aszlig aszlig commented Jan 9, 2021

I really hate the very concept of this file (the reason being that I think "owner" implies some form of BDFL rather than just being
notified), but since there were recent changes in auto-patchelf.sh which I missed it's probably a good idea to add myself there solely for being notified, because ofborg can't seem to infer maintainer information here.

To make indentation consistent with all the other entries in the codeowners file, I also re-indented the other entries in the "Nixpkgs
Internals" block.

I really hate the very concept of this file (the reason being that I
think "owner" implies some form of BDFL rather than just being
notified), but since there were recent[1] changes[2] in auto-patchelf.sh
which I missed it's probably a good idea to add myself there solely for
being notified, because ofborg can't seem to infer maintainer
information here.

To make indentation consistent with all the other entries in the
codeowners file, I also re-indented the other entries in the "Nixpkgs
Internals" block.

[1]: NixOS#101142
[2]: NixOS#106830

Signed-off-by: aszlig <aszlig@nix.build>
@Ma27 Ma27 merged commit ac81227 into NixOS:master Jan 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants