On 08/23/12 05:52, Pascal Quantin wrote:
> 2012/8/19 Lori Jakab <lori@xxxxxxxxxxx <mailto:lori@xxxxxxxxxxx>>
>
> On 08/19/12 06:56, Pascal Quantin wrote:
> > Having a RSS link (or something similar) to the subversion file log
> >
> (http://anonsvn.wireshark.org/viewvc/trunk/epan/dissectors/packet-gsmtap.c?view=log
> > for example) could be a way to easily monitor the changes done to a
> > given file for whoever is interested. Of course it would not prevent
> > the offending commit but it would allow a faster feedback /
> bugfix if
> > needed (which could be sufficient from my point of vue). Would it be
> > an acceptable first step towards improving the monitoring for
> changes
> > by original authors / contributors?
>
> I think that some kind of automatic and *opt-in* method to notify the
> author(s) of a certain dissector that changes have been made to their
> code should be available. Both the special header field proposed
> by Evan
> ("watch") and Sylvain ("maintainer") and the RSS feed would fulfill
> this. And they are not mutually exclusive either. RSS would allow
> interested non-authors as well to easily follow changes, while the
> maintainer option could be integrated with bugzilla so that
> authors are
> automatically CC'ed on patches submitted for files they maintain.
>
>
> Hi,
>
> it appears that Wireshark git mirror
> (http://http://code.wireshark.org/git/) already has RSS / Atom links
> (thanks for the hint Jakub ;) ). So whoever is interested can already
> subscribe to feeds.
> To get the full commit history you can use:
> http://code.wireshark.org/git/?p=wireshark;a=atom
> http://code.wireshark.org/git/?p=wireshark;a=rss
> And to follow a specific file, you can use the 'f' parameter:
> http://code.wireshark.org/git/?p=wireshark;a=atom;f=epan/dissectors/packet-gmr1_bcch.c
> http://code.wireshark.org/git/?p=wireshark;a=rss;f=epan/dissectors/packet-gmr1_bcch.c
>
This is awesome, thanks a lot!
I didn't know about the Git repository, are patches in the 'git diff'
format against the master branch accepted as well? (The output differs
slightly from 'svn diff'.)
Regards,
-Lori
> Regards,
> Pascal.
>
>
> ___________________________________________________________________________
> 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