r.js not preserve comments

[outaTiME]

Hi guys,

i making some test cases and its imposible to preserve the comments from source files, in need to include my copyright info at the begining of file ...

take a look ...

  modules: [
    {
      name: "foo", // merge requirejs with foo file from src folder
      include: ["../vendor/requirejs/require", "foo"],
      create: true
    }
  ]

note: in my foo.js file from source folder we have an comment like ...

/*! wooooot copyright ... fooo ... blaaa ... 2011 */

i doing something wrong ??!?!

thks !

[jrburke]

Thanks for pointing this out. It was a bug. I updated the optimizer to preserve license comments. You can try r-edge.js if you want to confirm the fix.

This fix will be part of the 1.0.2 release that I hope to do very soon this week, perhaps even tonight.

[outaTiME]

Tonight, i will try …

thks for your speedy answer =)

Ariel Falduto
afalduto@gmail.com

On Nov 21, 2011, at 8:03 PM, James Burke wrote:

Thanks for pointing this out. It was a bug. I updated the optimizer to preserver license comments. You can try r-edge.js if you want to confirm the fix.

This fix will be part of the 1.0.2 release that I hope to do very soon this week, perhaps even tonight.

---

Reply to this email directly or view it on GitHub:
#41 (comment)

[geddski]

oh good, I was just wondering about this exact thing.

[geddski]

Hey, could this be made into a build profile option? (on by default) With how many libs I'm using I may just want to have main.js link to a separate license text file that contains all the licenses I'm using.

[jrburke]

@GEDDesign: good point. I opened issue #42 to track that request and targeted it for a 1.0.3 release, but I do not have a release target for it yet.

[geddski]

Cool, thanks.

[gjtorikian]

Sorry to bump this, but is there any way to remove license comments completely? I have 600 lines of license comments that I want gone.

[jrburke]

@gjtorikian setting preserveLicenseComments: false should work: https://github.com/jrburke/r.js/blob/master/build/example.build.js#L316