[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] RFC: build-sys: move slirp as git submodule pro
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH] RFC: build-sys: move slirp as git submodule project |
Date: |
Mon, 25 Mar 2019 23:41:35 +0100 |
Hi
On Mon, Mar 25, 2019 at 7:58 PM Eric Blake <address@hidden> wrote:
>
> On 3/25/19 1:50 PM, Eric Blake wrote:
> > On 3/25/19 1:04 PM, Marc-André Lureau wrote:
> >> The slirp project is now hosted on freedesktop at:
> >> https://gitlab.freedesktop.org/slirp.
> >>
> >> The libslirp source tree there is based on current qemu/slirp filtered
> >> through clang-format, and can thus be directly used through a
> >> git submodule.
> >
> > Indeed, doing 'diff -ur slirp /path/to/libslirp.git' shows a lot of
> > formatting differences. Is there an easy command to see that the
> > contents of the two are identical modulo the formatting changes?
>
> Or put another way, what is the precise clang-format command to run on
> qemu.git/slirp/, prior to doing my 'diff -ur /path/to/qemu.git/slirp
> /path/to/libslirp.git'? And that formula should be in the commit message :)
I used clang-format *.c *.h with the attached .clang-format and version 7.0.1.
It should be added upstream, so we can keep a consistent style:
https://gitlab.freedesktop.org/slirp/libslirp/merge_requests/3
>
> I don't care if you ran clang-format on every commit in the new
> repository so that it has a cleaner git history over all commits that
> got transferred (in fact, if you did, that's actually cool, as it will
> make git history trawling in libslirp nicer), only that the current
> state of the git repository and the current state of the new submodule
> are easily correlated.
>
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc. +1-919-301-3226
> Virtualization: qemu.org | libvirt.org
>
thanks
--
Marc-André Lureau
.clang-format
Description: Binary data