Bug 337655 - Move to use license (EUA) files/technique
Summary: Move to use license (EUA) files/technique
Status: RESOLVED FIXED
Alias: None
Product: JSDT
Classification: WebTools
Component: General (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows 7
: P3 blocker (vote)
Target Milestone: 3.3 M6   Edit
Assignee: Nitin Dahyabhai CLA
QA Contact: Nitin Dahyabhai CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 336519
  Show dependency tree
 
Reported: 2011-02-20 01:26 EST by David Williams CLA
Modified: 2011-03-03 13:25 EST (History)
0 users

See Also:


Attachments
patch for changes required in jsdt features (145.02 KB, patch)
2011-02-20 01:26 EST, David Williams CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Williams CLA 2011-02-20 01:26:15 EST
Created attachment 189354 [details]
patch for changes required in jsdt features

+++ This bug was initially created as a clone of Bug #336519 +++

There is a new Eclipse license (EUA) that includes a new line about EDL. 

And, rather than do a "mass file/property update" we should move to the new "PDE build" technique to reference the license file, which is then included at build-time, so future updates are easier. This issue is discussed in Kim's blog post; 
http://relengofthenerds.blogspot.com/2011/01/implementing-shared-licenses-with-37m5.html

I'll try a few test features ... say, xml features, just to make sure it works as expected then provide patches/instructions for a mass update.
Comment 1 David Williams CLA 2011-03-03 13:04:03 EST
I appears this change for licensing got partially committed, but not entirely. 

These attributes were released: 

 license-feature="org.eclipse.license"
      license-feature-version="1.0.0.qualifier" 

But none of the other changes ... so, ends up with "conflicts in property files" and causes the build to fail. 

Can you sort it out? Or want a new patch?
Comment 2 Nitin Dahyabhai CLA 2011-03-03 13:12:41 EST
I can sort it out.  Should just be a few minutes.
Comment 3 Nitin Dahyabhai CLA 2011-03-03 13:25:00 EST
That should do it.