From: Joel Brobecker <brobecker@adacore.com>
To: Michael <gdb-patches@cyberfiber.org>
Cc: gdb-patches@sourceware.org
Subject: Re: patch for testing purposes
Date: Fri, 01 Jan 2010 08:19:00 -0000 [thread overview]
Message-ID: <20100101081839.GQ2788@adacore.com> (raw)
In-Reply-To: <4B3DA076.8090508@cyberfiber.org>
> Here it is again. For your information, I added a few lines of code
> to the 'info float' statement. Hope you like them :)
You are going to hate me, but it's still not right. Sorry!
How are you generating your diff? From the markers in the diff patch,
it looks like you generated it without the assistance of a version-control
system. You are making things harder for you because you have to make sure
that the tree that you use as a reference matches exactly what you started
from. Otherwise, you end up with differences that should not be in
the patch, as you are seeing at least in your patch to the ChangeLog file.
I strongly recommend that you start the safest way: Checkout the sources
from either CVS or git, apply your changes to these sources, and then
use either CVS or git to generate the diff (either "cvs diff -up" or
"git diff"). I would both sending a diff to the ChangeLog file but
simply put the ChangeLog entry in the email body. You can have a look
at previous messages sent to this mailing-list as a guideline. Please
verify that the patch you are sending corresponds exactly to the changes
you made, no more, no less.
Please also make sure to say that you did change your test by running
the testsuite before and after your change, comparing the results,
and on which platform. I can help walking you through this process
if you need help. But first, let's make sure you know how to generate
the patch properly, and that you know how to present them to this
mailing-list.
I hope you're not feeling discouraged. This is only a matter of learning
the procedure, and once you know it, you won't have to go through this
again, for most if not all GNU projects.
--
Joel
next prev parent reply other threads:[~2010-01-01 8:19 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-30 7:57 Michael
2009-12-30 8:14 ` Joel Brobecker
2009-12-30 10:21 ` Eli Zaretskii
[not found] ` <4B3B3AB1.5060106@cyberfiber.org>
[not found] ` <20091230114518.GD548@adacore.com>
2009-12-31 9:30 ` Michael
2009-12-31 11:38 ` Michael
2009-12-31 11:51 ` Joel Brobecker
2009-12-31 22:09 ` Michael
2010-01-01 3:36 ` Joel Brobecker
2010-01-01 7:14 ` Michael
2010-01-01 8:19 ` Joel Brobecker [this message]
[not found] ` <20091231114605.GK2788@adacore.com>
2009-12-31 11:56 ` Michael
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=20100101081839.GQ2788@adacore.com \
--to=brobecker@adacore.com \
--cc=gdb-patches@cyberfiber.org \
--cc=gdb-patches@sourceware.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