Argh!  Just to make sure we're on the same page, is the conflict you're getting in PlaylistGroupingAction.cpp and .h?  I removed those files because they aren't needed anymore, and this seems to be causing conflicts.  I thought I took care of those when I rebased before I did the merge request, but I guess for some reason git is still mad at me for this.  A little searching around doesn't reveal any good documentation on how to deal with these merge conflicts with deleted files.<br>

<br>Also, unfortunately, something I did totally fubared my clone of amarok, and now every time I run "git pull --rebase remote master" I get a ton of conflicts.  So it will take some time for me to sort this stuff out, and I might just need to do a fresh clone.<br>

<br><div class="gmail_quote">On Thu, Sep 3, 2009 at 1:53 AM, Nikolaj Hald Nielsen <span dir="ltr"><<a href="mailto:nhnfreespirit@gmail.com" target="_blank">nhnfreespirit@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">


Tim<br>
<br>
I just tried your patch (merging your repo) and it causes a few<br>
conflicts with current master. Could you please rebase and fix the<br>
conflicts? Then I will be happy to push this to master (after a bit of<br>
testing)<br>
<font color="#888888"><br>
- Nikolaj<br>
</font><div><div></div><div><br>
On Thu, Sep 3, 2009 at 12:25 AM, Ian Monroe<<a href="mailto:ian.monroe@gmail.com" target="_blank">ian.monroe@gmail.com</a>> wrote:<br>
> On Wed, Sep 2, 2009 at 4:49 PM, Tim Bocek<<a href="mailto:tim.bocek@gmail.com" target="_blank">tim.bocek@gmail.com</a>> wrote:<br>
>> I just finished a patch that implements tying grouping schemes to playlist<br>
>> layouts, and created a merge request at:<br>
>><br>
>> <a href="http://www.gitorious.org/amarok/amarok/merge_requests/1415" target="_blank">http://www.gitorious.org/amarok/amarok/merge_requests/1415</a><br>
>><br>
>> If this is too much given the state of the freeze (it does add some extra<br>
>> strings), I'm happy to just let this sit until 2.2.1.<br>
><br>
> Also meant to mention: delete the commented out code. Commented code<br>
> is confusing and pointless.<br>
><br>
> Ian<br>
> _______________________________________________<br>
> Amarok mailing list<br>
> <a href="mailto:Amarok@kde.org" target="_blank">Amarok@kde.org</a><br>
> <a href="https://mail.kde.org/mailman/listinfo/amarok" target="_blank">https://mail.kde.org/mailman/listinfo/amarok</a><br>
><br>
_______________________________________________<br>
Amarok mailing list<br>
<a href="mailto:Amarok@kde.org" target="_blank">Amarok@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/amarok" target="_blank">https://mail.kde.org/mailman/listinfo/amarok</a><br>
</div></div></blockquote></div><br>