Re: Use of CMAKE_BUILD_TYPE=Debian

2016-12-12 Thread Pino Toscano
In data lunedì 12 dicembre 2016 14:41:21 CET, Jonathan Riddell ha scritto:
> I'm wondering if there's still a need for CMAKE_BUILD_TYPE=Debian which gets 
> set on builds which use
> /usr/share/pkg-kde-tools/lib/kf5_flags

Theoretically yes, given the idea of it is to use the build flags by
kdelibs/frameworks, minus the optimization flags (read by
dpkg-buildflags). Unfortunately, when the frameworks packaging was
imported from Ubuntu, this was overlooked.

> It's used by 
> /usr/share/pkg-kde-tools/cmake/DebianABIManager.cmake

The ABI Manager is something else, and the only relation with the Debian
build type is that it does something (currently) only on that build
type. (But could be made work also with others.)

> and kdelibs(4) debian/patches/add_debian_build_type.diff

That's the kdelibs 4 version of this, set using
/usr/share/pkg-kde-tools/lib/kde4_flags.

> The downside is it just adds some confusion, what does the build type
> mean in every other case?

Confusion for who?

> Calligra compiles different projects if you
> build with Debug or Release set and it's not clear what happens for
> packages with Debian set.

Please report a bug to calligra then, or any other source that change
what is build according to the build type: this is simply 

> If the kdelibs(4) patch is still needed should it not be ported to
> KF5?

Theoretically yes (in extra-cmake-modules), but it's too late at this
point in the current Debian lifecycle.

> Can the ABI Manager do with a flat set by the packages which use it?

Not sure what you mean here, can you please explain in a simplier way?

Thanks,
-- 
Pino Toscano

signature.asc
Description: This is a digitally signed message part.
-- 
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-kde-talk

Re: Use of CMAKE_BUILD_TYPE=Debian

2016-12-12 Thread Maximiliano Curia

¡Hola Jonathan!

El 2016-12-12 a las 14:41 +, Jonathan Riddell escribió:
I'm wondering if there's still a need for CMAKE_BUILD_TYPE=Debian which gets set on builds which use 
/usr/share/pkg-kde-tools/lib/kf5_flags


It's used by 
/usr/share/pkg-kde-tools/cmake/DebianABIManager.cmake



and kdelibs(4) debian/patches/add_debian_build_type.diff


The downside is it just adds some confusion, what does the build type 
mean in every other case?  Calligra compiles different projects if you 
build with Debug or Release set and it's not clear what happens for 
packages with Debian set.



If the kdelibs(4) patch is still needed should it not be ported to KF5? Can the 
ABI Manager do with a flat set by the packages which use it?


As mentioned in the irc, the generated cmake files contain rules that are 
specific to the build type used. AFICS, this means that changing the default 
build type would require a rebuild of every package that ships an 
autogenerated cmake file (and uses pkg-kde-tools).


Happy hacking,
--
"Any change looks terrible at first." -- Principle of Design Inertia
Saludos /\/\ /\ >< `/


signature.asc
Description: PGP signature
-- 
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-kde-talk

Re: Use of CMAKE_BUILD_TYPE=Debian

2016-12-12 Thread Lisandro Damián Nicanor Pérez Meyer
On lunes, 12 de diciembre de 2016 14:41:21 ART Jonathan Riddell wrote:
> I'm wondering if there's still a need for CMAKE_BUILD_TYPE=Debian which gets
> set on builds which use /usr/share/pkg-kde-tools/lib/kf5_flags
> 
> It's used by
> /usr/share/pkg-kde-tools/cmake/DebianABIManager.cmake
> 
> and kdelibs(4) debian/patches/add_debian_build_type.diff
> 
> The downside is it just adds some confusion, what does the build type
> mean in every other case?

I only remember there where good reasons to do that. But never digged into 
that myself.

Maybe Pino and/or Sune remember them.

> Calligra compiles different projects if you
> build with Debug or Release set

That sounds like a bug :-/

-- 
Sólo porque un mensaje pueda no ser recibido no implica que no
valga la pena enviarlo.

Lisandro Damián Nicanor Pérez Meyer
http://perezmeyer.com.ar/
http://perezmeyer.blogspot.com/


signature.asc
Description: This is a digitally signed message part.
-- 
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-kde-talk

Use of CMAKE_BUILD_TYPE=Debian

2016-12-12 Thread Jonathan Riddell
I'm wondering if there's still a need for CMAKE_BUILD_TYPE=Debian which gets 
set on builds which use
/usr/share/pkg-kde-tools/lib/kf5_flags

It's used by 
/usr/share/pkg-kde-tools/cmake/DebianABIManager.cmake

and kdelibs(4) debian/patches/add_debian_build_type.diff

The downside is it just adds some confusion, what does the build type
mean in every other case?  Calligra compiles different projects if you
build with Debug or Release set and it's not clear what happens for
packages with Debian set.

If the kdelibs(4) patch is still needed should it not be ported to KF5? Can the 
ABI Manager do with a flat set by the packages which use it?

Jonathan

-- 
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-kde-talk