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

Switch build system to Browserify #75

Merged
merged 15 commits into from
Jun 17, 2014
Merged

Switch build system to Browserify #75

merged 15 commits into from
Jun 17, 2014

Conversation

devongovett
Copy link
Member

This switches the build system from Importer to Browserify. Importer was kinda okish, but was buggy and suboptimal in terms of isolating modules, and our Node support was shabby at best because of it. Browserify fixes these problems: it's well maintained, uses a standardized module system (CommonJS, or more specifically, Node's module system), and makes Node and browsers able to use the same code easily. It will also allow us to use some of Node's features in the future, such as streams perhaps (I need to investigate this more).

This should cause no difference to the API. In browsers, a module is exported for CommonJS and AMD, and falls back to the AV global as before. In node, the API did change. We no longer need the special AV.require function to load codecs. You can now just require them normally:

var AV = require('av');
require('mp3');

I will publish aurora and all the codecs on npm, as well as upload prebuild versions (with source maps) to Github's releases tab as we release new versions. All of this is in the new Makefile. At some point I'd like to separate the currently built-in decoders and demuxers into their own modules on npm and in Github too so people have more control over what is included in their build.

Hopefully this will make things easier to build and debug, and easier to understand for newcomers to the project. 😄

@crackofdusk
Copy link
Member

👍

devongovett added a commit that referenced this pull request Jun 17, 2014
Switch build system to Browserify
@devongovett devongovett merged commit 97c717e into master Jun 17, 2014
@devongovett devongovett deleted the browserify branch June 17, 2014 17:24
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.

2 participants