Skip to content
This repository has been archived by the owner on Aug 1, 2018. It is now read-only.

Removed the prerequisite of coffee-script #2

Closed
wants to merge 1 commit into from

Conversation

timnew
Copy link

@timnew timnew commented Mar 15, 2013

Rewrite simple-server in javascript, to remove the explicit prerequisite of coffee-script

@balupton
Copy link
Member

Thanks for this, this will break with node's deprecation of require extensions. The ideal solution is to have simple-server pre-compiled. I don't currently have the time to do this, do you? We can use the same build and test strategies as the bevry projects to keep things consistent: http://github.com/bevry

@timnew
Copy link
Author

timnew commented Jun 19, 2013

I haven't heard about the deprecation of require extensions. Do you have any clue or reference about it?
I cannot find any document about the require extension from node.js' latest document. But I also haven't got any deprecation warning. So I have use this approach for more than 1 year.

@balupton
Copy link
Member

No worries, here's the word on the deprecation:

http://nodejs.org/changelog.html
2013.05.14 Version 0.10.6 (Stable), 5deb16
module: Deprecate require.extensions (isaacs)

nodejs/node-v0.x-archive#5430

jashkenas/coffeescript#2441

@balupton
Copy link
Member

Fixed in v1.0.1.

@balupton balupton closed this Oct 23, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants