On Sat, Jul 5, 2014 at 11:49 PM, Evan Huus <eapache@xxxxxxxxx> wrote:
> On Sat, Jul 5, 2014 at 2:46 PM, Wireshark code review
> <code-review-do-not-reply@xxxxxxxxxxxxx> wrote:
>>
>> URL:
>> https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=599b880e4c53613c243b0a39548968f34cdce0e3
>> Submitter: Guy Harris (guy@xxxxxxxxxxxx)
>> Changed: branch: master
>> Repository: wireshark
>>
>> Commits:
>>
>> 599b880 by Guy Harris (guy@xxxxxxxxxxxx):
>>
>> Handle the UTC timestamps in NetMon 2.3 files.
>>
>> This addresses part of, but not all of, the issues in bug ten
>> thousand,
>> one hundred, and ninety:
>>
>> https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=10190
>>
>> (I'm spelling it out to make sure Gerrit doesn't think this change
>> *does* address all the issues in that bug, and mark it as RESOLVED
>> FIXED; I feel like I have to treat Gerrit as a dog or small child from
>> whom I'm trying to keep a secret - "honey, I'm taking the dog to the
>> vee eee tee".)
>
>
> It would be nice to have different tags for Refs-Bug and Fixes-Bug, and have
> the bugzilla integration do The Right Thing for changes that refer to but do
> not fix a bug. Gerald, how easy is this? I believe OpenStack has a set of
> tags they use which we might look to for inspiration?
+1
I like OpenStack tags :
Closes-Bug: #1234567 -- use 'Closes-Bug' if the commit is intended to
fully fix and close the bug being referenced.
Partial-Bug: #1234567 -- use 'Partial-Bug' if the commit is only a
partial fix and more work is needed.
Related-Bug: #1234567 -- use 'Related-Bug' if the commit is merely
related to the referenced bug.
https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references
>
>>
>> Change-Id: Ic234130c1ea84cfaf47901485dca775e168f71d0
>> Reviewed-on: https://code.wireshark.org/review/2859
>> Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
>>
>>
>> Actions performed:
>>
>> from 3686713 Update libnghttp2 to latest master (add support for
>> draft-13)
>> adds 599b880 Handle the UTC timestamps in NetMon 2.3 files.
>>
>>
>> Summary of changes:
>> wiretap/netmon.c | 270
>> ++++++++++++++++++++++++++++++------------------------
>> 1 file changed, 148 insertions(+), 122 deletions(-)
>>
>> ___________________________________________________________________________
>> Sent via: Wireshark-commits mailing list
>> <wireshark-commits@xxxxxxxxxxxxx>
>> Archives: http://www.wireshark.org/lists/wireshark-commits
>> Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits
>>
>> mailto:wireshark-commits-request@xxxxxxxxxxxxx?subject=unsubscribe
>
>
>
> ___________________________________________________________________________
> Sent via: Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
> Archives: http://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
> mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe