Re: [PULL] http://kernellabs.com/hg/~mkrufky/dib0070

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Sep 28, 2009 at 12:21 PM, Michael Krufky <mkrufky@xxxxxxxxxxxxxx> wrote:
> On Mon, Sep 28, 2009 at 12:18 PM, Patrick Boettcher
> <pboettcher@xxxxxxxxxxxxxx> wrote:
>> On Mon, 28 Sep 2009, Michael Krufky wrote:
>>>>
>>>> You're now the 3rd guy to implement and to pull-request this to Mauro.
>>>>
>>>> We should have a page somewhere listing all PULL requests (if sent with a
>>>> certain syntax) which everyone can check before doing a thing...
>>>>
>>>> Sorry for the duplicated work.
>>>
>>> I think this could have been avoided if the person who noticed the
>>> issue had pushed his workaround patch to the mailing list for review
>>> before it appeared in the linuxtv.org master branch :-(
>>
>> Yes, but Mauro was in a hurry to fix it... and may have easily overseen my
>> pull request.
>>
>>> That patch needs to be reverted, otherwise the static dependency
>>> remains regardless of the change in the dib0070 header file.
>>>
>>> Perhaps it would make sense for you to rebase against my dib0070 tree
>>> before Mauro pull's either one.
>>
>> That's the real problem. hg pull --rebase doesn't work :P .
>>
>>> Who else put in a pull request for a similar patch other than you or I,
>>> Patrick?
>>
>> Internally in DiBcom a fix similar was prepared, because the pull request
>> was overlooked.
>
> Patrick,
>
> Only with your permission, I can do this rebase for you -- please meet
> me on irc.

OK, this is done.  Mauro, you can pull from either tree (or both) to
get all fixes without any conflicts :-)

Cheers,

Mike
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux