-
-
Notifications
You must be signed in to change notification settings - Fork 101
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
Compiler issues on newly reinstalled test-osuosl-aix72-ppc64-1 #2334
Comments
Does sound like a
Hmmm I've got a feeling we've seen that one before ... May be the same as #1320 although annoyingly I seemingly didn't cut & paste the error message in there. |
See also things like adoptium/temurin-build#1450 (comment) which mentions |
While using the correct PATH variable, there still seems to be a problem with the jdk16 build
Digging around, some sources seem to attribute this to a |
I thought the playbooks did user creation/verification: new build:
older system:
|
Looks like it is
|
I did run the entire AIX playbook on ojdk03. Which role in the playbooks should have fixed this? If there isn't already something in the playbooks that should fix this, I'll add a line that changes |
This is where is it has historically been done.
And, iirc, this is processed/managed via: p.s. it is a |
I see. My bad, what I meant by 'entire playbook' is I often miss that role as it was suggested to me long ago by @sej-jackson that the script's, |
Yes, that was the first issue I opened, and due to many misunderstandings on my part (on how ansible works) plus we have moved on, I dropped it as not resolvable by me. Perhaps I should start over again. :) |
I've no idea why the |
That is LONG before my time. Probably because it was easier to add to a script (could it have even pre-dated the use of ansible?) than meet the requirements for ansible for command execution. That was one of the reasons (i.e., more than just filesystem commands) I was triggered to open a issue #1547 about that file. |
is this still open? |
@Haroon-Khel What is the status of this - do we still have compiler issues on newly installed 7.2 machines? |
The only 'issue' is that the compiler on adopt08 is more strict - as it is patched - than the compiler (GA version, no patches) - as far as xlCv16 is concerned. Both are functioning as 'jenkins' nodes, and imho - this issue may be closed. |
This was probably the Still think it can be closed :p |
OK. Found it.
|
Just to be clear from your last comment - is the above statement still true, or is that xlc include file that's trying to use From an adopt perspective if it's only happening in the main openjdk build jobs and not when building the tests I'd be ok with not resolving this one (since we don't do any of the regular builds with AIX 7.2) ... Although there is an argument for querying it with the compiler team too as you say. |
The above statement is true on adopt03, which we had upgraded the OS to AIX 7.2 TL4.
|
FYI: I found an AIX 7.2, TL3 system, and it has the function in it's rte as well: (ie, in the include files, so I expect it in the libraries)
|
|
OK I've re-enabled |
And close in the morning :) |
success:
|
p.s. - only today changing the label on |
Building jdk16 I encountered this error
This may be accounted for by an improper PATH variable while building so I'll re test this, but I also his this error while building jdk11
The text was updated successfully, but these errors were encountered: