Replace fails in file with mixed line endings about gulp-html-replace HOT 8 CLOSED

Ugh line endings again... Could you please provide this odd file so i can fix it once and for all?

from gulp-html-replace.

Sure,

but how should I do that? It seems, i cannot attach it here.

from gulp-html-replace.

Would splitting lines based on \n only (completely ignore \r) be the right approach here? If you do that the \r, if it's present, would just be part of the line. And you can handle any mixture of \n or \r\n lines. I don't think there is any need to actually test for \r\n.

from gulp-html-replace.

@ghost23 Maybe Dropbox, Google Drive worked pretty well in this comment too: #19 (comment) Any file sharing thing basically. Thank you.

@bmacnaughton It might work, but there may be some weird problems in rare cases. I better look at that file and run some tests before i do anything about it.

from gulp-html-replace.

That makes sense :)
https://www.dropbox.com/s/s5sej5grjr1fccv/line_endings-html.html?dl=0

from gulp-html-replace.

I think it works OK but you do have to make a decision on line 26 about what linefeed sequence to insert.

It wouldn't handle OLD Mac files with a \r line separator, but I don't think that really matters any longer.

from gulp-html-replace.

@bmacnaughton I'm using split by \n, this seems to be working ok, thanks for the idea. I think i better modify the code to not do "split" at all, but this is something for the future releases.

@ghost23, @charlescaron i've added your files to the "test" folder. I couldn't ask for a permission, there is no such thing as private messages on github, but if you're not ok with this (maybe those files are from some super-secret project or something) then please open an issue and i'll remove them.

from gulp-html-replace.

Thanks for the note. I think you're right - no split at all would be better but that's a bigger change. The history of line endings still makes life complicated.

from gulp-html-replace.