Wireshark mailing list archives
Re: RFD: Creating subdirectories in epan/dissectors/
From: Roland Knall <rknall () gmail com>
Date: Thu, 30 Aug 2012 15:23:27 +0200
Hi Would you like to enforce a value for the minimum number of subsequent files in the subdirectories? As I wrote the opensafety package, I would like to split it up a little bit to make it more maintainable, as well as include two new subdissectors, which use the openSAFETY protocol, but are not necessarily part of it. regards, Roland On Thu, Aug 30, 2012 at 3:19 PM, Evan Huus <eapache () gmail com> wrote:
On Thu, Aug 30, 2012 at 3:14 AM, Anders Broman <a.broman () bredband net> wrote:Evan Huus skrev 2012-08-30 04:31: On Wed, Aug 29, 2012 at 7:28 PM, Anders Broman <a.broman () bredband net> wrote: Jeff Morriss skrev 2012-08-30 00:29: Evan Huus wrote: I'm not 100% convinced either way, but I have to admit I do like having all the dissectors in the same directory. "make -j 40" (on my 32-vCPU SPARC) works better that way ;-). I'm pretty sure an autotools-generated Makefile will already recurse to fill the given job-count as long as there aren't any weird dependencies in place, so it shouldn't make any difference. Can't speak for cmake or windows builds. Only if all the files are in one Makefile (e.g., epan/dissectors/Makefile). If each subdir has its own Makefile then each directory is processed one at a time (in my experience). More seriously, I imagine I'd find it easier to do: vi epan/dissectors/packet-xmpp<tab><tab> instead of: vi epan/dissectors/packet-xmpp<tab><tab>[1]^H^H^H^H^H^H^H^H^H^H^Hxmpp<tab>/<tab><tab> [1] insert grumpy remark here Fair enough. So another tweak to the suggested naming: packet-xmpp/xmpp-whatever.c In fact taking your suggestion of removing "packet-" from all the file names would also achieve the same thing. I'm not particularly fond of the idea - just being conservative perhaps; but how many subdirectories are acceptable before it gets out of hand - 1000, one for every protocol in WS or a smaller number ;-) I expect most dissectors will stay in the root of epan/dissectors/. My understanding is that this would only be in a few select cases (bluetooth and xmpp are the ones that come to mind) where there is a clear logical grouping of a large number (16 for bluetooth, 14 for xmpp) of files. It makes sense to put them in their own folder. I just gave epan/dissectors/ a quick scan. Here are the other large groupings that immediately stand out: - aim (23 files) - dcerpc (111 files) - gsm (23 files) - h### (33 files) (is there a better generic name for this category?) - ipmi (12 files) - smb (15 files) - zbee (14 files) There are lots and lots of smaller groups of 4-5 files, but I wouldn't expect them to get their own folder. More input is always welcome :) h### comes from ITU (http://www.itu.int/en/ITU-T/publications/Pages/recs.aspx) ITU-T Series of Recommendations H: Audiovisual and multimedia systems I'm not sure we would like to have all of them in the same folder but that depends on the logic we want to apply, likewise some of those gsm protocol has nothing else in common than being used in Mobile system of GSM type (e.g not dissection for one protocol spread in several files) should UMTS and LTE have their own folders as well in that case? ANSI/CDMA? Perhaps it's more clear cut in aim dcerpc etc and makes more sense.I was just going by the existing file names, so if they're not actually logically related then they shouldn't be grouped. Being conservative on a first pass makes sense, and we can see how it works out for the really obvious cases. Unless there are any other objections I'd like to propose we start with these four as a sort of trial run: - aim - bluetooth - dcerpc - xmpp And with the naming scheme: packet-xmpp/xmpp-whatever.c Evan ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Re: RFD: Creating subdirectories in epan/dissectors/, (continued)
- Re: RFD: Creating subdirectories in epan/dissectors/ Steve Karg (Aug 25)
- Re: RFD: Creating subdirectories in epan/dissectors/ Evan Huus (Aug 25)
- Re: RFD: Creating subdirectories in epan/dissectors/ Jeff Morriss (Aug 27)
- Re: RFD: Creating subdirectories in epan/dissectors/ Evan Huus (Aug 25)
- Re: RFD: Creating subdirectories in epan/dissectors/ Jeff Morriss (Aug 29)
- Re: RFD: Creating subdirectories in epan/dissectors/ Evan Huus (Aug 29)
- Re: RFD: Creating subdirectories in epan/dissectors/ Jeff Morriss (Aug 29)
- Re: RFD: Creating subdirectories in epan/dissectors/ Anders Broman (Aug 29)
- Re: RFD: Creating subdirectories in epan/dissectors/ Evan Huus (Aug 29)
- Re: RFD: Creating subdirectories in epan/dissectors/ Anders Broman (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Evan Huus (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Roland Knall (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Evan Huus (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Graham Bloice (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Bill Meier (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Evan Huus (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Anders Broman (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Mike Morrin (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Jakub Zawadzki (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Jeff Morriss (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Evan Huus (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Ed Beroset (Aug 30)
- Re: RFD: Creating subdirectories in epan/dissectors/ Evan Huus (Aug 29)
- Re: RFD: Creating subdirectories in epan/dissectors/ Steve Karg (Aug 25)