Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 433098 - media-sound/clementine prints debug messages to terminal
Summary: media-sound/clementine prints debug messages to terminal
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Nikoli
URL:
Whiteboard:
Keywords: InOverlay
: 433459 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-08-28 20:58 UTC by Michael Niggli
Modified: 2012-10-08 14:33 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
introduce a debug use flag and conditionally get rid of debug messages (clementine-1.0.1-r3.ebuild,4.07 KB, text/plain)
2012-08-28 20:58 UTC, Michael Niggli
Details
patch for ebuild, works fine for me (clementine-1.0.1-r2.ebuild.patch,773 bytes, text/plain)
2012-10-08 06:45 UTC, Nikoli
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Niggli 2012-08-28 20:58:24 UTC
Created attachment 322478 [details]
introduce a debug use flag and conditionally get rid of debug messages

Clementine's CMakeLists.txt contains a check whether CMAKE_BUILD_TYPE is "Release" that disables the printing of debugging messages when running clementine. However, since the build type on Gentoo is "Gentoo", this check of course fails.

It would be nice if we could get rid of those messages for regular builds (maybe with a debug use flag in case somebody wants/needs them!?).

I've attached an ebuild based on clementine-1.0.1-r2.ebuild that does just that; it runs sed over the CMakeLists.txt unless the debug use flag is set. There's probably a better way of doing this, but I don't know it :-)
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2012-09-03 11:56:01 UTC
*** Bug 433459 has been marked as a duplicate of this bug. ***
Comment 2 Nikoli 2012-10-07 21:12:18 UTC
Should be fixed in multimedia overlay for 1.1.0_rc1 and 9999. After 1.1.0 release the fix will be in portage.
Comment 3 Nikoli 2012-10-08 06:45:39 UTC
Created attachment 325970 [details]
patch for ebuild, works fine for me

ssuominen, please commit patch for clementine-1.0.1-r2.ebuild
Comment 4 Sergey Popov gentoo-dev 2012-10-08 10:00:50 UTC
+  08 Oct 2012; Sergey Popov <pinkbyte@gentoo.org> clementine-1.0.1-r2.ebuild:
+  Supress debug output when USE='debug' is not set wrt bug #433098. Thanks to
+  Michael Niggli for discovering this issue and Nikoli for solving it

Also i have added missing inherit on flag-o-matic eclass. Please, test and report
Comment 5 Nikoli 2012-10-08 10:30:25 UTC
It is not missing, please remove second inherit from ebuild.
make-utils.eclass does 'append-cppflags -DNDEBUG' when use debug is disabled and already has 'inherit toolchain-funcs multilib flag-o-matic base'
Comment 6 Sergey Popov gentoo-dev 2012-10-08 11:31:39 UTC
Nikoli, i know it, but there are two cons to do it:

1) developer's policy prefer direct inherits when using functions in ebuild;
2) repoman complains about using this without inheriting flag-o-matic.