This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 512224 - Use PPC64 and ppc64 LE machines from redhat
Summary: Use PPC64 and ppc64 LE machines from redhat
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.7   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.7 M7   Edit
Assignee: Sravan Kumar Lakkimsetti CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 512227 512228 512282
Blocks:
  Show dependency tree
 
Reported: 2017-02-15 09:17 EST by Sravan Kumar Lakkimsetti CLA
Modified: 2017-04-10 14:52 EDT (History)
6 users (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 Sravan Kumar Lakkimsetti CLA 2017-02-15 09:17:29 EST 
use ppc64 and ppc64LE machines from redhat
Comment 1 Eclipse Genie CLA 2017-02-16 03:42:58 EST 
New Gerrit change created: https://git.eclipse.org/r/91248
Comment 3 Eclipse Genie CLA 2017-02-16 06:17:25 EST 
New Gerrit change created: https://git.eclipse.org/r/91272
Comment 5 Eclipse Genie CLA 2017-02-16 06:47:58 EST 
New Gerrit change created: https://git.eclipse.org/r/91273
Comment 7 Alexander Kurtakov CLA 2017-02-22 06:51:17 EST 
No further work is expected here AFAIK. Can we close this one?
Comment 8 Sravan Kumar Lakkimsetti CLA 2017-02-22 07:01:05 EST 
(In reply to Alexander Kurtakov from comment #7)
> No further work is expected here AFAIK. Can we close this one?

Launcher changes are still pending. Lets close this after launcher changes also done
Comment 9 Leo Ufimtsev CLA 2017-03-06 15:53:48 EST 
The patch (697d67ccbde294e1ea488f55efbf6047ebce6ed9) that changed build.sh hard-codes a path to the java on the machine:
export JAVA_HOME=" /usr/lib/jvm/java-1.8.0-openjdk-1.8.0.121-1.b14.fc24.ppc64"

This will break if java is updated. Instead I suggest the javapath is found dynamically. For example something like:

> # Cross-platform method of finding JAVA_HOME.
> # Tested on Fedora 24 and Ubuntu 16
> JAVA_HOME=`readlink -f /usr/bin/java | sed "s:jre/bin/java::"`
(See build.sh)

The above works reliably. It has worked on my machine all the way from Fedora 24 up to newest Fedora 25 versions.
Comment 10 Leo Ufimtsev CLA 2017-03-28 10:42:43 EDT 
@Sravan, 

I'd be happy to submit the relevant patch, if I do, will you be able to test it?
Comment 11 Sravan Kumar Lakkimsetti CLA 2017-03-30 02:45:36 EDT 
(In reply to Leo Ufimtsev from comment #10)
> @Sravan, 
> 
> I'd be happy to submit the relevant patch, if I do, will you be able to test
> it?

That should be good. I can do the test for you. Thanks
Comment 12 Eclipse Genie CLA 2017-04-03 10:43:49 EDT 
New Gerrit change created: https://git.eclipse.org/r/94306
Comment 13 Leo Ufimtsev CLA 2017-04-03 10:44:31 EDT 
(In reply to Sravan Kumar Lakkimsetti from comment #11)
> (In reply to Leo Ufimtsev from comment #10)
> > @Sravan, 
> > 
> > I'd be happy to submit the relevant patch, if I do, will you be able to test
> > it?
> 
> That should be good. I can do the test for you. Thanks

(In reply to Eclipse Genie from comment #12)
> New Gerrit change created: https://git.eclipse.org/r/94306

Here is the patch. Is it possible for you to test it before we merge it?
(Or does it need to be merged to test?)

Thank you.
Comment 15 Alexander Kurtakov CLA 2017-04-06 14:09:33 EDT 
Sravan, can we close this one and all the dependent ones?
Comment 16 Sravan Kumar Lakkimsetti CLA 2017-04-07 02:09:45 EDT 
Yes this can be closed now