Skip to content

nodejs-0_10: remove#19665

Closed
cko wants to merge 1 commit intoNixOS:masterfrom
cko:remove_nodejs-0_10
Closed

nodejs-0_10: remove#19665
cko wants to merge 1 commit intoNixOS:masterfrom
cko:remove_nodejs-0_10

Conversation

@cko
Copy link
Copy Markdown
Member

@cko cko commented Oct 18, 2016

Motivation for this change

node.js 0.10 reaches end of LTS in a few days (see https://github.com/nodejs/LTS for details). Therefore I removed it and set 3 dependant packages to broken as they don't build anymore


node.js 0.10 reaches end of LTS in a few days (see https://github.com/nodejs/LTS for details). Therefore I removed it and set 3 dependant packages to broken as they don't build anymore
@mention-bot
Copy link
Copy Markdown

@cko, thanks for your PR! By analyzing the history of the files in this pull request, we identified @flosse, @pikajude and @abbradar to be potential reviewers.

@groxxda
Copy link
Copy Markdown
Contributor

groxxda commented Oct 18, 2016

Updating statsd should fix it: statsd/statsd#565

CC @garbas ?

@NeQuissimus NeQuissimus mentioned this pull request Oct 20, 2016
7 tasks
@grahamc
Copy link
Copy Markdown
Member

grahamc commented Oct 22, 2016

@andrewrk would you like to fix groovebasin before it is marked as broken?

@vbgl would you like to fix opa before it is marked as broken?

I propose we merge in 24hrs unless we hear back.

@andrewrk
Copy link
Copy Markdown
Member

Let's mark groovebasin as broken. I need to do some work upstream to make it work on the new node and that should not block nixpkgs

@vbgl vbgl mentioned this pull request Oct 23, 2016
7 tasks
@vbgl
Copy link
Copy Markdown
Contributor

vbgl commented Oct 23, 2016

I’ve just submitted a PR that fixes the build of opa with a recent node_js. Hope this helps.

@grahamc
Copy link
Copy Markdown
Member

grahamc commented Oct 28, 2016

Merged in 80dfbfd9493b37a981686cc0041c60d382d46541, thank you!

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

Successfully merging this pull request may close these issues.

6 participants