Skip to content

Commit

Permalink
Merge pull request #491 from adityab/amd-unsupport
Browse files Browse the repository at this point in the history
Don't try to support AMD loaders in socket.io
  • Loading branch information
rauchg committed Nov 10, 2012
2 parents 4bf6f14 + b5585aa commit 63f1838
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions bin/builder.js
Original file line number Diff line number Diff line change
Expand Up @@ -85,15 +85,13 @@ var baseTransports = {

/**
* Wrappers for client-side usage.
* This enables usage in top-level browser window, client-side CommonJS systems and AMD loaders.
* This enables usage in top-level browser window and client-side CommonJS systems.
* If doing a node build for server-side client, this wrapper is NOT included.
* @api private
*/
var wrapperPre = "\nvar io = ('undefined' === typeof module ? {} : module.exports);\n(function() {\n";

var wrapperPost = "\nif (typeof define === \"function\" && define.amd) {" +
"\n define([], function () { return io; });" +
"\n}\n})();";
var wrapperPost = "\n})();";


/**
Expand Down

1 comment on commit 63f1838

@pineliu
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have the same problem on using socket.io-client with Dojo. This change solves the problem. Thank you!

Please sign in to comment.