qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Filip Navara <filip.navara@gmail.com>
Cc: Paolo Bonzini <bonzini@gnu.org>,
	Glauber Costa <glommer@redhat.com>,
	Jan Ondrej <ondrejj@salstar.sk>, Alexander Graf <agraf@suse.de>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] Re: [PATCH] Fix checksum writing in signboot.sh
Date: Sun, 02 Aug 2009 14:15:53 +0300	[thread overview]
Message-ID: <4A757569.9010309@redhat.com> (raw)
In-Reply-To: <5b31733c0908020325k3e035b91n3b0089a2f865de3@mail.gmail.com>

On 08/02/2009 01:25 PM, Filip Navara wrote:
>> While the patch is good, the code is unreadable.  Can we mandate python for
>> such tricks?
>>      
>
> No, please, no! Throwing additional tools at the problem is only going
> to make it worse for Windows users. I'm not happy with using sh script
> as it already added dependency on coreutils, but at least that's easy
> to install. Python is a nightmare compared to that.
>    

Is Python really so difficult to install under Windows?  How many times 
do you have to click 'Next'?

Note that Windows users can usually use prebuilt binaries, so the 'Next' 
nightmare only affects a small number of Windows developers.

> BTW, for years in ReactOS we had a way to build host tools with host
> CC and these tools were written in plain ordinary C. This worked great
> for both Windows and Linux builds and also for cross-compiling.
>    

But then you have to write those tools in C, which is annoying.

-- 
error compiling committee.c: too many arguments to function

  reply	other threads:[~2009-08-02 11:13 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-01  9:48 [Qemu-devel] [PATCH] Fix checksum writing in signboot.sh Alexander Graf
2009-08-02 10:04 ` [Qemu-devel] " Avi Kivity
2009-08-02 10:25   ` Filip Navara
2009-08-02 11:15     ` Avi Kivity [this message]
2009-08-02 11:58       ` Alexander Graf
2009-08-02 12:21         ` Avi Kivity
2009-08-02 21:29         ` [Qemu-devel] " Sebastian Herbszt
2009-08-03  2:30         ` [Qemu-devel] " Anthony Liguori
2009-08-03  6:12           ` Paolo Bonzini
2009-08-03 12:55           ` Avi Kivity

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=4A757569.9010309@redhat.com \
    --to=avi@redhat.com \
    --cc=agraf@suse.de \
    --cc=bonzini@gnu.org \
    --cc=filip.navara@gmail.com \
    --cc=glommer@redhat.com \
    --cc=ondrejj@salstar.sk \
    --cc=qemu-devel@nongnu.org \
    /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).