Bug 522560 - [cg] Non-standard JET templates
Summary: [cg] Non-standard JET templates
Status: NEW
Alias: None
Product: QVTd
Classification: Modeling
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows NT
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 485089 485764
Blocks:
  Show dependency tree
 
Reported: 2017-09-21 04:25 EDT by Ed Willink CLA
Modified: 2017-09-21 05:37 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ed Willink CLA 2017-09-21 04:25:13 EDT
This bug traces the progress in removing non-standard JET templates.

Currently /org.eclipse.qvtd.compiler/templates/model/*.jet has fixes for Bug 485089 and Bug 485764.

EMF 2.14M2 has a fix for Bug 485764, so that could be removed iff we raise the EMF lowerbound to 2.14. But we still need Bug 485089 so we might as well retain Oxygen compatibility during Photon development.
Comment 1 Ed Willink CLA 2017-09-21 05:37:40 EDT
In order to incorporate the other EMF 2.14M2 whitespace changes the GIT history shows commits for:

- the improved QVTd files (up to 2017-09-21)
- the original EMF files as at 2016-04-07
- the improved QVTd files again as a change wrt the original EMF
- the new EMF files as at 2017-09-21
- the fixed cherry-picked change

---

EMF 2.14 via Bug 428088 introduces new API to support @deprecated tags. This GenModel API is not available in Oxygen/EMF 2.13 and since it is at JET time it does not need to be guarded by the prevailing run-time level. We must therefore decide:

a) suppress the 2.14 @deprecated support for now
b) falsely claim to run on 2.13 - users get CG failures
c) redirect the new @deprecated support to stubs on EMF 2.13
d) continue unchanged

There are about 80 redirections to about 5 new API methods. Possible but tedious and a significant amount to review/fixup for the next change.

Suggest: stay as is until M6, then impose an EMF 2.14 lowerbound.

(Impact the temporarily missing @deprecated support is for autogenerated QVTd models; will anyone care. Indeed what does the new support actually do?)

Branch ewillink/522560 shows the history awaiting exploitation at M6.