Opened 11 years ago

Closed 11 years ago

#38337 closed update (fixed)

kdepim4-runtime and kdepim4: update to 4.10.1

Reported by: NicosPavlov Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 2.1.3
Keywords: Cc: sharky@…, bjoern.ricks@…, mkae (Marko Käning)
Port: kdepim4 kdepim4-runtime

Description

Kdepim ports can be updated to version 4.10.1, following the update of the rest of kde ports. The attached port for kdepim4 also contains a workaround for boost 1.53 as the build crashes.

Attachments (6)

Portfile-kdepim4-runtime.diff (903 bytes) - added by NicosPavlov 11 years ago.
Portfile-kdepim4.diff (1.0 KB) - added by NicosPavlov 11 years ago.
patch-assuanserverconnection.diff (411 bytes) - added by NicosPavlov 11 years ago.
patch-CMakeLists.txt (365 bytes) - added by NicosPavlov 11 years ago.
Patch to deactivate documentation build
Portfile-kdepim4-runtime.2.diff (1.2 KB) - added by NicosPavlov 11 years ago.
Portfile-kdepim4.2.diff (1.6 KB) - added by NicosPavlov 11 years ago.

Download all attachments as: .zip

Change History (27)

Changed 11 years ago by NicosPavlov

Changed 11 years ago by NicosPavlov

Attachment: Portfile-kdepim4.diff added

Changed 11 years ago by NicosPavlov

comment:1 Changed 11 years ago by mkae (Marko Käning)

This MIGHT close also https://bugs.kde.org/show_bug.cgi?id=261509 if meinproc4 isn't called anymore.

comment:2 Changed 11 years ago by mkae (Marko Käning)

Another suggestion for a change to this port is to remove the dblatex dependency, since a warning during the configure phase points out why it is needed:

info:configure -----------------------------------------------------------------------------
:info:configure -- The following OPTIONAL packages could NOT be located on your system.
:info:configure -- Consider installing them to enable more features from this software.
:info:configure -----------------------------------------------------------------------------
:info:configure    * dblatex  <http://dblatex.sourceforge.net/>
:info:configure      The DocBook to LaTeX converter
:info:configure      Required for generating Kontact-Touch license information.
:info:configure
:info:configure -----------------------------------------------------------------------------

However, if you think the license information IS needed we could make the dblatex dep an opt-in variant.

I suggest this since it is a hell of a job to install dblatex for nothing on systems which don't have the comfort of binary archives.

comment:3 Changed 11 years ago by mkae (Marko Käning)

Cc: mk@… added

Cc Me!

comment:4 Changed 11 years ago by mkae (Marko Käning)

I got this when tryin to build kdepim4-runtime:

