Skip to content
Permalink
Browse files
Trim out the module wrappers during the build process. Follow-up to #…
…7011.
  • Loading branch information
jeresig committed Sep 9, 2010
1 parent 82b35a4 commit 1ed405176bfac58f35c3027006e152a4725fd63b
Showing 1 changed file with 2 additions and 0 deletions.
@@ -56,6 +56,8 @@ ${JQ}: selector ${MODULES}
@@mkdir -p ${DIST_DIR}

@@cat ${MODULES} | \
sed 's/.function..jQuery...{//' | \
sed 's/}...jQuery..;//' | \
sed 's/Date:./&'"${DATE}"'/' | \
${VER} > ${JQ};

3 comments on commit 1ed4051

@rwaldron
Copy link
Member

@rwaldron rwaldron commented on 1ed4051 Sep 12, 2010

Choose a reason for hiding this comment

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

Was the build process supposed to be updated to change:

var jQuery = (function() { ... to ... (function() {

and

return window.jQuery = window.$ = jQuery; ... to ... window.jQuery = window.$ = jQuery;

?

@rwaldron
Copy link
Member

@rwaldron rwaldron commented on 1ed4051 Sep 12, 2010

Choose a reason for hiding this comment

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

sed 's/var jQuery = (f/(f/' |
sed 's/return window/window/' | \

@jeresig
Copy link
Member Author

@jeresig jeresig commented on 1ed4051 Sep 12, 2010

Choose a reason for hiding this comment

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

No, it was fine leaving it as-is (works both standalone and part of the build process).

Please sign in to comment.