On Mon, Apr 30, 2018 at 9:08 AM, Lukas Wunner <lu...@wunner.de> wrote: > On Mon, Apr 30, 2018 at 08:29:57AM +0200, Daniel Vetter wrote: >> We're relying on get fetch failing and set -e, but that doesn't work >> if we parse nothing: In that case git fetch just fetches upstream and >> succeed. Dave stumbled over this by feeding a .mbox with dos linefeeds >> into dim. > > How about eliminating the carriage returns (not linefeeds!) by > automatically funneling stdin through /usr/bin/tr -d '\r' > > (Sorry, I'm not familiar enough with dim source code and this > use case to provide a patch.)
Yeah that's easily doable, there's only a few entry points where we parse external mboxes. I'm just wondering whether it's worth the bother (since mbox files shouldn't have dos style line endings). -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dim-tools mailing list dim-tools@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dim-tools