Patches for OpenSolaris
Adriaan de Groot
groot at kde.org
Wed Jan 21 20:33:06 CET 2009
Here at CampKDE I showed the Amarok dudes how it would run on OpenSolaris; it
does take a fistful of patches (mostly simple ones). Casey suggested I
actually send them in instead of blogging only :)
The three const-match patches make declaration and definition of various
methods match in terms of constness; gcc thinks this is fine, but for
compliers that consider constness in resolution or linking, this is a problem.
Free is in <stdlib.h> and doesn't necessarily live in the :: namespace.
The findmysql one is possibly not suitable for inclusion -- it adds some more
search paths and things to find MySQL in the default install locations on
Solaris.
The md5 code uses the linux kernel types u_int32_t instead of standard integer
types like uint32_t.
namespace foo { extern bar; } gives me a foo::bar symbol instead of a ::bar
symbol, so the namespace patch moves one extern around.
The scriptengine has some constness issues and a ; too many. I think Casey has
already committed that last one.
The amarok-wide patch is rather klumky, since it just forces in the Solaris
wide character functions instead of doing some suitable conditional
compilation. I'd be ok with maintaing this one separately.
The DEBUG_BLOCK macro uses the gcc-ism __PRETTY_FUNCTION__; see qdebug for one
way of dealing with this instead. I've gone for the boring approach of
#defining it to a string that is semi-sort-of-meaningful.
Finally, no-webkit removes WebKit from linking, because it conflicts with kjs
in KDE -- those are harmless collissions, i think, but it's annoying to see
all those show up on linking.
[ade]
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-const-match.2.diff
Type: text/x-patch
Size: 370 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-const-match.3.diff
Type: text/x-patch
Size: 786 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0001.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-const-match.diff
Type: text/x-patch
Size: 406 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0002.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-free-stdlib.diff
Type: text/x-patch
Size: 445 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0003.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-findmysql.diff
Type: text/x-patch
Size: 540 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0004.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-linuxism-uint.diff
Type: text/x-patch
Size: 3852 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0005.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-namespaced-extern.diff
Type: text/x-patch
Size: 447 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0006.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-scriptengine-const-ref.diff
Type: text/x-patch
Size: 869 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0007.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-scriptengine-syntax.diff
Type: text/x-patch
Size: 407 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0008.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-wide.diff
Type: text/x-patch
Size: 948 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0009.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-debug.diff
Type: text/x-patch
Size: 678 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0010.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-no-webkit.diff
Type: text/x-patch
Size: 293 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/amarok-devel/attachments/20090121/ee5decea/attachment-0011.diff
More information about the Amarok-devel
mailing list