all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Marius Bakke <mbakke@fastmail.com>
To: Hartmut Goebel <h.goebel@crazy-compilers.com>, guix-devel@gnu.org
Subject: Re: [PATCH 2/7] gnu: Add kdelibs4support.
Date: Mon, 21 Nov 2016 18:40:57 +0100	[thread overview]
Message-ID: <87h97091ee.fsf@kirby.i-did-not-set--mail-host-address--so-tickle-me> (raw)
In-Reply-To: <1479744699-27226-3-git-send-email-h.goebel@crazy-compilers.com>

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

Hartmut Goebel <h.goebel@crazy-compilers.com> writes:

> * gnu/packages/kde-frameworks.scm (kdelibs4support): New variable.
> ---
>  gnu/packages/kde-frameworks.scm | 88 +++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 88 insertions(+)
>
> diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
> index 0e6b8fb..a537aa4 100644
> --- a/gnu/packages/kde-frameworks.scm
> +++ b/gnu/packages/kde-frameworks.scm
> @@ -47,6 +47,7 @@
>    #:use-module (gnu packages polkit)
>    #:use-module (gnu packages python)
>    #:use-module (gnu packages qt)
> +  #:use-module (gnu packages tls)
>    #:use-module (gnu packages version-control)
>    #:use-module (gnu packages web)
>    #:use-module (gnu packages xml)
> @@ -2849,3 +2850,90 @@ better integration of Qt applications when running on a KDE Plasma
>  workspace.")
>      ;; triple licensed
>      (license (list license:gpl2+ license:lgpl2.0 license:lgpl2.0+))))
> +
> +;; Porting Aids
> +;;
> +;; Porting Aids frameworks provide code and utilities to ease the transition
> +;; from kdelibs 4 to KDE Frameworks 5. Code should aim to port away from this
> +;; framework, new projects should avoid using these libraries.
> +
> +(define-public kdelibs4support
> +  (package
> +    (name "kdelibs4support")
> +    (version "5.27.0")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append
> +             "mirror://kde/stable/frameworks/"
> +             (version-major+minor version) "/portingAids/"
> +             name "-" version ".tar.xz"))
> +       (sha256
> +        (base32 "17b8d5b9w27251k4r5xc17115nc3k1agv7j7gkmdiybjyilj1n91"))))
> +    (build-system cmake-build-system)
> +    (native-inputs
> +     `(("dbus" ,dbus)
> +       ("docbook-xml" ,docbook-xml-4.4) ;; optional, but not found!

Do you mean it's not found even when present? Perhaps just comment it
out, with a "TODO" or "XXX" explaining what this would add, but the
build system needs some changes to pick it up correctly.

> +       ("extra-cmake-modules" ,extra-cmake-modules)
> +       ("perl", perl)
> +       ("perl-uri" ,perl-uri)
> +       ("pkg-config" ,pkg-config))) ; to find headers of network manager

I think most packagers knows what pkg-config is for, so the explanation
seems unnecessary. Are you sure it's only used for network-manager? I
feel bad for complaining about *too many* comments however :)

> +    (inputs
> +     `(("kcompletion" ,kcompletion)
> +       ("kconfig" ,kconfig)
> +       ("kconfigwidgets" ,kconfigwidgets)
> +       ("kcrash" ,kcrash)
> +       ("kdbusaddons" ,kdbusaddons)
> +       ("kded" ,kded)
> +       ("kdesignerplugin" ,kdesignerplugin)
> +       ("kdoctools" ,kdoctools)
> +       ("kglobalaccel" ,kglobalaccel)
> +       ("kguiaddons" ,kguiaddons)
> +       ("ki18n" ,ki18n)
> +       ("kiconthemes" ,kiconthemes)
> +       ("kio" ,kio)
> +       ("knotifications" ,knotifications)
> +       ("kparts" ,kparts)
> +       ("kservice" ,kservice)
> +       ("ktextwidgets" ,ktextwidgets)
> +       ("kunitconversion", kunitconversion)
> +       ("kwidgetsaddons" ,kwidgetsaddons)
> +       ("kwindowsystem" ,kwindowsystem)
> +       ("kxmlgui" ,kxmlgui)
> +       ("libsm", libsm)
> +       ("networkmanager-qt", networkmanager-qt)
> +       ("openssl", openssl)
> +       ("qtbase" ,qtbase)
> +       ("qtsvg" ,qtsvg)
> +       ("qttools" ,qttools)
> +       ("qtx11extras" ,qtx11extras)))
> +    (arguments
> +     `(#:tests? #f ; FIXME: 6/39 tests fail.
> +       #:phases
> +       (modify-phases %standard-phases
> +         (add-after 'unpack 'cmake-find-docbook
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (substitute* "cmake/FindDocBookXML4.cmake"
> +               (("^.*xml/docbook/schema/dtd.*$")
> +                "xml/dtd/docbook\n"))
> +             #t))

Is this related to the docbook-xml comment in native-inputs? Does this
make it work? I would prefer to have the comment here in that case. E.g.
"This is necessary to find schemas from docbook-xml" or similar.

> +         (add-before 'check 'check-setup
> +           (lambda _
> +             (setenv "HOME" (getcwd))
> +             (setenv "CTEST_OUTPUT_ON_FAILURE" "1") ; Enable debug output
> +             (setenv "QT_QPA_PLATFORM" "offscreen")

Is this variable common in the various KDE build phases? If it's
explained somewhere else, it's probably not necessary to add the same
comment (as I requested in the previous patch) to all of them.

Otherwise LGTM!

> +             #t)))))
> +    (home-page "https://community.kde.org/Frameworks")
> +    (synopsis "KDE Frameworks 5 porting aid from KDELibs4")
> +    (description "This framework provides code and utilities to ease the
> +transition from kdelibs 4 to KDE Frameworks 5.  This includes CMake macros and
> +C++ classes whose functionality has been replaced by code in CMake, Qt and
> +other frameworks.
> +
> +Code should aim to port away from this framework eventually.  The API
> +documentation of the classes in this framework and the notes at
> +http://community.kde.org/Frameworks/Porting_Notes should help with this.")
> +    ;; a variety of licenses
> +    (license (list license:gpl2 license:gpl2+ license:lgpl2.0 license:lgpl2.0+
> +                   license:lgpl2.1+ license:expat license:bsd-2 license:bsd-3
> +                   license:public-domain))))

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

  reply	other threads:[~2016-11-21 17:41 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-21 16:11 [PATCH 0/7] KDE Framework: Tier 4 and porting aids Hartmut Goebel
2016-11-21 16:11 ` [PATCH 1/7] gnu: Add KDE frameworkintegration Hartmut Goebel
2016-11-21 17:25   ` Marius Bakke
2016-11-21 18:01     ` Hartmut Goebel
2016-11-21 18:20       ` Marius Bakke
2016-11-21 18:30         ` Marius Bakke
2016-11-21 16:11 ` [PATCH 2/7] gnu: Add kdelibs4support Hartmut Goebel
2016-11-21 17:40   ` Marius Bakke [this message]
2016-11-21 16:11 ` [PATCH 3/7] gnu: Add khtml Hartmut Goebel
2016-11-21 17:42   ` Marius Bakke
2016-11-21 16:11 ` [PATCH 4/7] gnu: Add kjs Hartmut Goebel
2016-11-21 17:44   ` Marius Bakke
2016-11-21 16:11 ` [PATCH 5/7] gnu: Add kjsembed Hartmut Goebel
2016-11-21 17:46   ` Marius Bakke
2016-11-21 16:11 ` [PATCH 6/7] gnu: Add kmediaplayer Hartmut Goebel
2016-11-21 17:47   ` Marius Bakke
2016-11-21 16:11 ` [PATCH 7/7] gnu: Add kross Hartmut Goebel
2016-11-21 17:50   ` Marius Bakke

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87h97091ee.fsf@kirby.i-did-not-set--mail-host-address--so-tickle-me \
    --to=mbakke@fastmail.com \
    --cc=guix-devel@gnu.org \
    --cc=h.goebel@crazy-compilers.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.