[RFR] templates://phonon/{phonon-backend-null.templates}

2011-03-22 Thread Christian PERRIER
Please find, for review, the debconf templates and packages descriptions for 
the phonon source package.

This review will last from Tuesday, March 22, 2011 to Friday, April 01, 2011.

Please send reviews as unified diffs (diff -u) against the original
files. Comments about your proposed changes will be appreciated.

Your review should be sent as an answer to this mail.

When appropriate, I will send intermediate requests for review, with
[RFRn] (n=2) as a subject tag.

When we will reach a consensus, I send a Last Chance For
Comments mail with [LCFC] as a subject tag.

Finally, the reviewed templates will be sent to the package maintainer
as a bug report, and a mail will be sent to this list with [BTS] as
a subject tag.

Rationale:
--- phonon.old/debian/phonon-backend-null.templates 2011-03-14 
20:13:00.515104440 +0100
+++ phonon/debian/phonon-backend-null.templates 2011-03-18 06:51:42.699253466 
+0100
@@ -3,15 +3,18 @@
 _Description: Warning: Phonon is not functional
 
 Template: phonon-backend-null/isnt_functional
-Type: note
-_Description: Phonon multimedia capabilities are disabled
- Phonon based (i.e. most KDE and some Qt) applications will produce no audio or
- video output because there is no real Phonon backend installed on your system.
+Type: error
+#flag:comment:4
+# Translators: do not translate variable names
+# only translate (recommended) (the one in parenthesis)
+_Description: Missing backend for Phonon
+ Phonon-based (i.e. most KDE and some Qt) applications will produce no audio or
+ video output because there is no real Phonon backend installed on this system.
  Typically, this is an undesirable configuration which you might have gotten to
  unintentionally.
  .
- If you wish to restore full Phonon multimedia capabilities, install a real
- Phonon backend with your favourite package manager. The following Phonon
- backend packages seem to be available for your system at the moment:
+ If you wish to restore full Phonon multimedia capabilities, you
+ should install a real Phonon backend. The following Phonon
+ backend packages seem to be available for this system at the moment:
  .
  ${recommended_backend} (recommended)${other_backends}

I think that error is more well suited for this warning

flag:comment (see po-debconf(7)) is helpful because translating the
last paragraph is tricky: translators should not translate variable
names BUT should translate the word recommended.

s/Phonon based/Phonon-based
s/your/this: the system is maybe not my system

Turn imperative form install foo into a suggestion you should
install foo.Drop useless with your favourite package manager.

--- phonon.old/debian/control   2011-03-14 20:13:00.515104440 +0100
+++ phonon/debian/control   2011-03-18 06:55:23.436332507 +0100
@@ -17,11 +17,11 @@
  ${phonon:Recommended-Backend} | phonon-backend
 Suggests: ${phonon:Recommended-Backend}, phonon-backend-gstreamer,
  phonon-backend-xine, phonon-backend-mplayer
-Description: metapackage for the Phonon multimedia framework
+Description: Phonon multimedia framework - metapackage

Standardize short description with our usual general common
description - specific description style.

This is repeated over packages.

  Phonon is the Qt 4 multimedia API, which provides a task-oriented abstraction
  layer for capturing, mixing, processing, and playing audio and video content.
  .
- This metapackage ensures that a user has working Phonon configuration
+ This metapackage ensures that a user has a working Phonon configuration
  installed on the system (the core library and at least one backend).

Missing article.
 

@@ -55,12 +55,12 @@
  ${sameVersionDep:libqt4-dev:libphononexperimental4}
 Replaces: libphonon-dev ( 4:4.6.0really4.4.4)
 Breaks: libphonon-dev ( 4:4.6.0really4.4.4)
-Description: development files for the Phonon-Experimental library
+Description: Phonon-Experimental multimedia framework - development files
  Phonon is the Qt 4 multimedia API, which provides a task-oriented abstraction
  layer for capturing, mixing, processing, and playing audio and video content.
  .
  This package contains the development files needed to build Phonon backends or
- internal applications which need the Phonon-Experimental library. Third party
+ internal applications which need the Phonon-Experimental library. Third-party
  applications should not depend on this library. Please read README.Debian if
  you consider to Build-Depend on this package.
 

Missing dash in third-party


Template: phonon-backend-null/isnt_functional_title
Type: title
_Description: Warning: Phonon is not functional

Template: phonon-backend-null/isnt_functional
Type: error
#flag:comment:4
# Translators: do not translate variable names
# only translate (recommended) (the one in parenthesis)
_Description: Missing backend for Phonon
 Phonon-based (i.e. most KDE and some Qt) applications will produce no audio or
 video output because there is no real Phonon backend installed on this system.
 Typically, this 

