qemu-devel
[Top][All Lists]
Advanced

[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

Attachment: .clang-format
Description: Binary data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]