Re: [PATCH 0/8] portability: partial port to Mac OS X

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday, July 20, 2010 15:05:27 François Revol wrote:
> Le 20 juil. 2010 à 20:58, Mike Frysinger a écrit :
> > On Tuesday, July 20, 2010 13:10:59 François Revol wrote:
> >> I currently need sfdisk to build SD card images of the ARM port of Haiku
> >> <http://haiku-os.org/>, and I noticed it wasn't available on OSX or
> >> other OSes, so I had a look at porting it. I tried to port the whole of
> >> util-linux-ng but I had to introduce some configure flags to exclude
> >> parts that aren't relevant anyway. The following patches attempt to
> >> make it more portable. I didn't try building on Haiku yet, but surely
> >> they will help there as well.
> > 
> > something is wrong with your mailer -- none of your e-mails have their
> > messages wrapped properly or the patch showing up inline.  please use
> > git-
> 
> wrapped ? as in ?

the bodies should be line wrapped to the correct length.  all of yours are 
wrapping when you decide to start a new line.

> > send-email because it does the right thing.
> 
> Except I'm not sure it will work here, git has the bad habit of doings
> things as if everyone was using linux (like, git-format-patch which
> doesn't formats patches but email bodies actually).

git-format-patch is doing the correct thing.  it needs to provide all the 
supplementary metadata so that it can be sucked back into another repo.  
formatting that info as an mbox is an advantage since it's already a well 
defined format.

instead of "fighting the system" and ultimately doing it wrong, use the tools 
already provided since they are known to do it right.
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.


[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux