[lkml]   [2002]   [Jan]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: Why not "attach" patches?
    In article <005901c19dec$59a89e30$0201a8c0@HOMER>,
    Martin Eriksson <> wrote:
    >Why do many of you not _attach_ patches instead of merging them with the
    >mail? It's so much cleaner and easier to have a "xxx-yyy.patch" file
    >attached to the mail which can be saved in an appropriate directory. Also,
    >the whitespace is always retained that way.

    Attached patches are _horrible_ once you have many patches that you want
    to maintain in a sane way and apply in one go.

    In particular, with in-line text patches, I can:

    - see the patch easily when reading email, without the need to do
    anything special to inspect the attachment, regardless of what email
    client I happen to use.

    - keep emails as emails, and save them to folders etc, without
    losing any information of where the patch came from, while at the
    same time the folders are _also_ the patch and work with standard
    tools like "diffstat".

    - easily just "reply" to the person, and quote the part of the patch
    I have problems with.

    - save all the emails I want to apply in one single email folder
    ("doit"), and do a simple

    patch -p1 < ~/doit

    to apply all of them at the same time.

    Note that NONE of these are practical with attachments.

    In short: if your mailer eats whitespace or causes similar corruption,
    just FIX THE MAILER. There is no excuse for a mailer that corrupts the

    And while attachments may _appear_ convenient, they most definitely are
    not. They require special care and cannot be batched or edited with
    normal tools.

    That may not matter if you just have one or two patches a week to worry
    about, but trust me - attachments are crap. Use them for binary data
    that cannot be edited or combined, _not_ for stuff you expect to be able
    to actually change and extract pieces of with regular tools.

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:23    [W:0.023 / U:34.284 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site