Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] master bcae048: Update to the latest ver
Hi,
2014-09-26 21:12 GMT+02:00 Evan Huus <eapache@xxxxxxxxx>:
> Since it looks like PIDL is also using git, how do people feel about
> using git submodules instead of maintaining our own copy of PIDL?
I would avoid submodules because it adds a lot of complexity without
providing little value.
If we would like to depend on external projects' files we don't want
to store in our source we can add them to build tools.
Cheers,
Balint
>
> On Fri, Sep 26, 2014 at 3:02 PM, Wireshark code review
> <code-review-do-not-reply@xxxxxxxxxxxxx> wrote:
>> URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=bcae0488fc09cbb264b1a816be74e5e0721847b3
>> Submitter: Guy Harris (guy@xxxxxxxxxxxx)
>> Changed: branch: master
>> Repository: wireshark
>>
>> Commits:
>>
>> bcae048 by Guy Harris (guy@xxxxxxxxxxxx):
>>
>> Update to the latest version from the Samba Git repository.
>>
>> From the Samba log:
>>
>> commit bfdc874e8b98c8ea147dbcc986f96ad4f73d800f
>> Author: Jelmer Vernooij <jelmer@xxxxxxxxx>
>> Date: Sat Aug 30 01:59:26 2014 +0200
>>
>> Various updates to the pidl README file.
>>
>> Remove samba3/samba4-specific comments, add comments about backends and files.
>>
>> Change-Id: Id2253ce85eab7a684b2c50d25f6f2604dc146a8e
>> Signed-Off-By: Jelmer Vernooij <jelmer@xxxxxxxxx>
>> Reviewed-by: David Disseldorp <ddiss@xxxxxxxxx>
>>
>> Autobuild-User(master): David Disseldorp <ddiss@xxxxxxxxx>
>> Autobuild-Date(master): Sun Aug 31 23:47:49 CEST 2014 on sn-devel-104
>>
>> commit 6824f1aa67f0a75df5c94921e334c2b7c7771611
>> Author: Jelmer Vernooij <jelmer@xxxxxxxxx>
>> Date: Sat Aug 30 01:59:25 2014 +0200
>>
>> Remove trailing whitespace.
>>
>> Change-Id: I1e0948da34bac278edc62cd63dedd08112426e7a
>> Signed-Off-By: Jelmer Vernooij <jelmer@xxxxxxxxx>
>> Reviewed-by: David Disseldorp <ddiss@xxxxxxxxx>
>>
>> Change-Id: Ifd445bf32aca2d30a6e501fc8c8dd030471ad284
>> Reviewed-on: https://code.wireshark.org/review/4312
>> Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
>>
>>
>> Actions performed:
>>
>> from c90acf2 Qt: Capture fixes.
>> adds bcae048 Update to the latest version from the Samba Git repository.
>>
>>
>> Summary of changes:
>> tools/pidl/README | 35 ++++++++++++++++++-----------------
>> 1 file changed, 18 insertions(+), 17 deletions(-)
>> ___________________________________________________________________________