Skip to content

Commit

Permalink
Fix build script so -nodebug.js doesn't have a terminated comment, de…
Browse files Browse the repository at this point in the history
…rp. Use same 12 lines as jslint rules have been moved into IIFE.
  • Loading branch information
scottschiller committed Jun 7, 2017
1 parent cfd12d6 commit 72b365f
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions build.xml
Original file line number Diff line number Diff line change
Expand Up @@ -144,12 +144,6 @@
<!-- fetch the license from the full source, and prepend it to the jsmin builds. -->

This comment has been minimized.

Copy link
@schutzstaffel911

schutzstaffel911 Mar 12, 2018

not too good at this just started with someghing thaat should be simple


<!-- a cheap hack: Read the first N lines of a file -->
<loadfile property="sm2_license_jslint" srcFile="script/soundmanager2.js">
<filterchain>
<headfilter lines="15"/>
</filterchain>
</loadfile>

<loadfile property="sm2_license" srcFile="script/soundmanager2.js">
<filterchain>
<headfilter lines="12"/>
Expand All @@ -161,7 +155,7 @@

<!-- prepend license -->
<!-- this is stupid, there must be a better concat + prepend way, but I couldn't seem to find it. -->
<replaceregexp file="script/soundmanager2-nodebug.js" match="(.*+)" flags="s" replace="${sm2_license_jslint}\1" />
<replaceregexp file="script/soundmanager2-nodebug.js" match="(.*+)" flags="s" replace="${sm2_license}\1" />
<replaceregexp file="script/soundmanager2-nodebug-jsmin.js" match="(.*+)" flags="s" replace="${sm2_license}\1" />

<!-- show size -->
Expand Down

2 comments on commit 72b365f

@scottschiller
Copy link
Owner Author

Choose a reason for hiding this comment

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

Ha, I probably did something dumb here. Thanks. I haven't updated this in an ice age or two. 😅

@schutzstaffel911
Copy link

Choose a reason for hiding this comment

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

can you show me please i'll understand better,i dont mean to be a bug to you but your really smart at this and im just new its all so confusing at times

Please sign in to comment.