Np! I will do one for gcl.
-- Mohamed Mansour

On Thu, Sep 10, 2009 at 3:31 AM, John Abd-El-Malek <[email protected]> wrote:

> This is just for Rietveld issues.  We can possibly do the same to the
> commit log in gcl/git scripts, again patches welcome :)
> Thanks Mohamed for the patch, I committed it and made it live with two
> minor changes.
>
>
> On Wed, Sep 9, 2009 at 10:04 PM, PhistucK <[email protected]> wrote:
>
>> Are we only talking about when it is viewed in Rietveld?What about when
>> it is actually committed, can the change log be tweaked that way,
>> automatically, too?
>> That would be nice when reading change logs.
>>
>> Thank you!
>>
>> ☆PhistucK
>>
>>
>>
>> On Thu, Sep 10, 2009 at 05:01, John Abd-El-Malek <[email protected]>wrote:
>>
>>> Thanks, glad you enjoy it :)
>>> This was a side distraction to fix the annoying issue of having to
>>> manually copying and pasting the bug ids.  It severely tested my regex-fu
>>> and since the multiple bugs case should be rare, I'll hide behind the excuse
>>> that if they're duplicates they should be marked as such and only end up
>>> with one, and if they're different bugs there should be separate changelists
>>> ;)  The rietveld change is at
>>> http://code.google.com/p/rietveld/source/detail?r=455, if anyone sends
>>> me a patch to make it work with multiple bug ids, I'd be happy to push it.
>>>
>>>
>>> On Wed, Sep 9, 2009 at 6:15 PM, Mohamed Mansour <[email protected]>wrote:
>>>
>>>> Very nice, no longer I have to put crbug.com/#.  Whoever did that
>>>> autolink, can you support multiple bugs as well? The format bugdroid uses 
>>>> is
>>>> BUG=1, 2, 3
>>>> - Mohamed Mansour
>>>>
>>>>
>>>>
>>>> On Wed, Sep 9, 2009 at 8:58 PM, Paweł Hajdan Jr. <
>>>> [email protected]> wrote:
>>>>
>>>>> I just noticed that BUG=1234 lines are autolinked to the correct bug
>>>>> when viewed in Rietveld (codereview.chromium.org). This is very
>>>>> useful, thanks!
>>>>>
>>>>>
>>>>
>>>>
>>>>
>>>
>>>
>>>
>>
>> >>
>>
>

--~--~---------~--~----~------------~-------~--~----~
Chromium Developers mailing list: [email protected] 
View archives, change email options, or unsubscribe: 
    http://groups.google.com/group/chromium-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to