Bug 1852

Summary:	draksound refers to "above list" when the list actually appears below
Product:	Mageia	Reporter:	Remco Rijnders <remco>
Component:	RPM Packages	Assignee:	Thierry Vignaud <thierry.vignaud>
Status:	RESOLVED FIXED	QA Contact:
Severity:	enhancement
Priority:	Normal	CC:	marja11, yurchor
Version:	Cauldron	Keywords:	Junior_job
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	drakxtools-13.58-1.mga1.src.rpm	CVE:
Status comment:
Attachments:	Proposed fix

Description Remco Rijnders 2011-06-19 12:47:33 CEST

Description of problem:

When choosing a sound driver, the below message is shown:

#. -PO: keep the double empty lines between sections, this is formatted a la LaTeX
#: harddrake/sound.pm:506
#, c-format
msgid ""
"If you really think that you know which driver is the right one for your card\n"
"you can pick one from the above list.\n"
"\n"
"The current driver for your \"%s\" sound card is \"%s\" "
msgstr ""


However, the list presented actually appears below this text.

Thierry Vignaud 2011-10-04 17:38:55 CEST

Keywords: (none) => Junior_job

Thierry Vignaud 2011-10-04 17:39:21 CEST

Severity: normal => enhancement

Comment 1 Marja Van Waes 2012-01-08 18:27:32 CET

pinging. because nothing happened to this report since more than 3 months ago, and it still has the status NEW or REOPENED

@ Thierry
Please set status to ASSIGNED if you think this bug was assigned correctly. If for work flow reasons you can't do that, then please put OK on the whiteboard instead.

@ Remco

Is this bug still valid?

CC: (none) => marja11

Manuel Hiebel 2012-10-31 20:04:08 CET

Version: 1 => Cauldron

Comment 2 Remco Rijnders 2013-06-23 18:24:24 CEST

Created attachment 4158 [details]
Proposed fix

Comment 3 Remco Rijnders 2013-06-23 18:25:32 CEST

Yes, this bug is still valid.

@tv: I'm unsure how to effectuate the proposed patch so all translators are aware of the updated string. Can you help and/or explain the process needed to follow?

Comment 4 Remco Rijnders 2014-04-09 13:21:31 CEST

Yuri, you seem to have updated the translation catalogs for all languages before. Can you explain the procedure to me? I'd really like to close this bug report :)

CC: (none) => yurchor

Comment 5 Yuri Chornoivan 2014-04-09 14:09:43 CEST

(In reply to Remco Rijnders from comment #4)
> Yuri, you seem to have updated the translation catalogs for all languages
> before. Can you explain the procedure to me? I'd really like to close this
> bug report :)

There is a wiki page for this. ;)

https://wiki.mageia.org/en/Fixing_strings_in_code_without_breaking_translations

Or better just fix the string in the code and POT template, push and Transifex will do the rest. Do not do "make update_n_merge" in /po dir because tv does not like this. ;)

Comment 6 Mageia Robot 2014-04-10 20:42:20 CEST

commit 5b21e2b5c4ffd16ce1f830a13ab9625b7468d45c
Author: Remco Rijnders <remco@...>
Date:   Thu Apr 10 20:42:09 2014 +0200

    The list with drivers appears below, really (mga#1852)
---
 Commit Link:
   http://gitweb.mageia.org/software/drakx/commit/?id=5b21e2b5c4ffd16ce1f830a13ab9625b7468d45c

Comment 7 Remco Rijnders 2014-04-10 20:47:49 CEST

Fixed, hopefully :-)

Status: NEW => RESOLVED
Resolution: (none) => FIXED