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

Adding Power support(ppc64le) with continuous integration/testing so that project stays architecture independent. #31

Closed
wants to merge 1 commit into from

Conversation

asellappen
Copy link

Please review and merge the changes ,This is part of the Ubuntu distribution for ppc64le.
This helps us simplify testing later when distributions are re-building and re-releasing. For more info tag @gerrith3.

@akavel
Copy link
Owner

akavel commented Oct 6, 2020

I don't understand the purpose of this PR, can you please explain it better to me? Until that, I'm marking this as invalid [1].

@akavel akavel added the invalid label Oct 6, 2020
@asellappen
Copy link
Author

asellappen commented Oct 7, 2020 via email

@gerrith3
Copy link

gerrith3 commented Oct 7, 2020

@asellappen no need to quote the entire email - all of that info is in the thread here. ;)

@akavel I realize this is/should be a pure go implementation but 1) there is a small chance of a deviation in go itself between Intel and ppc64le. Not likely, but not ideal. Being able to catch any projects which do have a difference as early as possible is a good thing. Second, we have found go projects which use assembly level call outs for optimization or reaches out to fixed location intel based binaries to supplement their work. View this is a simple sweep to catch either of those cases. They should be rare/unlikely (but also so should be any defects in general ;-) but since the resources are free, the work to do the testing is all included here, and the time to do the testing is free/run in parallel by Travis, there shouldn't be any downsides to this patch.

@asellappen
Copy link
Author

@akavel ,Hope the above clarification would be sufficient and pls let us know if you need further,

@asellappen
Copy link
Author

@akavel ,,any update on this request ?

@akavel
Copy link
Owner

akavel commented Oct 22, 2020

@gerrith3 the original submission contained wording: "This is part of the Ubuntu distribution for ppc64le." - what is "This" meaning there? Also, can you please give some proof as to this statement ("part of the Ubuntu distribution for ppc64le") - e.g. some link that could convince me this is true? Also, are you planning to check somehow if the builds passed before distributing, or is it intended only to softly warn me that I might be breaking your stuff? Sorry for the hassle, but given "hacktober", weird and low effort PRs out of the blue, esp. with crappy grammar & wording, unfortunately require much more scrutiny and the proof's burden is on the authors at this time.

@asellappen asellappen changed the title Add poweron architecture ppc64le to travis build Adding Power support(ppc64le) with continuous integration/testing so that project stays architecture independent. Nov 2, 2020
@akavel akavel closed this in 3e78fc3 Dec 6, 2020
@gerrith3
Copy link

gerrith3 commented Dec 8, 2020

@akavel is this what you are looking for? You can also look at the Ubuntu Universe manifest and probably at Debian as well.

ubuntu@gerrit:~$ sudo apt-cache search rsrc
golang-github-akavel-rsrc-dev - Go library for embedding binary resources in executables
ubuntu@gerrit:~$ uname -a
Linux gerrit 5.4.0-54-generic #60-Ubuntu SMP Fri Nov 6 10:37:33 UTC 2020 ppc64le ppc64le ppc64le GNU/Linux

@gerrith3
Copy link

gerrith3 commented Dec 8, 2020

And btw, that is a stock Linux install, no additional repositories added.

@akavel
Copy link
Owner

akavel commented Dec 11, 2020

As to what I'm looking for, I'd really be grateful for some link on the internet on the Ubuntu website where I could verify this. You seem to mention an "Ubuntu Universe manifest" - I don't know what it is, but if it's something viewable on the internets on ubuntu website, is there a chance you might point me with a link to where rsrc shows up in it? You seem to be familiar with it, so you might be able to generate such a link much easier than if I started to research what "Ubuntu Universe manifest" is.

Unfortunately, "hacktober" is a difficult month for contributions (as I explained above), and this contribution was started in a super low quality way, giving the air of the worst among hacktober offenders (believe me, I've seen my number of them). I.e. a weird, low effort contribution out of the blue, with some wild unsubstantated and unproven claims, and poor writing style as a cherry on top. Since then I still haven't seen any "hard proof" in this thread, that would be difficult to fake, showing it's not one of the hacktober bad apples. However hacktober is slowly waning away in the past, the dust is starting to settle, and you at least use grammar & interpunction which soothes my nerves. So I extended my good will and assumed there's nonzero chance you might be up to something, so for as long as this doesn't seem to impediment me, I added the target to Travis in meantime. I took care to try and do it in a way that would make sure this PR won't count to "hacktober".

Other than the link that I'm asking for again, there's still one more question that I posed in the previous comment that you didn't answer yet:

Also, are you planning to check somehow if the builds passed before distributing, or is it intended only to softly warn me that I might be breaking your stuff?

That question is related specifically to your mention about:

Being able to catch any projects which do have a difference as early as possible [...]

I'd like to understand in what way you (?) are "catching any projects which do have a difference"? And how you observe "a difference"? And based on that, in what way is adding a ppe64le target in travis in this project related to "being able to catch any projects which do have a difference"?

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

Successfully merging this pull request may close these issues.

3 participants