On Saturday 14 May 2005 09:09, Silvan wrote: > On Saturday 14 May 2005 01:30 am, Stephen Torri wrote: > > I have never submitted a patch to rosegarden yet so I cannot claim to > > any previous efforts. My copy of rosegarden is from CVS so I made sure > > the patch was against the HEAD version. > > OK, that was someone else then. The guy was trying to be helpful by > cleaning up all the warnings, and the way he did it broke our compatibility > with KDE 3.1 or something.
Explanation needed here. I was the conservative party trying to keep compatibility with old KDE 3.1 libs. Requirements for Rosegarden [1] are KDE 3.x, Qt 3.x, and GCC 3.x, but as nobody cares for it, i guess that KDE 3.0 is no longer supported. The minimum KDE libraries that I can test are KDE 3.1 [2], but RG should compile and work under newer releases [3] too. Some of your changes may break compatibility with KDE 3.1, and some other don't, let's see some examples: * KStdAccel::key() -> KStdAccel::shortcut() OK, because key() is deprecated even in KDE 3.1, and shortcut() is available for KDE 3.1 and above. * KLineEditDlg -> KInputDialog This change breaks compatibility. KInputDialog has been introduced in KDE 3.2, but KLineEditDlg is still available under KDE 3.4, as a deprecated class. * KIO::NetAccess::download() Same as above, the three argument method was introduced in KDE 3.2 I couldn't compile RG with your patch, so there can be other issues with GCC, Qt or KDE. I don't know. [1] Requirements http://www.rosegardenmusic.com/getting/requirements/ [2] KDE 3.1 Library Reference http://developer.kde.org/documentation/library/3.1-api/index.html [3] KDE 3.4 Library Reference Regards, Pedro ------------------------------------------------------- This SF.Net email is sponsored by Oracle Space Sweepstakes Want to be the first software developer in space? Enter now for the Oracle Space Sweepstakes! http://ads.osdn.com/?ad_id=7393&alloc_id=16281&op=click _______________________________________________ Rosegarden-devel mailing list [email protected] - use the link below to unsubscribe https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
