search

OpenAoE / vblade

The Minimal Userland AoE Target
GNU General Public License v2.0
51 stars 16 forks source link

freebsd mtu + bpf fix #1

Closed carton-of-mice closed 10 years ago

carton-of-mice commented 10 years ago

I admit I haven't read the pull request manual. Appears I can't cherry-pick on pull-requests. Sorry 'bout that.

ecashin commented 10 years ago

OK, I'm merging now.

We use mercurial all the time without any whitespace corruption, so I'm not sure what happened.

I'll take your advice and look into my buffer boundary concerns after merging and with reference to linux.c's example.

Thanks!

carton-of-mice commented 10 years ago

In this case, I exported the patch by piping the output of hg diff between my local revisions to a file. This replaced tabs with spaces and caused the problems. I will manually apply further patches to the git repo from now on.