Re: Revive the happy-shark repository?

[Jaap Keuter <jaap.keuter () xs4all nl>]

Hi,

As for the options proposed by Dario,
1) git submodules basically pins a specific commit of an external repository into your repository. It also requires 
additional git commands to checkout and move the ‘pin’ forward when anything is added to the external repo which is 
desired in the git repo. In my experience it is useful to include specific (release-)tags from external libraries, 
which develop on their own pace, not s much for development in parallel which is our use case.
2) git lfs could be interesting, but also makes it more complex for users, needing to install additional git features. 
In my experience something to be used strategically, I was a bit underwhelmed when finding 128 byte capture files 
stored with git lfs.

Although I’ve little experience with happy-shark, in theory I would like to see a (especially a TCP) test suite 
independent of Wireshark version, which is could probably best served by happy-shark.

Thanks,
Jaap


> On 22 Jan 2021, at 21:15, Dario Lombardo <lomato () gmail com> wrote:
>
> Talking about your options:
> 1) my concern here is that we would have 2 repos, with parallel lifecycles, that are not enforced to stay aligned. A 
> change in a dissector would benefit from a test case, but such a testcase in happy-shark would be proposed after the 
> code merge in the main repo. That would slow down the process, wouldn't it?
> 2) this is the current situation. Ideal in the sense that a change carries the code and the testcase. Suboptimal 
> because as soon as the testcases grow, the repo gets too heavy, as you said.
>
> If the concern is not to make the repo too heavy we may investigate other options as well.
> 1) use git submodules
> 2) use git lfs
> Option 2 sounds promising: "Git Large File Storage (LFS) replaces large files such as audio samples, videos, 
> datasets, and graphics with text pointers inside Git". We do have a dataset. Moreover gitlab.com <http://gitlab.com/> 
> supports LFS.
> Unfortunately I don't have direct experience with either submodules and lfs, hence I cannot provide more than just 
> raw ideas.
>
> On Fri, Jan 22, 2021 at 6:25 PM Gerald Combs <gerald () wireshark org <mailto:gerald () wireshark org>> wrote:
> Hi all,
>
> Years ago we added a repository for dissector regression tests at https://github.com/wireshark/happy-shark 
> <https://github.com/wireshark/happy-shark>. Unfortunately it hasn't received much attention, and instead we've been 
> adding dissector tests in the main repository. Should we
>
> - Import happy-shark into GitLab and move our current dissector tests there?
>
> - Retire happy-shark and do all of our testing in the main repository?
>
> - Something else?
>
> I'm leaning toward the first option for the simple reason that it will minimize the number of files we accrue in 
> test/captures.
> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org <mailto:wireshark-dev () wireshark org>>
> Archives:    https://www.wireshark.org/lists/wireshark-dev <https://www.wireshark.org/lists/wireshark-dev>
> Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev 
> <https://www.wireshark.org/mailman/options/wireshark-dev>
>              mailto:wireshark-dev-request () wireshark org <mailto:wireshark-dev-request () wireshark 
> org>?subject=unsubscribe
>
>
> -- 
> Naima is online.
> ___________________________________________________________________________
> Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
> Archives:    https://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://www.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:    https://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe