Skip to content

Errors upgrading from 2.2.0 to 2.3.x #933

Closed

Description

@chrswk

I tried to update our existing web projects from 2.2.0 to 2.3.0/2.3.1 and get the following errors. I couldn't find a existing issue or something in the change notes that would hint at what could cause this. I'm fine with running 2.2.0, but thought I would share. This is running on Windows 10 with Java 1.8.0_66 (64-bit, Server VM).

2.2.0 (File Ref)

jjs -scripting src\main\resources\lib\r.js -- -o build.js dir=target\dist
Optimizing (standard.keepLines.keepWhitespace) CSS file: (...)
Optimizing (standard.keepLines.keepWhitespace) CSS file: (...)
Optimizing (standard.keepLines.keepWhitespace) CSS file: (...)
Optimizing (standard.keepLines.keepWhitespace) CSS file: (...)

Tracing dependencies for: (...)

Tracing dependencies for: (...)

(...)

2.3.0: (File Ref)

jjs -scripting src\main\resources\lib\r.js -- -o build.js dir=target\dist
src\main\resources\lib\r.js:22357:8 In strict mode, function declarations can only occur at program or function body level. You should use a function expression here instead.
        function addFile(file, fileUrl) {
        ^
src\main\resources\lib\r.js:22447:1 Expected , but found ;
};
 ^
src\main\resources\lib\r.js:22498:1 Expected ; but found )
});
 ^
src\main\resources\lib\r.js:27768:4 Expected ) but found function
    function setBaseUrl(fileName) {
    ^
src\main\resources\lib\r.js:27778:4 Expected eof but found }
    }
    ^

2.3.1 (Current Master)

jjs -scripting src\main\resources\lib\r.js -- -o build.js dir=target\dist
src\main\resources\lib\r.js:260 ReferenceError: "setTimeout" is not defined

Activity

  1. added this to the 2.3.2 milestone on Sep 16, 2016
  2. jrburke commented on Sep 16, 2016

    @jrburke
    Member

    I think this was caused by a problem with a change in 2.3.0, filed requirejs/requirejs#1588 to track the fix, and I just pushed a change to this repo to include that fix in r.js. You can try it to confirm if it fixes the problem by using the latest r.js snapshot:

    https://raw.githubusercontent.com/requirejs/r.js/master/dist/r.js

    Please post back to let me know if it fixes the problem, and then I'll schedule a 2.3.2 sooner rather than later to get this in an official release.

  3. chrswk commented on Sep 16, 2016

    @chrswk
    Author

    I can confirm that this fixes the issue. The build runs without errors.

    Thank you!

  4. jrburke commented on Sep 17, 2016

    @jrburke
    Member
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions