Skip to contentSkip to conversation

Change the UDP Checksum code to work with ARM data alignment.

PATCH New Tom Evans 5 participants 9 messages Aug 27, 2008 1 message not in the archive

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

Tom Evans

author
#1 lore raw

PATCH 1/1 Change the UDP Checksum code to work with ARM data alignment.

Quoted 8 lines · Wolfgang Denk
Quoted 2 lines · In message <[email protected]> you wrote:
Quoted 2 lines

I followed Ben's instructions and here's the resulting patch.

I'm sorry, but this patch is corrupted and cannot be applied.

In the original code, there were no spaces but TAB characters used for indentation. Someone (you?)

Not me.

Quoted 1 line

Something is severely broken in your setup.

Windows or Thunderbird, take your pick. :-)

I'll try to email from one of our physical or virtual linux machines. If that doesn't work I'll have to send the patch as a zipped attachment. Or gzipped. Is this allowable or does it have to be "real email text"?

-
===
Tom Evans Tom.Evans at ceos.com.au
CEOS Pty Ltd www.ceos.com.au
3/17 Burgundy St, Heidelberg,
Victoria 3084, Australia

Phone (+61 3) 9458 4955
Direct Unsupported
FAX (+61 3) 9458 4966
Mobile 0405 776 431

#2 lore raw

PATCH 1/1 Change the UDP Checksum code to work with ARM data alignment.

Hi Tom,

Quoted 12 lines · Tom Evans
Quoted 8 lines · Wolfgang Denk
Quoted 2 lines · In message <[email protected]> you wrote:
Quoted 2 lines

I followed Ben's instructions and here's the resulting patch.

I'm sorry, but this patch is corrupted and cannot be applied.

In the original code, there were no spaces but TAB characters used for indentation. Someone (you?)

Not me.

Quoted 8 lines
Quoted 1 line

Something is severely broken in your setup.

Windows or Thunderbird, take your pick. :-)

I'll try to email from one of our physical or virtual linux machines. If that doesn't work I'll have to send the patch as a zipped attachment. Or gzipped. Is this allowable or does it have to be "real email text"?

The only 100%* way to send patches is using git-send-email, but it's a small pain to set up especially if you're not going to be using it much. I've had decent luck with Thunderbird, as long as you select 'Preformat' for the patch part of your e-mail and send it as plain text. Please try this. If it still doesn't work I'll fix it manually.

regards,
Ben

#3 lore raw

PATCH 1/1 Change the UDP Checksum code to work with ARM data alignment.

Dear Ben Warren,

Quoted 3 lines · In message <[email protected]> you wrote:

The only 100%* way to send patches is using git-send-email, but it's a small pain to set up especially if you're not going to be using it much.

oops? What do you mean by "pain to set up" ? You don;t have to set up anything.

Just type:

git-send-email --to u-boot at lists.denx.de patch...

Quoted 3 lines

I've had decent luck with Thunderbird, as long as you select 'Preformat' for the patch part of your e-mail and send it as plain text. Please try this. If it still doesn't work I'll fix it manually.

You do not need Thunderbird or any other MUA at all.

Best regards,

Wolfgang Denk

Signature · 6 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Eeeek!
'eval' on strings should have been named 'evil'.    -- Tom Phoenix in
        <[email protected]>
#4 lore raw

PATCH 1/1 Change the UDP Checksum code to work with ARM data alignment.

Quoted 9 lines · Wolfgang Denk
Quoted 2 lines

The only 100%* way to send patches is using git-send-email, but it's a small pain to set up especially if you're not going to be using it much.

oops? What do you mean by "pain to set up" ? You don;t have to set up anything.

Just type:

git-send-email --to u-boot at lists.denx.de patch...

You forgot about the configuration you have done in your ~/.gitconfig file ([sendemail] section, smtpserver...). But is not really hard to configure. And it really is worth the effort.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================

#5 lore raw

PATCH 1/1 Change the UDP Checksum code to work with ARM data alignment.

Dear Stefan Roese,

Quoted 8 lines · In message <[email protected]> you wrote:
Quoted 4 lines

Just type:

git-send-email --to u-boot at lists.denx.de patch...

You forgot about the configuration you have done in your ~/.gitconfig file ([sendemail] section, smtpserver...). But is not really hard to configure. And it really is worth the effort.

No, I did not forget this:

	-> ls -l ~/.gitconfig
	ls: cannot access /home/wd/.gitconfig: No such file or directory

Best regards,

Wolfgang Denk

Signature · 5 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Testing can show the presense of bugs, but not their absence.
                                                   -- Edsger Dijkstra
#6 lore raw

PATCH 1/1 Change the UDP Checksum code to work with ARM data alignment.

Quoted 4 lines · Wolfgang Denk
Quoted 4 lines
Quoted 3 lines

Just type:

git-send-email --to u-boot at lists.denx.de patch...

Quoted 8 lines
Quoted 3 lines

You forgot about the configuration you have done in your ~/.gitconfig file ([sendemail] section, smtpserver...). But is not really hard to configure. And it really is worth the effort.

No, I did not forget this:

	-> ls -l ~/.gitconfig
	ls: cannot access /home/wd/.gitconfig: No such file or directory

Hmmm. Then I'm really curious how git-send-email know to which SMTP server to send this mail.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================

#7 lore raw

PATCH 1/1 Change the UDP Checksum code to work with ARM data alignment.

Dear Stefan Roese,

Quoted 3 lines · In message <[email protected]> you wrote:

Hmmm. Then I'm really curious how git-send-email know to which SMTP server to send this mail.

It runs "sendmail" to send the message, assuming you have a working mail configuration on your host.

Best regards,

Wolfgang Denk

Signature · 4 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
This is now.  Later is later.
#8 lore raw

PATCH 1/1 Change the UDP Checksum code to work with ARM data alignment.

Quoted 6 lines · Wolfgang Denk

Dear Stefan Roese,

Quoted 2 lines · In message <[email protected]> you wrote:

Hmmm. Then I'm really curious how git-send-email know to which SMTP server to send this mail.

Quoted 6 lines

It runs "sendmail" to send the message, assuming you have a working mail configuration on your host.

Best regards,

Wolfgang Denk

"...assuming you have a working mail configuration on your host." Winders hosts don't have a working mail configuration ("sendmail" or localhost port 25 support), which means that the people that need git-send-email worst (those stuck in a Winders-centric corporate borg) require setup and luck.

If you are a borger (that would be Germanization of "lives in a borg," IIRC), your Exchange server may support port 25 (IIUC, it is an optional configuration item, but probably configured).

You first need to know the name of your Exchange server.
1. In Outlook, select the menu selection Tools/Email Accounts which brings up a dialog allowing you to "View or change existing e-mail accounts"
2. Click the "Next" button which brings up the next dialog box.
3. Select the "Microsoft Exchange Server" account and click the "Change" button.
4. This dialog box shows your Exchange server name. Hopefully it supports port 25 communications.
5. Click the "Cancel" button to exit without changing your Outlook setup.

You may need to authenticate, in which case you would use "domain\user" (where "domain" is your ActiveDirectory domain, "user" is your user name, and the backslash *is* a backslash and may need to be quoted/doubled).

I have done this with Thunderbird[1] on Windows (check my headers, but please don't tell the IT staff ;-), but I have *not* done it with git-send-email. If anybody tries this, please let us know how it worked out (success or failure).

Thanks & good luck,
gvb

[1] The Exchange server I connect to actually supports IMAP as well as port 25 and it works very well (surprisingly).

#9 lore raw

PATCH 1/1 Change the UDP Checksum code to work with ARM data alignment.

In reply to Wolfgang Denk
Quoted 8 lines · Wolfgang Denk

Dear Ben Warren,

Quoted 3 lines · In message <[email protected]> you wrote:

The only 100%* way to send patches is using git-send-email, but it's a small pain to set up especially if you're not going to be using it much.

Quoted 7 lines

oops? What do you mean by "pain to set up" ? You don;t have to set up anything.

Just type:

git-send-email --to u-boot at lists.denx.de patch...

Sure, if you have easy access to an SMTP server and have sendmail running it's trivial. If you use gmail, there are a few hoops you need to jump through regarding authentication. All-in-all, worth the effort if you're going to do it more than once.

regards,
Ben