Skip to content

Commit

Permalink
Fix error when generating AMD/UMD dependencies
Browse files Browse the repository at this point in the history
Generating AMD/UMD dependencies lead to an error:

  $ bin/pegjs --format amd --dependency $:jquery examples/arithmetics.pegjs
  dependencyIds is not defined

  $ bin/pegjs --format umd --dependency $:jquery examples/arithmetics.pegjs
  dependencyIds is not defined

This commit fixes the problem, which was caused by a mistake done in
d2569b9.
  • Loading branch information
dmajda committed Oct 10, 2016
1 parent 83d748d commit ef3abf3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/compiler/passes/generate-js.js
Expand Up @@ -1275,7 +1275,7 @@ function generateJS(ast, options) {

amd() {
let dependencyVars = Object.keys(options.dependencies);
let dependencyIds = dependencyIds.map(v => options.dependencies[v]);
let dependencyIds = dependencyVars.map(v => options.dependencies[v]);
let dependencies = "["
+ dependencyIds.map(
id => "\"" + js.stringEscape(id) + "\""
Expand Down Expand Up @@ -1313,7 +1313,7 @@ function generateJS(ast, options) {
umd() {
let parts = [];
let dependencyVars = Object.keys(options.dependencies);
let dependencyIds = dependencyIds.map(v => options.dependencies[v]);
let dependencyIds = dependencyVars.map(v => options.dependencies[v]);
let dependencies = "["
+ dependencyIds.map(
id => "\"" + js.stringEscape(id) + "\""
Expand Down

0 comments on commit ef3abf3

Please sign in to comment.