Re: [RFR] templates://phonon/{phonon-backend-null.templates}

2011-03-22 Thread Modestas Vainius
Hello,

On antradienis 22 Kovas 2011 08:23:23 Christian PERRIER wrote:
 I think that error is more well suited for this warning

I disagree. Warning is not an error by definition. It is not wrong to have 
phonon-backend-null installed if you know what you are doing and you are doing 
this intentionally. As I made some changes to prefer phonon-backend-null 
whenever libphonon-dev is installed (for buildds), I added this warning to 
avoid *accidental* installations of this package on the user systems.

What is more, from debconf-devel(7):

error  This  datatype  is  used  for error messages, such as input validation   
   errors.  Debconf will show a question of this
   type even if the priority is too high or the user has already seen it.

The behaviour of this datatype clearly conflicts with my intentions and 
current scripts. I will not accept this change. Other changes are ok.

-- 
Modestas Vainius mo...@debian.org


signature.asc
Description: This is a digitally signed message part.


Re: [RFR] templates://phonon/{phonon-backend-null.templates}

2011-03-22 Thread Justin B Rye
(My patch goes along with the request to revert to Type: note.)

Christian PERRIER wrote:
 +_Description: Missing backend for Phonon

The spelling backend is developerese - the standard English spelling
is back end (and it's not that dictionaries are lagging behind
normal usage; see e.g. http://en.wikipedia.org/wiki/Back_end;).  But
since I've already seen how hard it is to get developers to accept
this I'll suggest the compromise form back-end within English text. 

 + Phonon-based (i.e. most KDE and some Qt) applications will produce no audio 
 or
 + video output because there is no real Phonon backend installed on this 
 system.
   Typically, this is an undesirable configuration which you might have gotten 
 to
   unintentionally.

This use of -based is a potentially confusing shorthand - things
like KTorrent aren't derivatives of Phonon, they just depend on it.
Maybe we could simplify the syntax a bit and say:

Applications using Phonon (the Qt multimedia framework for KDE) will produce
no audio or video output because there is no real Phonon back-end installed
on this system. This is typically an unintended configuration.

   .
 + If you wish to restore full Phonon multimedia capabilities, you
 + should install a real Phonon backend. The following Phonon
 + backend packages seem to be available for this system at the moment:

As long as we can trust the detection (what does this depend on?) we
could rely on the colon to imply most of this:

To restore full Phonon multimedia capabilities, install one of the Phonon
back-end packages which are currently available for this system:
   
(This turns install back into an imperative, but with the preceding
purpose clause it's advisory rather than a command.)

The package descriptions have really good boilerplate already, so my
corrections are mostly very fiddly:

  Package: phonon
[...]
 + This metapackage ensures that a user has a working Phonon configuration
   installed on the system (the core library and at least one backend).

Package installations work per-computer, not per-user.

This metapackage ensures that the system has a working Phonon configuration
installed (the core library and at least one back-end).

  Package: libphonon-dev
[...]
   This package contains the development files needed to build Qt 4 
 applications
   using Phonon cross-platform multimedia framework.
 ^the

  Package: libphononexperimental-dev
[...]
 +Description: Phonon-Experimental multimedia framework - development files

Or should these packages be 
   Description: Phonon multimedia framework - experimental library development 
files
...and so on?  But that's not in my patch.

   This package contains the development files needed to build Phonon backends 
 or
 + internal applications which need the Phonon-Experimental library. 
 Third-party
   applications should not depend on this library. Please read README.Debian if
   you consider to Build-Depend on this package.

That should be if you are considering Build-Depending on..., or just if
you are considering a build-dependency on

  Package: libphononexperimental4
[...]
  This package contains the Phonon-Experimental library which might be needed 
 by
^,
  Phonon backends or other internal applications. Third-party applications
  should not depend on this library. Please read README.Debian before using 
 this
  library.
 
 Package: phonon-backend-null
[...]
  This package does not provide any real Phonon backend. It can be used to
  disable Phonon audio/video capabilities. Please note that Phonon behaviour
  with this package installed has not been well tested.

s/behaviour/behavior/
 
 Package: phonon-dbg
[...]
  This package contains debugging symbols of the Phonon library and main
  backends You should install this package when you are about to report a crash
  or want to debug a problem with Phonon or its backends.

   This package contains debugging symbols for the Phonon library and its main
   back-ends. Install it if you want to report a crash or debug a problem with
   Phonon or its back-ends.

* * *
Obligatory Why-The-Name Appendix: http://phonon.kde.org/cms/1020
-- 
JBR with qualifications in linguistics, experience as a Debian
sysadmin, and probably no clue about this particular package
Template: phonon-backend-null/isnt_functional_title
Type: title
_Description: Warning: Phonon is not functional

Template: phonon-backend-null/isnt_functional
Type: note
#flag:comment:4
# Translators: do not translate variable names
# only translate (recommended) (the one in parenthesis)
_Description: Missing backend for Phonon
 Applications using Phonon (the Qt multimedia framework for KDE) will produce
 no audio or video output because there is no real Phonon back-end installed
 on this system. This is typically an unintended configuration.
 .
 To restore full Phonon multimedia capabilities, install one of the Phonon
 

Re: [RFR] templates://phonon/{phonon-backend-null.templates}

2011-03-22 Thread Christian PERRIER
Quoting Modestas Vainius (mo...@debian.org):

 I disagree. Warning is not an error by definition. It is not wrong to have 
 phonon-backend-null installed if you know what you are doing and you are 
 doing 
 this intentionally. As I made some changes to prefer phonon-backend-null 
 whenever libphonon-dev is installed (for buildds), I added this warning to 
 avoid *accidental* installations of this package on the user systems.


This is a grey area. error datatype has the advantage of being
displayed differently than simple notes (which users often tend to
skip too quickly).

But, anyway, I won't fight that much on this..:-)




signature.asc
Description: Digital signature


Processing of soprano_2.6.0+dfsg.1-2_amd64.changes

2011-03-22 Thread Debian FTP Masters
soprano_2.6.0+dfsg.1-2_amd64.changes uploaded successfully to localhost
along with the files:
  soprano_2.6.0+dfsg.1-2.dsc
  soprano_2.6.0+dfsg.1-2.debian.tar.gz
  soprano-daemon_2.6.0+dfsg.1-2_amd64.deb
  libsoprano4_2.6.0+dfsg.1-2_amd64.deb
  libsoprano-dev_2.6.0+dfsg.1-2_amd64.deb
  libsoprano-doc_2.6.0+dfsg.1-2_all.deb
  libsoprano-dbg_2.6.0+dfsg.1-2_amd64.deb

Greetings,

Your Debian queue daemon (running on host franck.debian.org)


-- 
To UNSUBSCRIBE, email to debian-qt-kde-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/e1q26js-0007x7...@franck.debian.org



Bug#608991: plasma-widgets-workspace: no icons shown for lock/logout plasmoid

2011-03-22 Thread Timo Boettcher
This seems to be fixed, at least the icons are shown now.



-- 
To UNSUBSCRIBE, email to debian-qt-kde-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/2011034117.ga32...@spinnennetz.org



Processing of polkit-kde-1_0.99.0-1_amd64.changes

2011-03-22 Thread Debian FTP Masters
polkit-kde-1_0.99.0-1_amd64.changes uploaded successfully to localhost
along with the files:
  polkit-kde-1_0.99.0-1.dsc
  polkit-kde-1_0.99.0.orig.tar.bz2
  polkit-kde-1_0.99.0-1.debian.tar.gz
  polkit-kde-1_0.99.0-1_amd64.deb

Greetings,

Your Debian queue daemon (running on host franck.debian.org)


-- 
To UNSUBSCRIBE, email to debian-qt-kde-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/e1q2bbc-00036s...@franck.debian.org



Bug#617725: Bug#617656: qtwebkit: FTBFS: MediaPlayerPrivatePhonon.h:27:29: fatal error: phononnamespace.h: No such file or directory

2011-03-22 Thread Julien Cristau
Hi Qt maintainers,

On Thu, Mar 10, 2011 at 12:16:40 +0100, Cyril Brulebois wrote:

 Source: qtwebkit
 Version: 2.1.0~2011week09-1
 Severity: serious
 Justification: FTBFS
 
On Mon, Mar 14, 2011 at 11:43:28 +, Hector Oron wrote:

 Source: qtwebkit
 Version: 2.1.0~2011week09-1
 Severity: serious
 Justification: FTBFS
 
On Fri, Mar 11, 2011 at 00:02:49 +0100, Ralf Treinen wrote:

 Package: icheck,qtmobility-dev
 Version: icheck/0.9.7-6.1
 Version: qtmobility-dev/1.2.0~beta0+git3174-1
 Severity: serious
 User: trei...@debian.org
 Usertags: edos-file-overwrite
 
Is there any progress on these bugs?  They should be fixed ASAP, please.
Even if there's no progress yet a note to that effect, or a notice that
you need help, would be appreciated.

Cheers,
Julien



-- 
To UNSUBSCRIBE, email to debian-qt-kde-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20110323012252.gn3...@radis.liafa.jussieu.fr