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

Remove Cargo.lock from .gitignore #10

Closed
s1341 opened this issue Mar 18, 2020 · 8 comments
Closed

Remove Cargo.lock from .gitignore #10

s1341 opened this issue Mar 18, 2020 · 8 comments

Comments

@s1341
Copy link

s1341 commented Mar 18, 2020

Hi,
Would you be willing to remove the Cargo.lock from the .gitignore and commit it?

It would greatly simplify installation on nixos, and would allow me to try to add page to the nixpkgs repo.

Thanks
s1341

@s1341
Copy link
Author

s1341 commented Mar 18, 2020

see https://doc.rust-lang.org/cargo/guide/cargo-toml-vs-cargo-lock.html for reference (and the comment in your own .gitignore: page is an executable, not a library).

@I60R
Copy link
Owner

I60R commented Mar 18, 2020

Hello,

Sure, yes I'll do that.

Thank you for making nix package and for this helpful information about Cargo.lock

@s1341
Copy link
Author

s1341 commented Mar 19, 2020

Thanks. Works!

For your reference, here is the nix derivation for v2.3.2:

{ stdenv, rustPlatform, fetchFromGitHub }:

rustPlatform.buildRustPackage rec {
  pname = "page";
  version = "v2.3.2";

  src = fetchFromGitHub {
    owner = "I60R";
    repo = pname;
    rev = version;
    sha256 = "1plnli0h9zcmncjn8xv1114lp4pw18hajcpmb1cbmhn43isqn3v5";
  };

  cargoSha256 = "0z1qlz0kpj2vpg3snl51qhmdgs71d2g6bs9mjkb11j545z9b0rrf";

  meta = with stdenv.lib; {
    description = "Use neovim as pager";
    homepage = "https://github.com/I60R/page";
    platforms = platforms.all;
  };
}

@s1341
Copy link
Author

s1341 commented Mar 19, 2020

Closing.

@s1341 s1341 closed this as completed Mar 19, 2020
@s1341
Copy link
Author

s1341 commented Mar 19, 2020

NixPkgs PR: NixOS/nixpkgs#82949

@I60R
Copy link
Owner

I60R commented Mar 19, 2020

Great!

Would you also like to create PR that adds installation steps for NixOS to README in order to remain in commit history?

@s1341
Copy link
Author

s1341 commented Mar 19, 2020

Will do once the pr is merged.

@s1341
Copy link
Author

s1341 commented Mar 22, 2020

hey @I60R

We're encountering an issue when running page under the nix-review shell. We get an error like the following:

/nix/store/kp60cyq5iig35gzp7l2pr0ilxxa4m919-bash-interactive-4.4-p23/bin/bash: warning: here-document at line 0 delimited by end-of-file (wanted `EOF')

Can you look at NixOS/nixpkgs#82949 and see if you can help us figure it out?

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

No branches or pull requests

2 participants