Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[JENKINS-37430] Add support for ppc64le #1

Closed
wants to merge 2 commits into from

Conversation

kuisathaverat
Copy link

@kuisathaverat kuisathaverat commented Nov 2, 2016

JENKINS-37430

@reviewbybees
it adds support for Power8 platform, these are the commits related with the PR

jnr/jffi@7e22759
jnr/jnr-unixsocket@767ed30
jnr/jnr-ffi@6d8cdc0

@kuisathaverat
Copy link
Author

@stephenc WDYT about this upgrade of plugins to support ppc64le

@stephenc
Copy link
Member

stephenc commented Nov 7, 2016

Did you check if Jenkins has been updated to match?

This is somewhat similar to the bouncycastle issue

@kuisathaverat
Copy link
Author

kuisathaverat commented Nov 7, 2016

@stephenc in core there are a previous version of jnr-posix, I am going to review if it works without touch it in both sides.

<dependency>
      <groupId>com.github.jnr</groupId>
      <artifactId>jnr-posix</artifactId>
      <version>3.0.1</version>
</dependency>

keep  jnr-posix same version of core
@kuisathaverat
Copy link
Author

@stephenc I tested that we can keep jnr-posix on same version of core

@kuisathaverat
Copy link
Author

kuisathaverat commented Nov 17, 2016

@reviewbybees Please, if someone can review it will be appreciated

@abayer
Copy link

abayer commented Dec 19, 2016

🐝

@oleg-nenashev
Copy link

Would be nice to see the full list of changes in libs.
I suppose we pick much more than a particular fix. Regarding the custom build of 1.0.3.1-jenkins-20276, we should ensure that the all fixes from this custom build are integrated to the upstream

@jglick
Copy link
Member

jglick commented Dec 19, 2016

we should ensure that the all fixes from this custom build are integrated to the upstream

Right, I think that is the main task.

@kuisathaverat
Copy link
Author

@jglick @oleg-nenashev AFAIT by the last comment in jnr/jnr-ffi#23, JENKINS-20276 it is not longer an issue on this library

@kuisathaverat kuisathaverat changed the title Add support for ppc64le [JENKINS-37430] Add support for ppc64le Sep 6, 2017
@kuisathaverat
Copy link
Author

@reviewbybees done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants