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

ci: update PGI build (old one no longer signed) #4260

Merged
merged 6 commits into from
Oct 20, 2022

Conversation

rwgk
Copy link
Collaborator

@rwgk rwgk commented Oct 20, 2022

Description

WIP

@rwgk
Copy link
Collaborator Author

rwgk commented Oct 20, 2022

Active help from someone with a vested interest in this platform will be highly appreciated.

Strictly speaking, my only interest is to not have to scroll through the CI results for each CI run, only to be sure it's the PGI failure and not something else. I'm happy to help out, but I just don't know anything about PGI versions, I'm not the best person to fix this.

@henryiii henryiii mentioned this pull request Oct 20, 2022
@henryiii
Copy link
Collaborator

henryiii commented Oct 20, 2022

Someone with vested interest or knowledge should look into moving our old CentOS (commented out) test to Alma8. I'd do it but it uses weird download URLs with dates in them; not sure if there's now a better way, more like CentOS 7.

(Or Rocky linux 8, they list that here: https://docs.nvidia.com/hpc-sdk/hpc-sdk-release-notes/index.html )

@rwgk
Copy link
Collaborator Author

rwgk commented Oct 20, 2022

0cba2ce led to this error:

/opt/nvidia/hpc_sdk/Linux_x86_64/22.9/compilers/share/llvm/bin/opt: /tmp/nvc++FzpDM9eezJ0.ll:104547:121: error: invalid redefinition of function '_ZNSt10_Head_baseILm0EDTcl6__testIN8pybind118nodeleteEELi0EEELb0EE7_M_headERS3_'
define linkonce_odr %struct._ZN71_GLOBAL__N__47____w_pybind11_pybind11_tests_test_smart_ptr_cpp_3177280610MyObject4aE** @_ZNSt10_Head_baseILm0EDTcl6__testIN8pybind118nodeleteEELi0EEELb0EE7_M_headERS3_(%struct._ZSt10_Head_baseILm0EDTcl6__testIN8pybind118nodeleteEELi0EEELb0EE.1* %__b.arg) #0 inlinehint mustprogress !dbg !51945 {
                                                                                                                        ^

@henryiii henryiii changed the title WIP Unbreak failing PGI build ci: update PGI build (old one no longer signed) Oct 20, 2022
@henryiii henryiii marked this pull request as ready for review October 20, 2022 12:49
@henryiii henryiii merged commit 17c6809 into pybind:master Oct 20, 2022
@github-actions github-actions bot added the needs changelog Possibly needs a changelog entry label Oct 20, 2022
henryiii pushed a commit to henryiii/pybind11 that referenced this pull request Oct 20, 2022
* Simply replace "22.3" with "22.9" to see what happens.

* Remove PYBIND11_TEST_FILTER to see what happens.

* Revert "Remove PYBIND11_TEST_FILTER to see what happens."

This reverts commit 0cba2ce.

* Remove only test_smart_ptr.cpp to see what happens.

* Revert "Remove only test_smart_ptr.cpp to see what happens."

This reverts commit 8e9df22.

* Remove only test_virtual_functions.cpp to see what happens.
@henryiii henryiii removed the needs changelog Possibly needs a changelog entry label Oct 20, 2022
EthanSteinberg pushed a commit to EthanSteinberg/pybind11 that referenced this pull request Oct 20, 2022
* Simply replace "22.3" with "22.9" to see what happens.

* Remove PYBIND11_TEST_FILTER to see what happens.

* Revert "Remove PYBIND11_TEST_FILTER to see what happens."

This reverts commit 0cba2ce.

* Remove only test_smart_ptr.cpp to see what happens.

* Revert "Remove only test_smart_ptr.cpp to see what happens."

This reverts commit 8e9df22.

* Remove only test_virtual_functions.cpp to see what happens.
@rwgk rwgk deleted the pgi_update branch October 21, 2022 03:54
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.

2 participants