[PySide] Call for help: regression bugs to be fixed

[Matti Airas]

On 28.02.2012 20:40, ext Andy Kittner wrote:
> I could probably also do at least some of them.

Great! :-)

> Since I never worked with gerrit before I looked around a bit and found:
>
>   http://wiki.qt-project.org/Gerrit_Introduction
>   http://wiki.qt-project.org/Setting_up_Gerrit
>
> I guess they apply to PySide as well (including the mentioned 
> commit-hooks?)

Yes, these instructions apply to PySide as well.

> If I understood everything correctly essentially I would need to:
>  - clone the repository
>  - set up the hooks
>  - make the change
>  - push the change back to the master
> ?
> Anything else I need to consider?

The push needs to be for the refs/for/master as the target ref. And 
after this, you need to either notify some reviewers about the new 
contribution (using this mailing list or the IRC channel) or manually 
add reviewers using the Gerrit web interface. Then, the patches can be 
reviewed and hopefully approved.

> Secondly I think some merge in the shiboken repository went wrong.
> The following files are in the repository:
>   libshiboken/basewrapper.cpp.orig
>   libshiboken/basewrapper_p.h.orig

OK, this is awkward. Don't know what happened but I had managed to leave 
these in the working set when flattening the repos. I've fixed the issue 
in my clone and requested the Qt admins to force push the fixed commit in.

Thanks a lot for the heads up!

> And the build fails because ParentInfo::children is not consistently
> used as a std::list.

OK, my bad as well. I only did minimal reverts to the offending commits 
and did not test whether the code compiles after that. On the other 
hand, this approach should save a few edits when reimplementing the set 
use. :-)

Cheers,

ma.