On 08/18/2012 10:30 AM, Slávek Banko wrote:
Let me disagree.
Can I mention again that, included "renaming" patches are:
- Rename old tq methods that no longer need a unique name
- Remove additional unneeded tq method conversions
- Rename obsolete tq methods to standard names
- Rename a few stragglers
- Fix inadvertent "TQ" changes
These patches change such names, which anyway are in tqtinterface handled using "#define" - for example: #define tqsetSizePolicy setSizePolicy. These patches can not break the ABI. And not require recompilation of others.
Exceptions that require recompiling a few other packages I mentioned earlier: http://trinity-devel.pearsoncomputing.net/?0::9413
513ffc6e patch is different from the above. This patch changes the ABI and affects a large number of packages, which would have to be recompiled. Therefore, this patch (though is very small) for me seems with a too big impact than they should have a patches for SRU.
Slavek
Francios, Slavek,
I do not understand the issue at the ABI level, but I would advise only implementing the *necessary* name changes. I have had fits with the k->t renames on numerous files breaking builds that are frustrating to fix. I don't mind doing it if the change is necessary, but wholesale renaming just for the sake of it should be avoided.
I will defer to your wisdom on this, but we absolutely need kpowersave to work. I don't mind rebuilding, so if it makes sense do it. But the foundation of 3.5.13-1 should be as close to 3.5.13 as possible...