:info:build                  from /opt/macports-test/var/macports/build/_Users_marko_WC_SVN_MacPorts_kde_kdepim4-runtime/kdepim4-runtime/work/kdepim-runtime-4.10.1/agents/nepomukfeeder/nepomukfeederagent.cpp:23:
:info:build /opt/macports-test/include/boost/preprocessor/config/config.hpp:86:77: warning: "__GXX_EXPERIMENTAL_CXX0X__" is not defined
:info:build /opt/macports-test/var/macports/build/_Users_marko_WC_SVN_MacPorts_kde_kdepim4-runtime/kdepim4-runtime/work/kdepim-runtime-4.10.1/agents/nepomukfeeder/nepomukfeederagent.cpp:66: error: using ‘typename’ outside of template
:info:build /opt/macports-test/var/macports/build/_Users_marko_WC_SVN_MacPorts_kde_kdepim4-runtime/kdepim4-runtime/work/kdepim-runtime-4.10.1/agents/nepomukfeeder/nepomukfeederagent.cpp:66: error: using ‘typename’ outside of template
:info:build make[2]: *** [agents/nepomukfeeder/CMakeFiles/akonadi_nepomuk_feeder.dir/nepomukfeederagent.o] Error 1
:info:build make[2]: Leaving directory `/opt/macports-test/var/macports/build/_Users_marko_WC_SVN_MacPorts_kde_kdepim4-runtime/kdepim4-runtime/work/build'
:info:build make[1]: *** [agents/nepomukfeeder/CMakeFiles/akonadi_nepomuk_feeder.dir/all] Error 2
:info:build make[1]: *** Waiting for unfinished jobs....

comment:5 Changed 11 years ago by mkae (Marko Käning)

comment:6 in reply to:  2 Changed 11 years ago by mkae (Marko Käning)

Replying to mk@…:

Another suggestion for a change to this port is to remove the dblatex dependency, since a warning during the configure phase points out why it is needed:

This suggestion refers to port kdepim4!

comment:7 in reply to:  1 Changed 11 years ago by NicosPavlov

Replying to mk@…:

This MIGHT close also https://bugs.kde.org/show_bug.cgi?id=261509 if meinproc4 isn't called anymore.

No, I don't think it does. I usually do not have problems with meinproc4 myself, but I heard that it is common, and it is called in this case. It is not present in this port yet, but we could use the usual doc deactivation code from other kde ports also for kdepim4

if {![variant_isset docs]} {
    patchfiles      patch-CMakeLists.diff
}

Changed 11 years ago by NicosPavlov

Attachment: patch-CMakeLists.txt added

Patch to deactivate documentation build

comment:8 Changed 11 years ago by mkae (Marko Käning)

Yes, something like that should be done.

BTW, does kdepim4 build in your case? Didn't you see the above error? I am here on SL.

comment:9 Changed 11 years ago by NicosPavlov

Replying to mk@…:

I got this when tryin to build kdepim4-runtime:

:info:build /opt/macports-test/include/boost/preprocessor/config/config.hpp:86:77: warning: "__GXX_EXPERIMENTAL_CXX0X__" is not defined

I can confirm the error on ML when trying to build with

sudo port install kdepim4-runtime configure.compiler=apple-gcc-4.2

I would assume that the line with __GXX_EXPERIMENTAL_CXX0X__ indicates that libstd++ should be used, and that we would have to force a newer compiler for that, like other ports:

if {${configure.compiler} != "clang" && ${os.platform} == "darwin"} {
    depends_build-append    port:clang-3.1
    configure.compiler      macports-clang-3.1
}
Last edited 11 years ago by ryandesign (Ryan Carsten Schmidt) (previous) (diff)

comment:10 Changed 11 years ago by NicosPavlov

It appears that kdepim4 has the same issue with compilers.

I attached updated diffs for the two Portfiles.

Changed 11 years ago by NicosPavlov

comment:12 Changed 11 years ago by mkae (Marko Käning)

I proceeded fine so far with CMakeLists.txt.diff analogously to kmymoney4(-devel), but now I ran into this:

:info:configure -----------------------------------------------------------------------------
:info:configure -- The following REQUIRED packages could NOT be located on your system.
:info:configure -- You must install these packages before continuing.
:info:configure -----------------------------------------------------------------------------
:info:configure    * NepomukWidgets  <http://www.kde.org>
:info:configure      The NepomukWidget libraries
:info:configure      Nepomuk extends the search and tagging functionalities in AkonadiConsole
:info:configure
:info:configure -----------------------------------------------------------------------------

Changed 11 years ago by NicosPavlov

Attachment: Portfile-kdepim4.2.diff added

comment:13 Changed 11 years ago by NicosPavlov

Kdepim is missing the new port nepomuk-widgets, apparently. Added in the new diff version.

comment:14 Changed 11 years ago by mkae (Marko Käning)

Still no success:

:info:build Generating collectiongeneralpage.moc
:info:build Generating sendmdnhandler.moc
:info:build opt/macports-test/include/boost/type_traits/detail/has_binary_operator.hp:50: Parse error at "BOOST_JOIN"
:info:build automoc4: process for /opt/macports-test/var/macports/build/_Users_marko_WC_SVN_MacPorts_kde_kdepim4/kdepim4/work/build/kleopatra/assuanserverconnection.moc failed: Unknown error
:info:build pid to wait for: 0
:info:build Generating moc_signencryptfilescommand.cpp

comment:15 in reply to:  14 Changed 11 years ago by mkae (Marko Käning)

Replying to mk@…:

Still no success:

Was my mistake. I hadn't used your patch and just added the missing dep but didn't notice the patchfiles*-append*...

Now it should work.

comment:16 Changed 11 years ago by mkae (Marko Käning)

OK, now kdepim4 builds fine here on my SL.

I recommend removing dblatex from the list of deps to avoid installation of the docbook to latex converter by adding this:

if {![variant_isset docs]} {
    patchfiles-append      patch-CMakeLists.diff
    depends_lib-delete     dblatex
}
Last edited 11 years ago by mkae (Marko Käning) (previous) (diff)

comment:17 in reply to:  16 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to mk@…:

    depends_lib-delete     dblatex

You mean depends_lib-delete port:dblatex

comment:18 Changed 11 years ago by mkae (Marko Käning)

Yes, of course. :-)

comment:19 Changed 11 years ago by mkae (Marko Käning)

Nicos, have you something against committing this port including the exclusion of dblatex for the port?

P.S.: BTW, I haven't tried to start anything from the successfully installed kdepim4 port, because I was afraid it would interfere with my Mail.app... :-/

comment:20 Changed 11 years ago by NicosPavlov

dblatex is clearly announced as optional, so it could indeed seem logical to include it as a dependency only in the +docs variant.

Then, I have nothing against committing, I am just waiting for the usual maintainer timeout duration.

comment:21 Changed 11 years ago by NicosPavlov

Resolution: fixed
Status: newclosed

Maintainer timeout, committed in r104033 and r104034.

Note: See TracTickets for help on using tickets.