Phonon include headers mismatch

[Thiago Macieira]

Let's decide on one thing only and stick to it:

Phonon 4.0 (KDE 4.0) headers are in Phonon/*
Phonon 4.1 (Qt 4.4) headers are in phonon/*

http://doc.trolltech.com/4.4/phonon.html says #include <Phonon>, but 
there's no such file in Qt sources. It does in KDE sources.

http://doc.trolltech.com/4.4/phonon-module.html says #include <phonon>. 
That exists in Qt sources, but not in KDE's.

The Qt examples use #include <phonon/lowercaseheader.h> (see 
http://doc.trolltech.com/4.4/phonon-musicplayer-mainwindow-h.html)

Qt sources don't have per class forwarding headers, but the kdesupport 
sources (trunk, to-be-4.3) do.

So I'm tempted to say we screwed up with Qt 4.4.0 and should rectify it 
immediately. 

We capitalise the P in the include dir, change the examples. For 
compatibility, we can provide a symlink for case-sensitive filesystems. 
And we add the per-class forwarding headers.

Do we have an agreement?

-- 
  Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-multimedia/attachments/20080629/362a41c3/attachment.sig>
-------------- next part --------------
_______________________________________________
kde-multimedia mailing list
kde-multimedia at kde.org
https://mail.kde.org/mailman/listinfo/kde-multimedia