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

sys-libs/liburing: pass --cxx parameter to configure #24733

Closed
wants to merge 1 commit into from

Conversation

jepio
Copy link
Contributor

@jepio jepio commented Mar 24, 2022

This is necessary to fix cross-compilation of liburing, as cxx (default g++) is used for linking applications that are built.

Bug: https://bugs.gentoo.org/836316

@gentoo-bot gentoo-bot added need assignment It was impossible to assign the PR correctly. Please assign it manually. no bug found No Bug/Closes found in the commits. no signoff One or more commits do not indicate GCO sign-off. labels Mar 24, 2022
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-03-24 11:36 UTC
Newest commit scanned: 7dde715
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/e27bb6520f/output.html

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-03-24 12:06 UTC
Newest commit scanned: 17010b2
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/018bc1deb7/output.html

Provide the correct value to --cxx to fix cross-compilation. liburing uses the
value of the cxx variable for linking. This happens to work for native builds
but breaks when cross-compiling, as evidenced by the below output (from
Flatcar):

>>> Configuring source in /build/arm64-usr/var/tmp/portage/sys-libs/liburing-2.1-r1/work/liburing-2.1 ...
 * .arm64: running multilib-minimal_abi_src_configure
prefix                        /usr
includedir                    /usr/include
libdir                        /usr/lib64
libdevdir                     /usr/lib64
relativelibdir
mandir                        /usr/share/man
datadir                       /usr/share
stringop_overflow             yes
array_bounds                  yes
__kernel_rwf_t                yes
__kernel_timespec             yes
open_how                      no
statx                         yes
C++                           yes
has_ucontext                  yes
CC                            aarch64-cros-linux-gnu-gcc
CXX                           g++

...

aarch64-cros-linux-gnu-gcc -D_GNU_SOURCE -D__SANE_USERSPACE_TYPES__ -I../src/include/ -include ../config-host.h -O2 -pipe -mtune=generic -g -D_GNU_SOURCE -Wall -Wextra -Wno-unused-parameter -Wno-sign-compare -L../src/ -o statx statx.c helpers.o -luring -Wstringop-overflow=0 -Warray-bounds=0
g++ -D_GNU_SOURCE -D__SANE_USERSPACE_TYPES__ -I../src/include/ -include ../config-host.h -O2 -pipe -mtune=generic -g -O2 -pipe -mtune=generic -g -D_GNU_SOURCE -Wall -Wextra -Wno-unused-parameter -Wno-sign-compare -L../src/ -std=c++11 -o sq-full-cpp sq-full-cpp.cc helpers.o -luring -Wstringop-overflow=0 -Warray-bounds=0
/usr/lib/gcc/x86_64-pc-linux-gnu/10.3.0/../../../../x86_64-pc-linux-gnu/bin/ld: helpers.o: Relocations in generic ELF (EM: 183)
/usr/lib/gcc/x86_64-pc-linux-gnu/10.3.0/../../../../x86_64-pc-linux-gnu/bin/ld: helpers.o: Relocations in generic ELF (EM: 183)
/usr/lib/gcc/x86_64-pc-linux-gnu/10.3.0/../../../../x86_64-pc-linux-gnu/bin/ld: helpers.o: Relocations in generic ELF (EM: 183)
/usr/lib/gcc/x86_64-pc-linux-gnu/10.3.0/../../../../x86_64-pc-linux-gnu/bin/ld: helpers.o: Relocations in generic ELF (EM: 183)
/usr/lib/gcc/x86_64-pc-linux-gnu/10.3.0/../../../../x86_64-pc-linux-gnu/bin/ld: helpers.o: Relocations in generic ELF (EM: 183)
/usr/lib/gcc/x86_64-pc-linux-gnu/10.3.0/../../../../x86_64-pc-linux-gnu/bin/ld: helpers.o: Relocations in generic ELF (EM: 183)
/usr/lib/gcc/x86_64-pc-linux-gnu/10.3.0/../../../../x86_64-pc-linux-gnu/bin/ld: helpers.o: error adding symbols: file in wrong format
collect2: error: ld returned 1 exit status
make[1]: *** [Makefile:165: sq-full-cpp] Error 1
make[1]: *** Waiting for unfinished jobs....

Bug: https://bugs.gentoo.org/836316
Signed-off-by: Jeremi Piotrowski <jpiotrowski@microsoft.com>
@jepio jepio changed the title sys-libs/liburing: pass --cxx paramter to configure sys-libs/liburing: pass --cxx paramter to configure [please reassign] Mar 28, 2022
@gentoo-bot gentoo-bot changed the title sys-libs/liburing: pass --cxx paramter to configure [please reassign] sys-libs/liburing: pass --cxx paramter to configure Mar 28, 2022
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @jepio
Areas affected: ebuilds
Packages affected: sys-libs/liburing

sys-libs/liburing: polynomial-c

At least one of the listed packages is maintained entirely by non-GitHub developers!

Linked bugs

Bugs linked: 836316


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added need assignment It was impossible to assign the PR correctly. Please assign it manually. bug linked Bug/Closes found in footer, and cross-linked with the PR. and removed need assignment It was impossible to assign the PR correctly. Please assign it manually. no bug found No Bug/Closes found in the commits. no signoff One or more commits do not indicate GCO sign-off. labels Mar 28, 2022
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-03-28 09:18 UTC
Newest commit scanned: 0118735
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/209358dd08/output.html

@jepio jepio changed the title sys-libs/liburing: pass --cxx paramter to configure sys-libs/liburing: pass --cxx parameter to configure Mar 28, 2022
@jepio jepio deleted the liburing-fix-x-compile branch March 29, 2022 06:37
t-lo added a commit to flatcar-archive/portage-stable that referenced this pull request Jun 10, 2022
This change adds ARM64 support for
        dev-libs/ding-libs
        sec-policy/selinux-sssd
        sys-libs/liburing
by un-masking ARM64 architecture.

Also, it bumps liburing-2.1 to upstream -rc2 and ingests a cross
compiler fix (gentoo/gentoo#24733).

Signed-off-by: Thilo Fromm <thilo@kinvolk.io>
t-lo added a commit to flatcar-archive/portage-stable that referenced this pull request Jun 10, 2022
This change bumps liburing-2.1 to upstream -rc2 and ingests a cross
compiler fix (gentoo/gentoo#24733).

Also, it removes the mask for ARM64 in sec-policy/selinux-sssd to match
Gentoo upstream.

Signed-off-by: Thilo Fromm <thilo@kinvolk.io>
t-lo added a commit to flatcar-archive/portage-stable that referenced this pull request Jun 15, 2022
This change bumps liburing-2.1 to upstream -rc2 and ingests a cross
compiler fix (gentoo/gentoo#24733).

Also, it removes the mask for ARM64 in sec-policy/selinux-sssd to match
Gentoo upstream.

Signed-off-by: Thilo Fromm <thilo@kinvolk.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug linked Bug/Closes found in footer, and cross-linked with the PR. need assignment It was impossible to assign the PR correctly. Please assign it manually.
Projects
None yet
3 participants