Warning: Can't synchronize with repository "(default)" (Unsupported version control system "svn": No module named svn). Look in the Trac log for more information.

Ticket #2472 (closed enhancement: fixed)

Opened 2 years ago

Last modified 2 years ago

TGScheduler should use the logging API instead of print

Reported by: bochecha Owned by: vinces1979
Priority: normal Milestone: 1.5
Component: Scheduler Version: trunk
Severity: normal Keywords:
Cc:

Description

When deploying a TG app as a Windows service, stdout and stderr can't be used directly (they can crash the service).

Patch is attached, it's been tested running a TG1 app on Windows XP, both as a service or in development mode.

Attachments

use-logging-api-instead-of-print.patch Download (2.0 KB) - added by bochecha 2 years ago.

Change History

Changed 2 years ago by bochecha

comment:1 Changed 2 years ago by lmacken

Patch looks sane to me.

comment:2 Changed 2 years ago by Chris Arndt

  • Owner changed from faide to vinces1979

comment:3 Changed 2 years ago by vinces1979

  • Status changed from new to assigned
  • Type changed from defect to enhancement
  • Milestone changed from __unclassified__ to 1.5

Same here I implement

comment:4 Changed 2 years ago by Chris Arndt

So can this ticket be closed, Vince?. If yes, just close it yourself, please, and specify the revision number where you implemented this, if applicable.

comment:5 follow-up: ↓ 6 Changed 2 years ago by vinces1979

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

Yeah I just finished checking in and testing

Implemented in r7032

comment:6 in reply to: ↑ 5 Changed 2 years ago by vinces1979

Replying to vinces1979:

Yeah I just finished checking in and testing

Implemented in r7032

noticed a few changes got back out, put those back in r7033

Note: See TracTickets for help on using tickets.