mythtv-setup wrapper script calls mythfilldatabase without the configured arguments

Bug #1099149 reported by matlo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mythbuntu
Fix Released
High
Thomas Mashos

Bug Description

When I quit mythtv-setup, it asks me wether or not to run mythfilldatabase.

If I choose to run mythfilldatabase, it is run without the arguments configured in mythtv-setup.

Revision history for this message
matlo (mat-lau) wrote :

Fix: get the arguments from the database and add them in the mythfilldatabase command.

Revision history for this message
Thomas Mashos (tgm4883) wrote :

Assigning to superm1 since he has access to packaging

Changed in mythbuntu:
assignee: nobody → Mario Limonciello (superm1)
importance: Undecided → Medium
status: New → Triaged
importance: Medium → High
Revision history for this message
Mario Limonciello (superm1) wrote :

In general the idea of this patch sounds good, but we should not be reading in mysql.txt. Instead should be reading in config.xml.

Revision history for this message
Thomas Mashos (tgm4883) wrote :

We really do appreciate you opening this ticket to help improve Mythbuntu, but it needs to be closed for a number of reasons. The biggest one is that upstream has moved on to a new version and believes this to be fixed. Could you please verify if this issue still exists in the latest version?

Please do not let the closing of this ticket dissuade you from opening a new ticket if this (or any other) problem occurs with the newer versions.

Changed in mythbuntu:
status: Triaged → Expired
Revision history for this message
matlo (mat-lau) wrote :

It still occurs in newer versions (trusty, utopic, vivid).

matlo (mat-lau)
Changed in mythbuntu:
status: Expired → New
Revision history for this message
Thomas Mashos (tgm4883) wrote :

@mat-lau are you able to fix your patch to use config.xml instead of mysql.txt? mysql.txt has been deprecated for years.

Revision history for this message
matlo (mat-lau) wrote :

Hi,
I made a new patch that I tested in a Ubuntu 14.04 derivate distribution (Linux Mint 17.3).
It requires adding a dependency on libxml-simple-perl to mythtv-backend.

Revision history for this message
Thomas Mashos (tgm4883) wrote :

Thanks @mat-lau I've just applied this to our 0.28 packaging.

Changed in mythbuntu:
assignee: Mario Limonciello (superm1) → Thomas Mashos (tgm4883)
status: New → Fix Committed
Revision history for this message
Thomas Mashos (tgm4883) wrote :

I've merged this to the 0.27 packaging and it should be in the next build.

Changed in mythbuntu:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.