[bug#72878] [PATCH] gnu: plasma-nm: Fix build on aarch64

[bug#72878] [PATCH] gnu: plasma-nm: Fix build on aarch64

[Doğan Çeçen]

* gnu/packages/qt.scm (qtcoro-qt6): pass -fPIC explicitly

Change-Id: Ifb26600945548d84df2cc70318781d87253af476
---
 gnu/packages/qt.scm | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
index a91bebf358..974af2a342 100644
--- a/gnu/packages/qt.scm
+++ b/gnu/packages/qt.scm
@@ -153,10 +153,11 @@ (define-public qcoro-qt5
     (build-system qt-build-system)
     (arguments
      (list #:configure-flags
-           #~(list (string-append "-DUSE_QT_VERSION="
+           #~(list "-DCMAKE_CXX_FLAGS=-fPIC"
+                   (string-append "-DUSE_QT_VERSION="
                                   #$(version-major
-                                     (package-version
-                                      (this-package-input "qtbase")))))))
+                                      (package-version
+                                        (this-package-input "qtbase")))))))
     (native-inputs (list dbus))         ;for tests
     (inputs (list qtbase-5 qtdeclarative-5 qtwebsockets-5))
     (home-page "https://qcoro.dvratil.cz/")
@@ -169,6 +170,7 @@ (define-public qcoro-qt6
   (package
     (inherit qcoro-qt5)
     (name "qcoro-qt6")
+    (synopsis "C++ Coroutine Library for Qt6")
     (inputs (modify-inputs (package-inputs qcoro-qt5)
               (replace "qtbase" qtbase)
               (replace "qtdeclarative" qtdeclarative)

base-commit: d5312370b46ace47e138d84e1bb28e5651cee94b
-- 
2.45.2

[bug#72878] [PATCH] gnu: plasma-nm: Fix build on aarch64

[Doğan Çeçen]

Sorry for not adding a little context, I was battling with my email config.

Anyhow, I was getting the following error when installing plasma-nm:

--8<---------------cut here---------------start------------->8---
ld: /gnu/store/drbppxblx1gf943dxilhwz5vags287ir-qcoro-qt6-0.10.0/lib/libQCoro6DBus.a(qcorodbuspendingcall.cpp.o): relocation R_AARCH64_ADR_PREL_PG_HI21 against symbol `_ZN23QDBusPendingCallWatcher8finishedEPS_@@Qt_6' which may bind externally can not be used when making a shared object; recompile with -fPIC
ld: /gnu/store/drbppxblx1gf943dxilhwz5vags287ir-qcoro-qt6-0.10.0/lib/libQCoro6DBus.a(qcorodbuspendingcall.cpp.o)(.text+0xb0): unresolvable R_AARCH64_ADR_PREL_PG_HI21 relocation against symbol `_ZN23QDBusPendingCallWatcher8finishedEPS_@@Qt_6'
ld: final link failed: bad value
--8<---------------cut here---------------end--------------->8---

The problem seemed to be qcoro-qt6. I checked its logs on x86_64, and it seems -fPIC is being passed:

https://ci.guix.gnu.org/build/5466920/details

Unfortunately, aarch64 build is still Scheduled:

https://ci.guix.gnu.org/build/5454563/details

Anyhow, the suggestion that the linker gave has worked, which is adding -fPIC.

[bug#72878] [PATCH] gnu: plasma-nm: Fix build on aarch64

[Z572]

[-- Attachment #1: Type: text/plain, Size: 1856 bytes --]

Doğan Çeçen <sepeth@fastmail.com> writes:

Title should "gnu: qcoro-qt: Build with -fPIC.", because it change qcoro-qt{5,6}.

> * gnu/packages/qt.scm (qtcoro-qt6): pass -fPIC explicitly

* gnu/packages/qt.scm (qcoro-qt5, qcoro-qt6)[arguments]: Adjust
  configure-flags to add -fPIC to CMAKE_CXX_FLAGS.
  
>
> Change-Id: Ifb26600945548d84df2cc70318781d87253af476
> ---
>  gnu/packages/qt.scm | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
> index a91bebf358..974af2a342 100644
> --- a/gnu/packages/qt.scm
> +++ b/gnu/packages/qt.scm
> @@ -153,10 +153,11 @@ (define-public qcoro-qt5
>      (build-system qt-build-system)
>      (arguments
>       (list #:configure-flags
> -           #~(list (string-append "-DUSE_QT_VERSION="
> +           #~(list "-DCMAKE_CXX_FLAGS=-fPIC"
> +                   (string-append "-DUSE_QT_VERSION="
>                                    #$(version-major
> -                                     (package-version
> -                                      (this-package-input "qtbase")))))))
> +                                      (package-version
> +                                        (this-package-input "qtbase")))))))
>      (native-inputs (list dbus))         ;for tests
>      (inputs (list qtbase-5 qtdeclarative-5 qtwebsockets-5))
>      (home-page "https://qcoro.dvratil.cz/")
> @@ -169,6 +170,7 @@ (define-public qcoro-qt6
>    (package
>      (inherit qcoro-qt5)
>      (name "qcoro-qt6")
> +    (synopsis "C++ Coroutine Library for Qt6")

please split to a new patch.

>      (inputs (modify-inputs (package-inputs qcoro-qt5)
>                (replace "qtbase" qtbase)
>                (replace "qtdeclarative" qtdeclarative)
>
> base-commit: d5312370b46ace47e138d84e1bb28e5651cee94b

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

[bug#72878] [PATCH] gnu: qcoro-qt: Build with -fPIC

[Doğan Çeçen]

* gnu/packages/qt.scm (qcoro-qt5, qcoro-qt6)[arguments]: Adjust
  configure-flags to add -fPIC to CMAKE_CXX_FLAGS.

Change-Id: I5ed2ee18cec120b282863ce6a5f6b6b149059515
---
 gnu/packages/qt.scm | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
index b99ce365d7..635fc37639 100644
--- a/gnu/packages/qt.scm
+++ b/gnu/packages/qt.scm
@@ -153,10 +153,11 @@ (define-public qcoro-qt5
     (build-system qt-build-system)
     (arguments
      (list #:configure-flags
-           #~(list (string-append "-DUSE_QT_VERSION="
+           #~(list "-DCMAKE_CXX_FLAGS=-fPIC"
+                   (string-append "-DUSE_QT_VERSION="
                                   #$(version-major
-                                     (package-version
-                                      (this-package-input "qtbase")))))))
+                                      (package-version
+                                        (this-package-input "qtbase")))))))
     (native-inputs (list dbus))         ;for tests
     (inputs (list qtbase-5 qtdeclarative-5 qtwebsockets-5))
     (home-page "https://qcoro.dvratil.cz/")
