On Fri, Sep 26, 2014 at 3:19 PM, Alexis La Goutte <[email protected]> wrote: > On Fri, Sep 26, 2014 at 9:12 PM, Evan Huus <[email protected]> wrote: >> 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? > > How to work submodules ? > > You need to add a step after git clone to add also submodules ? > or submodules it directly include in master ? > > Because there is also x11 dissector (mesa & xrand...) will be use submodules > too
You have to run "git submodule --init" after cloning the repository (we could maybe make that part of the build process or something). The only thing that is included in our master is the URL of the other repository, and the SHA to use. The nice thing is that it makes updating (which Guy is doing here manually) really easy, you just "git pull" in the submodule and then commit the one-line SHA change to our repository. >> >> On Fri, Sep 26, 2014 at 3:02 PM, Wireshark code review >> <[email protected]> wrote: >>> URL: >>> https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=bcae0488fc09cbb264b1a816be74e5e0721847b3 >>> Submitter: Guy Harris ([email protected]) >>> Changed: branch: master >>> Repository: wireshark >>> >>> Commits: >>> >>> bcae048 by Guy Harris ([email protected]): >>> >>> Update to the latest version from the Samba Git repository. >>> >>> From the Samba log: >>> >>> commit bfdc874e8b98c8ea147dbcc986f96ad4f73d800f >>> Author: Jelmer Vernooij <[email protected]> >>> 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 <[email protected]> >>> Reviewed-by: David Disseldorp <[email protected]> >>> >>> Autobuild-User(master): David Disseldorp <[email protected]> >>> Autobuild-Date(master): Sun Aug 31 23:47:49 CEST 2014 on >>> sn-devel-104 >>> >>> commit 6824f1aa67f0a75df5c94921e334c2b7c7771611 >>> Author: Jelmer Vernooij <[email protected]> >>> Date: Sat Aug 30 01:59:25 2014 +0200 >>> >>> Remove trailing whitespace. >>> >>> Change-Id: I1e0948da34bac278edc62cd63dedd08112426e7a >>> Signed-Off-By: Jelmer Vernooij <[email protected]> >>> Reviewed-by: David Disseldorp <[email protected]> >>> >>> Change-Id: Ifd445bf32aca2d30a6e501fc8c8dd030471ad284 >>> Reviewed-on: https://code.wireshark.org/review/4312 >>> Reviewed-by: Guy Harris <[email protected]> >>> >>> >>> 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(-) >>> ___________________________________________________________________________ >>> Sent via: Wireshark-commits mailing list >>> <[email protected]> >>> Archives: http://www.wireshark.org/lists/wireshark-commits >>> Unsubscribe: https://wireshark.org/mailman/options/wireshark-commits >>> >>> mailto:[email protected]?subject=unsubscribe >> ___________________________________________________________________________ >> Sent via: Wireshark-dev mailing list <[email protected]> >> Archives: http://www.wireshark.org/lists/wireshark-dev >> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev >> mailto:[email protected]?subject=unsubscribe > ___________________________________________________________________________ > Sent via: Wireshark-dev mailing list <[email protected]> > Archives: http://www.wireshark.org/lists/wireshark-dev > Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev > mailto:[email protected]?subject=unsubscribe ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <[email protected]> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:[email protected]?subject=unsubscribe
