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

nixpkgs manual: Remove obsolete warning #21117

Merged
merged 1 commit into from Apr 23, 2017
Merged

Conversation

lheckemann
Copy link
Member

PR #815 has already been merged.

@mention-bot
Copy link

@lheckemann, thanks for your PR! By analyzing the history of the files in this pull request, we identified @vcunat, @edolstra and @dezgeg to be potential reviewers.

@vcunat
Copy link
Member

vcunat commented Dec 13, 2016

NixOS/nix#815 has been merged, but the default nix version does not contain it yet. See #19761 (comment)

@lheckemann
Copy link
Member Author

Ah right, perhaps the warning should just be reworded then...

@domenkozar
Copy link
Member

Ping

@vcunat
Copy link
Member

vcunat commented Feb 16, 2017

@domenkozar: any indication the situation has changed? I sadly still see 1.11.6 as the default, without that PR contained or picked.

@vcunat
Copy link
Member

vcunat commented Feb 16, 2017

Ah, I'm sorry – you probably meant that the rewording should be done.

@lheckemann
Copy link
Member Author

Since 1.11.8 is now the default version and contains the commit in question, I'm guessing this can be merged?

@vcunat vcunat merged commit 5c7f466 into NixOS:master Apr 23, 2017
vcunat pushed a commit that referenced this pull request Apr 23, 2017
PR #815 has already been cherry-picked into the default nix version 1.11.8.
(cherry picked from commit 5c7f466)
@vcunat
Copy link
Member

vcunat commented Apr 23, 2017

Right, even though the situation isn't ideal, as discussed in #24717.

drawnwren pushed a commit to drawnwren/nixpkgs that referenced this pull request Apr 23, 2017
PR NixOS#815 has already been cherry-picked into the default nix version 1.11.8.
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

4 participants