@@ -169,6 +170,7 @@ (define-public qcoro-qt6
   (package
     (inherit qcoro-qt5)
     (name "qcoro-qt6")
+    (synopsis "C++ Coroutine Library for Qt6")
     (inputs (modify-inputs (package-inputs qcoro-qt5)
               (replace "qtbase" qtbase)
               (replace "qtdeclarative" qtdeclarative)

base-commit: ad7211cfe0cba1b7a97308c0fec941c807c1c1e5
-- 
2.45.2

bug#72878: [PATCH] gnu: qcoro-qt: Build with -fPIC

[Zheng Junjie]

[-- Attachment #1: Type: text/plain, Size: 1720 bytes --]

Doğan Çeçen <sepeth@fastmail.com> writes:

> * gnu/packages/qt.scm (qcoro-qt5, qcoro-qt6)[arguments]: Adjust
>   configure-flags to add -fPIC to CMAKE_CXX_FLAGS.
>
> Change-Id: I5ed2ee18cec120b282863ce6a5f6b6b149059515
> ---
>  gnu/packages/qt.scm | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
> index b99ce365d7..635fc37639 100644
> --- a/gnu/packages/qt.scm
> +++ b/gnu/packages/qt.scm
> @@ -153,10 +153,11 @@ (define-public qcoro-qt5
>      (build-system qt-build-system)
>      (arguments
>       (list #:configure-flags
> -           #~(list (string-append "-DUSE_QT_VERSION="
> +           #~(list "-DCMAKE_CXX_FLAGS=-fPIC"
> +                   (string-append "-DUSE_QT_VERSION="
>                                    #$(version-major
> -                                     (package-version
> -                                      (this-package-input "qtbase")))))))
> +                                      (package-version
> +                                        (this-package-input "qtbase")))))))
>      (native-inputs (list dbus))         ;for tests
>      (inputs (list qtbase-5 qtdeclarative-5 qtwebsockets-5))
>      (home-page "https://qcoro.dvratil.cz/")
> @@ -169,6 +170,7 @@ (define-public qcoro-qt6
>    (package
>      (inherit qcoro-qt5)
>      (name "qcoro-qt6")
> +    (synopsis "C++ Coroutine Library for Qt6")

split it on new commit

>      (inputs (modify-inputs (package-inputs qcoro-qt5)
>                (replace "qtbase" qtbase)
>                (replace "qtdeclarative" qtdeclarative)
>
> base-commit: ad7211cfe0cba1b7a97308c0fec941c807c1c1e5


push, close.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]