Ubuntu
mythtv package

MythTV.pm prints message in STDERR if UPnP is used

Bug #327406 reported by Andrew Ruthven on 2009-02-09

Affects		Status	Importance	Assigned to	Milestone
	MythTV	Fix Released	Unknown	mythtv-bugs #6246
	mythtv (Ubuntu)	Fix Released	Undecided	Unassigned

Bug Description

Binary package hint: libmyth-perl

Currently whenever MythTV.pm falls back to using UPnP it prints a message
to STDERR stating this fact. This is totally unrequired and annoys users.

The attached patch stops this behaviour.

This bug has been reported to MythTV - http://svn.mythtv.org/trac/ticket/6246 but it is currently annoying users so it'd be great if it could go into Ubuntu sooner than MythTV will apply it.

Related branches

lp:ubuntu/lucid/mythtv

Revision history for this message

Andrew Ruthven (andrew-etc) wrote on 2009-02-09:

0001-Don-t-print-a-message-to-STDERR-if-we-re-using-UPnP.patch Edit (904 bytes, text/plain)

Revision history for this message

Dave Walker (davewalker) wrote on 2009-02-22:

Hi Andrew,

Thanks for reporting this bug and finding a patch, however - would it be better to change this output to be STDOUT rather than STDERR. Whilst I agree it's not an error that should be in STDERR, it is perhaps valid output to notify the user that UPnP is being used - so suggesting STDOUT.

Suggestions welcome.

Changed in mythtv:
assignee:	nobody → davewalker
status:	New → Incomplete

Revision history for this message

Andrew Ruthven (andrew-etc) wrote on 2009-02-22:

Hi Dave,

Given the number of users of mythtv-status that have asked me about the output I'd say the answer is "absolutely not".

This is a debugging message, it shouldn't be displayed to the users. They don't care if UPnP has been used as long as they receive the correct response.

Cheers!

Dave Walker (davewalker) on 2009-02-24

Changed in mythtv:
status:	Incomplete → Confirmed

Bug Watch Updater (bug-watch-updater) on 2009-05-24

Changed in mythtv:
status:	Unknown → New

Bug Watch Updater (bug-watch-updater) on 2009-10-15

Changed in mythtv:
status:	New → Fix Released

Mario Limonciello (superm1) on 2010-01-22

Changed in mythtv (Ubuntu):
status:	Confirmed → Triaged

Revision history for this message

Mario Limonciello (superm1) wrote on 2010-01-24:

As this issue has been fixed upstream, i'm going to mark it as fix committed for Ubuntu. It will get closed automatically when we upload a new snapshot of trunk into a development release.

Changed in mythtv (Ubuntu):
status:	Triaged → Fix Committed
assignee:	Dave Walker (davewalker) → nobody

Revision history for this message

Launchpad Janitor (janitor) wrote on 2010-02-17:

This bug was fixed in the package mythtv - 0.23.0~trunk23567-0ubuntu1

---------------
mythtv (0.23.0~trunk23567-0ubuntu1) lucid; urgency=low

  * Bump builds to start with 0.23
  * debian/control:
    - Fix vcs-bzr
    - Update depends to include ttf-droid, ttf-liberation, and ttf-dejavu.
      (LP: #224114)
    - Drop quilt build-dep, it appears to be included w/ dh7
    - Bump standards version
  * Introduces stable pulseaudio support (In Theory! :)) (LP: #311328)
  * Print message to stdout for MythTV.pm (LP: #327406)
  * debian/patches:
    - Drop 16_hal_shutdown. No longer necessary after r23518.
  * Allow shutdowns when mythtv is running (LP: #174712)
    - Comes from upstream changeset r23455.
  * Mythbackend now supports the --user flag to drop permissions from
    upstream changeset r23521.
  * Use the upstart script from the upstream contrib directory instead.
    Now uses the --user flag to drop permissions. (LP: #445953)
  * Update 30_python26_transition for changes upstream.
  * mythtv-database.templates:
    - refer to the proper location for backups.
  * Switch to use the upstream backup scripts. (LP: #469792)
  * Helps with HDTV stuttering. (LP: #451394)
  * Drop old debian/myth-*.sh scripts since we now use dbus for shutdown
    and reboot etc.
  * Drop old debian/mythtv-backend.default. It's just available for upgrades.
  * Drop old debian/mythtv-backend.init initscript. We're only supporting
    upstart going forward with these builds.
  * Drop unnecessary patch, 02_settings_pro.dpatch
  * Drop unused patch, 23_video_device_symlinks.dpatch
  * Drop unused patch, 25_skiploop_option.dpatch
  * Convert packaging to dh7.
  * Convert packaging to source v3 (w/ quilt).
  * Add debian/README.source as a symlink to the quilt README.source
  * Convert all .files into .install.
  * Refresh patches as quilt format instead
  * Rewrite get-orig-source rules to parse the changelog for version info.
  * Drop any of the Top Secret Options build flags
  * Include a custom snippet for debian/rules mythtv packages in libmyth-dev.
  * Drop old, unused templates from debian/templates.
-- Mario Limonciello <email address hidden> Tue, 16 Feb 2010 21:23:04 -0600

This bug was fixed in the package mythtv - 0.23.0~trunk23567-0ubuntu1

---------------
mythtv (0.23.0~trunk23567-0ubuntu1) lucid; urgency=low

* Bump builds to start with 0.23
  * debian/control:
    - Fix vcs-bzr
    - Update depends to include ttf-droid, ttf-liberation, and ttf-dejavu.
      (LP: #224114)
    - Drop quilt build-dep, it appears to be included w/ dh7
    - Bump standards version
  * Introduces stable pulseaudio support (In Theory! :)) (LP: #311328)
  * Print message to stdout for MythTV.pm (LP: #327406)
  * debian/patches:
    - Drop 16_hal_shutdown.  No longer necessary after r23518.
  * Allow shutdowns when mythtv is running (LP: #174712)
    - Comes from upstream changeset r23455.
  * Mythbackend now supports the --user flag to drop permissions from
    upstream changeset r23521.
  * Use the upstart script from the upstream contrib directory instead.
    Now uses the --user flag to drop permissions. (LP: #445953)
  * Update 30_python26_transition for changes upstream.
  * mythtv-database.templates:
    - refer to the proper location for backups.
  * Switch to use the upstream backup scripts. (LP: #469792)
  * Helps with HDTV stuttering. (LP: #451394)
  * Drop old debian/myth-*.sh scripts since we now use dbus for shutdown
    and reboot etc.
  * Drop old debian/mythtv-backend.default.  It's just available for upgrades.
  * Drop old debian/mythtv-backend.init initscript.  We're only supporting
    upstart going forward with these builds.
  * Drop unnecessary patch, 02_settings_pro.dpatch
  * Drop unused patch, 23_video_device_symlinks.dpatch
  * Drop unused patch, 25_skiploop_option.dpatch
  * Convert packaging to dh7.
  * Convert packaging to source v3 (w/ quilt).
  * Add debian/README.source as a symlink to the quilt README.source
  * Convert all .files into .install.
  * Refresh patches as quilt format instead
  * Rewrite get-orig-source rules to parse the changelog for version info.
  * Drop any of the Top Secret Options build flags
  * Include a custom snippet for debian/rules mythtv packages in libmyth-dev.
  * Drop old, unused templates from debian/templates.
 -- Mario Limonciello <superm1@ubuntu.com>   Tue, 16 Feb 2010 21:23:04 -0600