Comments you submit will be routed for moderation. If you have an account, please log in first.
Modify

Ticket #595 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

output to stdout/stderr

Reported by: biebl Owned by: alwin
Priority: normal Milestone: not applicable
Component: All Version: 1.2.x
Severity: tweak Keywords:
Cc:

Description

Hi,

it would be nice, if the output of kdesvn to stdout/stderr could be configurable and more fine grained.

Imho only critical errors should be printed to stderr, info/diag/debug messages should be off by default.

See also the corresponding Debian bug  http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=431616

Attachments

Change History

comment:1 Changed 4 years ago by alwin

  • Status changed from new to assigned

These are all output of kDebug() calls - them may switched off I thought on compile time or later with the kdebugdialog.

I may switch over all debug output to debug number 9510 which is called "kdevplatform (subversion)" and may match kdesvn :) (and include a hint into handbook/faq)

so users may easy switch off it. what about?

comment:2 follow-up: ↓ 3 Changed 4 years ago by Michael

I think I'd prefer a separate debug number, which is tied directly to the kdesvn program name.

If I search for svn or subversion, I get

7128 kio_svn 9510 kdevplatform (subversion)

I guess they are coming from the KDE4 plattform, not quite sure. Point is, I wouldn't immediately recognize which one to modify. Is kdesvn currently using the generic (0) debug number?

What do you think about changing the defaults (only debug messages going to stdout)?

comment:3 in reply to: ↑ 2 ; follow-up: ↓ 4 Changed 4 years ago by alwin

Replying to Michael:

I think I'd prefer a separate debug number, which is tied directly to the kdesvn program name.

It can not assigned outside kdelibs.

7128 kio_svn 9510 kdevplatform (subversion)

I guess they are coming from the KDE4 plattform, not quite sure. Point is, I wouldn't immediately recognize which one to modify. Is kdesvn currently using the generic (0) debug number?

Yes its using the generic. And using an own number makes no sense, users hadn't a real chance switch it off 'cause the kdebugdialog doesn't know something about. These numbers are defined inside kdelibs itself and looks ok.

What do you think about changing the defaults (only debug messages going to stdout)?

There are only debug-messages from development, I have no reasons doing more. I thought them will switched off in release-builds with -DKDE_NO_DEBUG_OUTPUT.

Yes, of course I'll reduce the debug outputs, most of them aren't needed this moment.

comment:4 in reply to: ↑ 3 Changed 4 years ago by mbiebl@…

Replying to alwin:

Replying to Michael:

I think I'd prefer a separate debug number, which is tied directly to the kdesvn program name.

It can not assigned outside kdelibs.

Hm, ok. Seems you are right. Just found $prefix/share/config/kdebug.areas

As  http://lists.kde.org/?l=kde-devel&m=106607933724723&w=2 explicitely recommends to use 0 (generic) for external apps, I guess I'd follow this recommendation and not change the debug area.

7128 kio_svn 9510 kdevplatform (subversion)

I guess they are coming from the KDE4 plattform, not quite sure. Point is, I wouldn't immediately recognize which one to modify. Is kdesvn currently using the generic (0) debug number?

Yes its using the generic. And using an own number makes no sense, users hadn't a real chance switch it off 'cause the kdebugdialog doesn't know something about. These numbers are defined inside kdelibs itself and looks ok.

Either way (using 0 or 9510), I think adding a note to the faq how to switch off the messages via kdedebugdialog is a good idea.

What do you think about changing the defaults (only debug messages going to stdout)?

Argh, I wanted to say: *no* debug messages on stdout by default. But afaics that is not possible with kDebug, as this is a global configuration and not per app.

There are only debug-messages from development, I have no reasons doing more. I thought them will switched off in release-builds with -DKDE_NO_DEBUG_OUTPUT.

Yes, of course I'll reduce the debug outputs, most of them aren't needed this moment.

comment:5 follow-up: ↓ 6 Changed 4 years ago by mbiebl@…

I've seen you changed the debug area. I thought this might be of interet to you:  http://www.kdedevelopers.org/node/3171

comment:6 in reply to: ↑ 5 ; follow-up: ↓ 7 Changed 4 years ago by alwin

  • Status changed from assigned to closed
  • Resolution set to fixed

Replying to mbiebl@gmail.com:

I've seen you changed the debug area. I thought this might be of interet to you:  http://www.kdedevelopers.org/node/3171

ah, wonderful did it in commit r1757. Debug output is reduced, so I close this bug (entry into faq follows)

comment:7 in reply to: ↑ 6 Changed 4 years ago by anonymous

Replying to alwin:

Replying to mbiebl@gmail.com:

I've seen you changed the debug area. I thought this might be of interet to you:  http://www.kdedevelopers.org/node/3171

ah, wonderful did it in commit r1757.

I'd also change kDebug(9510) back to kDebug.

Debug output is reduced, so I close this bug (entry into faq follows)

Not quite. I still get quite some output like:

svn_cmdline_init done QLayout: Attempting to add QLayout "" to SvnLogDlgImp? "logdialog", which already has a layout

That seems to be a bug somewhere in the code

The key is now: logmain-logcache Insert into map Getting headrev Getting headrev done

Most of it seems to come from svnqt, which uses qDebug...

comment:8 Changed 4 years ago by biebl

  • Reporter changed from mbiebl@… to biebl
View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.

Ihr Browser versucht gerade eine Seite aus dem sogenannten Internet auszudrucken. Das Internet ist ein weltweites Netzwerk von Computern, das den Menschen ganz neue Möglichkeiten der Kommunikation bietet.

Da Politiker im Regelfall von neuen Dingen nichts verstehen, halten wir es für notwendig, sie davor zu schützen. Dies ist im beidseitigen Interesse, da unnötige Angstzustände bei Ihnen verhindert werden, ebenso wie es uns vor profilierungs- und machtsüchtigen Politikern schützt.

Sollten Sie der Meinung sein, dass Sie diese Internetseite dennoch sehen sollten, so können Sie jederzeit durch normalen Gebrauch eines Internetbrowsers darauf zugreifen. Dazu sind aber minimale Computerkenntnisse erforderlich. Sollten Sie diese nicht haben, vergessen Sie einfach dieses Internet und lassen uns in Ruhe.

Die Umgehung dieser Ausdrucksperre ist nach §95a UrhG verboten.

Mehr Informationen unter www.politiker-stopp.de.