qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Laszlo Ersek <lersek@redhat.com>
To: Jean Delvare <jdelvare@suse.de>
Cc: Wei Huang <wei@redhat.com>,
	Peter Maydell <peter.maydell@linaro.org>,
	Andrew Jones <drjones@redhat.com>,
	Eduardo Habkost <ehabkost@redhat.com>,
	Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Gabriel L. Somlo" <somlo@cmu.edu>,
	Ard Biesheuvel <ard.biesheuvel@linaro.org>,
	Shannon Zhao <zhaoshenglong@huawei.co>,
	QEMU Developers <qemu-devel@nongnu.org>,
	Roy Franz <roy.franz@linaro.org>,
	Igor Mammedov <imammedo@redhat.com>,
	Paolo Bonzini <pbonzini@redhat.com>,
	Richard Henderson <rth@twiddle.net>
Subject: Re: [Qemu-devel] dmidecode repository (Was: [ARM SMBIOS V1 PATCH 0/6] SMBIOS Support for ARM)
Date: Mon, 10 Aug 2015 13:58:31 +0200	[thread overview]
Message-ID: <55C891E7.8050003@redhat.com> (raw)
In-Reply-To: <20150810094300.4fb50dad@endymion.delvare>

On 08/10/15 09:43, Jean Delvare wrote:
> On Thu, 6 Aug 2015 10:07:40 +0200, Laszlo Ersek wrote:
>> On 08/06/15 00:03, Jean Delvare wrote:
>>> On Wed, 05 Aug 2015 22:39:57 +0300, Ivan Khoronzhuk wrote:
>>>> There is no git repo for dmidecode.
>>>> Only CVS: http://savannah.nongnu.org/cvs/?group=dmidecode
>>>
>>> Correct. Savannah support git now so it should be possible to convert
>>> the CVS repository to git, and I'm all for it if it makes users and
>>> potential contributors happy.
>>
>> Yes, please do that, if you can find the time.
>>
>>> Just I don't know how this is done and
>>> did not have the time to look into it so far.
>>
>> I've never done it myself, so the only thing I could do to help is
>> google it for you, which would be useless. :)
> 
> OK, I think I came up with something that looks reasonably good:
> 
> http://git.savannah.gnu.org/cgit/dmidecode.git
> 
> Can anyone please check it out and verify that it looks sane and can be
> worked with?

I cloned it and built it with "make". (That's all the "testing" I did. :))

Ideas:
- please consider tagging commits that correspond to releases

- probably useful to tag the git commit somehow that marks the switch
  from CVS to git (eg. "last_patch_from_cvs").

- after building, "git status" lists the *.o files and the built
  binaries as untracked files. For the former, please add a .gitignore
  file. For the latter, please list them individually in .gitignore too,
  or else build things in a separate directory, and ignore everything
  inside that directory.

> If it's OK then I'll tag the CVS repository as deprecated.

If you can ascertain that the latest tree in git (at
"last_patch_from_cvs") matches the latest tree in CVS (with a recursive
diff excluding the SCM meta-dirs), there's no reason to delay switching
to git. If you realize later that something's "wrong", you can format
the new patches from git and reapply them to CVS. (But I don't expect
anything to go wrong.)

Thanks
Laszlo

  reply	other threads:[~2015-08-10 11:58 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-28  6:00 [Qemu-devel] [ARM SMBIOS V1 PATCH 0/6] SMBIOS Support for ARM Wei Huang
2015-07-28  6:00 ` [Qemu-devel] [ARM SMBIOS V1 PATCH 1/6] smbios: extract x86 smbios building code into a function Wei Huang
2015-07-28  6:00 ` [Qemu-devel] [ARM SMBIOS V1 PATCH 2/6] smbios: remove dependency on x86 e820 tables Wei Huang
2015-07-28  6:00 ` [Qemu-devel] [ARM SMBIOS V1 PATCH 3/6] smbios: pass ram size as a parameter to build smbios tables Wei Huang
2015-07-28  6:00 ` [Qemu-devel] [ARM SMBIOS V1 PATCH 4/6] smbios: move smbios code into a common folder Wei Huang
2015-07-28  6:00 ` [Qemu-devel] [ARM SMBIOS V1 PATCH 5/6] smbios: add smbios 3.0 support Wei Huang
2015-07-31 17:12   ` Laszlo Ersek
2015-07-28  6:00 ` [Qemu-devel] [ARM SMBIOS V1 PATCH 6/6] smbios: implement smbios support for mach-virt Wei Huang
2015-07-31  2:11   ` Shannon Zhao
2015-07-31  6:08     ` Wei Huang
2015-08-05 17:16 ` [Qemu-devel] [ARM SMBIOS V1 PATCH 0/6] SMBIOS Support for ARM Laszlo Ersek
2015-08-05 17:35   ` Peter Maydell
2015-08-05 18:35     ` Laszlo Ersek
2015-08-05 19:39       ` Ivan Khoronzhuk
2015-08-05 22:03         ` Jean Delvare
2015-08-06  8:07           ` Laszlo Ersek
2015-08-06  8:16             ` Ivan Khoronzhuk
2015-08-06 11:20               ` Jean Delvare
2015-08-10  7:43             ` [Qemu-devel] dmidecode repository (Was: [ARM SMBIOS V1 PATCH 0/6] SMBIOS Support for ARM) Jean Delvare
2015-08-10 11:58               ` Laszlo Ersek [this message]
2015-08-10 12:26                 ` Jean Delvare
2015-08-10 15:08                   ` Laszlo Ersek
2015-08-06 12:41       ` [Qemu-devel] [ARM SMBIOS V1 PATCH 0/6] SMBIOS Support for ARM Andrew Jones
2015-08-07 11:12       ` Ard Biesheuvel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55C891E7.8050003@redhat.com \
    --to=lersek@redhat.com \
    --cc=ard.biesheuvel@linaro.org \
    --cc=drjones@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=imammedo@redhat.com \
    --cc=ivan.khoronzhuk@linaro.org \
    --cc=jdelvare@suse.de \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=roy.franz@linaro.org \
    --cc=rth@twiddle.net \
    --cc=somlo@cmu.edu \
    --cc=wei@redhat.com \
    --cc=zhaoshenglong@huawei.co \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).