Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 342292 - [eclipse-build] Ship /usr/bin/efj
Summary: [eclipse-build] Ship /usr/bin/efj
Status: RESOLVED FIXED
Alias: None
Product: Linux Tools
Classification: Tools
Component: eclipse-build (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 0.8.0   Edit
Assignee: Chris Aniszczyk CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 342289
  Show dependency tree
 
Reported: 2011-04-08 10:00 EDT by Andrew Overholt CLA
Modified: 2011-06-23 16:48 EDT (History)
2 users (show)

See Also:

Attachments
current efj used in Fedora (139 bytes, text/plain)
2011-04-08 10:01 EDT, Andrew Overholt CLA 		no flags Details
potential patch (1.22 KB, patch)
2011-04-29 11:40 EDT, Andrew Overholt CLA 		no flags Details | Diff
mylyn/context/zip (975 bytes, application/octet-stream)
2011-04-29 11:40 EDT, Andrew Overholt CLA 		no flags Details
View All 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 Andrew Overholt CLA 2011-04-08 10:00:20 EDT 
Fedora ships a /usr/bin/efj that wraps the JDT's code formatter application.  It would be good to have this in eclipse-build.  I'll attach what we currently ship in Fedora.
Comment 1 Andrew Overholt CLA 2011-04-08 10:01:14 EDT 
Created attachment 192833 [details]
current efj used in Fedora
Comment 2 Andrew Overholt CLA 2011-04-29 11:40:29 EDT 
Created attachment 194372 [details]
potential patch

Here's a possible efj.sh and changes to build.xml to use it.
Comment 3 Andrew Overholt CLA 2011-04-29 11:40:31 EDT 
Created attachment 194373 [details]
mylyn/context/zip
Comment 4 Alexander Kurtakov CLA 2011-05-19 15:49:07 EDT 
Without testing the patch it looks to be doing exactly what's needed.
Comment 5 Andrew Overholt CLA 2011-05-19 16:34:31 EDT 
Committed in 9b8c9b0.

Chris, would you mind cherry-picking that commit onto eclipse37 and seeing if it puts the efj script in your next build?
Comment 6 Chris Aniszczyk CLA 2011-06-23 16:48:47 EDT 
Fixed.