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

Add nixpkgs documentation about how to create a derivation with Vim plugins #16501

Merged
merged 1 commit into from Jan 30, 2017

Conversation

MarcWeber
Copy link
Contributor

No description provided.

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @vcunat, @edolstra and @ericsagnes to be potential reviewers

@@ -0,0 +1,102 @@
---
title: User's Guide for Vim in Nixpkgs
author: Peter Simons
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy pasta author? (I'm guessing the author is Marc?)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MarcWeber ping


Then run

nix-shell -p vimUtils.vim_with_vim2nix --command "vim -c 'source generate.vim'"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am trying this on NixOS unstable, but I am getting:

error: attribute ‘vim_with_vim2nix’ missing, at (string):1:66
(use ‘--show-trace’ to show detailed location information)

When trying to launch this command.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

haaa OK my bad... I should have been more careful to check the patch for actual nix code change.

@matthiasbeyer
Copy link
Contributor

Any progress here? I think we can merge this as-is, can't we?

@vcunat vcunat self-assigned this Jan 30, 2017
@vcunat vcunat merged commit c3f56e9 into NixOS:master Jan 30, 2017
@vcunat
Copy link
Member

vcunat commented Jan 30, 2017

Looks OK to me, after fixing the author :-)

vcunat added a commit that referenced this pull request Jan 30, 2017
@vcunat
Copy link
Member

vcunat commented Jan 30, 2017

Thanks @MarcWeber! (I somehow missed this half a year ago.)

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

9 participants