This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 490965 - Split pldt.openacc into core and UI (and CLI)
Summary: Split pldt.openacc into core and UI (and CLI)
Status: RESOLVED FIXED
Alias: None
Product: PTP
Classification: Tools
Component: PLDT (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 9.1   Edit
Assignee: Jeffrey Overbey CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-04-03 11:01 EDT by Jeffrey Overbey CLA
Modified: 2016-06-02 11:01 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jeffrey Overbey CLA 2016-04-03 11:01:10 EDT 
Per a discussion on ptp-dev, we need to split the pldt.openacc plug-in into two plug-ins--core and UI--before contributing OpenACC refactoring support.  We also want to add code for running refactorings from the command line; it might make sense to put this code in a separate CLI plug-in, since it cannot/should not be compiled into the Eclipse plug-in.
Comment 1 Eclipse Genie CLA 2016-04-05 12:10:10 EDT 
New Gerrit change created: https://git.eclipse.org/r/69937
Comment 2 Eclipse Genie CLA 2016-04-05 13:19:25 EDT 
New Gerrit change created: https://git.eclipse.org/r/69952
Comment 3 Eclipse Genie CLA 2016-04-12 19:32:12 EDT 
New Gerrit change created: https://git.eclipse.org/r/70527
Comment 4 Eclipse Genie CLA 2016-04-12 19:32:14 EDT 
New Gerrit change created: https://git.eclipse.org/r/70526
Comment 5 Eclipse Genie CLA 2016-04-12 19:32:15 EDT 
New Gerrit change created: https://git.eclipse.org/r/70525
Comment 6 Eclipse Genie CLA 2016-04-12 19:45:06 EDT 
New Gerrit change created: https://git.eclipse.org/r/70528
Comment 7 Jeffrey Overbey CLA 2016-04-12 21:14:00 EDT 
Trying again to squash this into a single change...
Comment 8 Eclipse Genie CLA 2016-04-12 21:16:12 EDT 
New Gerrit change created: https://git.eclipse.org/r/70529
Comment 10 Greg Watson CLA 2016-04-14 19:51:53 EDT 
You should really have a Signed-off-by: line in the commit message. I'm not sure why it accepted the change without one...
Comment 11 Jeffrey Overbey CLA 2016-04-14 23:24:35 EDT 
Hmm, I was following the command line instructions at https://wiki.eclipse.org/Gerrit with the commit-msg hook <https://wiki.eclipse.org/Gerrit#Gerrit_Code_Review_Cheatsheet>.  I guess it generates the Change ID but not the other.
Comment 12 Greg Watson CLA 2016-04-14 23:32:45 EDT 
Ah, well if you'd used Eclipse then it would have done it for you...