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

heroku: 3.43.16 -> 6.6.14 #25983

Closed
wants to merge 1 commit into from
Closed

heroku: 3.43.16 -> 6.6.14 #25983

wants to merge 1 commit into from

Conversation

shosti
Copy link
Contributor

@shosti shosti commented May 22, 2017

Motivation for this change

This version of the CLI is no longer deprecated and uses pure nodejs.

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

This version of the CLI is no longer deprecated and uses pure nodejs.
@mention-bot
Copy link

@shosti, thanks for your PR! By analyzing the history of the files in this pull request, we identified @peterhoeg and @aflatter to be potential reviewers.

@peterhoeg
Copy link
Member

Doesn't yarn install need to access the internet to download dependencies?

@shosti
Copy link
Contributor Author

shosti commented May 22, 2017

Yes I think so, didn't realize that isn't allowed. So yarn install isn't kosher?

@shosti
Copy link
Contributor Author

shosti commented May 22, 2017

(I tried npm2nix but it doesn't seem to work with Node 7 yet...)

@Mic92
Copy link
Member

Mic92 commented May 22, 2017

@shosti take a look at node2nix. You probably can add your package to ./pkgs/development/node-packages.

@shosti
Copy link
Contributor Author

shosti commented May 22, 2017

node2nix doesn't currently work for this because heroku has a hard dependency on node 7, and node2nix only makes nix files for node <= 6. Maybe this should wait until node2nix gets updated?

@srhb
Copy link
Contributor

srhb commented Jul 18, 2017

@shosti It looks like there has recently been some commits on node2nix that give it a -7 flag which I assume allows node 7 packages. Worth a shot?

@Mic92
Copy link
Member

Mic92 commented Aug 12, 2017

If somebody is in hurry and needs heroku quickly:

# bashrc/zshrc
heroku(){
  docker run -it --rm -u $(id -u):$(id -g) -w "$HOME" \
    -v /etc/passwd:/etc/passwd:ro \
    -v /etc/group:/etc/group:ro \
    -v /etc/localtime:/etc/localtime:ro \
    -v /home:/home \
    -v /tmp:/tmp \
    -v /run/user/$(id -u):/run/user/$(id -u) \
    -v $(pwd):/workdir \
    -w /workdir \
    --name heroku \
    johnnagro/heroku-toolbelt "$@"
}

@DCPRevere
Copy link

DCPRevere commented Aug 26, 2017

@Mic92 Thank you very much for your addition! I found that it couldn't interact with the local directory such as is necessary when running create and the like, so I added two extra lines to fix this:

-v $(pwd):/workdir \
-w /workdir \

The first adds the working directory as a volume, and the second makes that directory the WORKDIR.

@abathur
Copy link
Member

abathur commented Feb 17, 2018

@shosti bump? It sounds like the node2nix issue isn't blocking since your PR adding node 7 support was merged in. Do you know if there are any other blocking issues?

@CMCDragonkai
Copy link
Member

CMCDragonkai commented Mar 10, 2018

The current heroku package is marked broken. Is this the only PR that makes heroku work again?

@garbas
Copy link
Member

garbas commented Mar 12, 2018

@CMCDragonkai i think we need to package this with node2nix.

/me runs node2nix

@shosti
Copy link
Contributor Author

shosti commented Mar 12, 2018

Yeah sorry I haven’t had time to get this working. node2nix seems to get the dependencies ok, but I haven’t figured out how to get the binaries wrapped properly. If someone wants to take this over that would be awesome.

@rail
Copy link

rail commented Apr 5, 2018

It looks like mkYarnPackage may be the easiest way to go.

@bsima
Copy link
Contributor

bsima commented May 10, 2018

As a workaround until this gets finished, here's how I got heroku cli installed on NixOS 18.03

git clone https://github.com/heroku/cli.git heroku-cli
cd heroku-cli
nix-env -f '<nixpkgs>' -iA nodePackages.node2nix
node2nix -8
nix-build -A package
nix-env -i ./result
heroku login

@domenkozar
Copy link
Member

I've opened #42869 as the next attempt at fixing this, I'm going to close this issue as it's outdated.

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