Skip to content
This repository

Hi! I fixed some calls to "sys" for you! #67

Closed
wants to merge 1 commit into from

3 participants

node-migrator-bot Sean McArthur Farrin Reid
node-migrator-bot

Hello seanmonstar!

I am migrationBot, an open-source bot, and I'm here to help you migrate your codebase to node v0.8!

Did you know that the "sys" module throws an error if your program tries to require it in node v0.8? To help keep your code running, I automatically replaced var sys = require('sys') with var util = require('util').

If you'd like to know more about these changes in node.js, take a look at https://github.com/joyent/node/commit/1582cf#L1R51 and https://github.com/joyent/node/blob/1582cfebd6719b2d2373547994b3dca5c8c569c0/ChangeLog#L51 and http://blog.jit.su/introducing-blakmatrix-and-migratorbot .

As for myself, I was written by your friendly neighborhood node ninjas at Nodejitsu, and you can find them at #nodejitsu on irc.freenode.net or with http://webchat.jit.su .

Enjoy!

--migrationBot

Sean McArthur seanmonstar closed this June 25, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Jun 23, 2012
Farrin Reid [fix] Changed require('util') to require('util') for compatibility wi…
…th node v0.8
bf6271c
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. 2  test/testigo/runners/simple.js
2  test/testigo/runners/simple.js
@@ -13,7 +13,7 @@ provides: [SimpleRunner]
13 13
 
14 14
 var sys, printer = function(type){
15 15
 	switch(type){
16  
-		case 'node': return require('sys').print;
  16
+		case 'node': return require('util').print;
17 17
 		case 'helma':
18 18
 		case 'ringo':
19 19
 		case 'flusspferd':
Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.