mysql_remove_eol_carret.patch change default path

[Yasufumi Kinoshita]

mysql_remove_eol_carret.patch

@@ -1434,6 +1436,10 @@
    NO_ARG, 1, 0, 0, 0, 0, 0},
   {"skip-line-numbers", 'L', "Don't write line number for errors.", 0, 0, 0, GET_NO_ARG,
    NO_ARG, 0, 0, 0, 0, 0, 0},
+ {"remove-eol-carret", OPT_REMOVE_EOL_CARRET, "Remove \\r before \\n in batch mode",
+ (uchar**)&opt_remove_eol_carret , (uchar**)&opt_remove_eol_carret, 0,
+ GET_BOOL,
+ NO_ARG, 0, 0, 0, 0, 0, 0},
   {"unbuffered", 'n', "Flush buffer after each query.", &unbuffered,
    &unbuffered, 0, GET_BOOL, NO_ARG, 0, 0, 0, 0, 0, 0},
   {"column-names", OPT_COLUMN_NAMES, "Write column names in results.",

--> opt_remove_eol_carret is 0.

@@ -51,7 +53,7 @@
   if (!(pos=intern_read_line(line_buff,&out_length, truncated)))
     return 0;
   if (out_length && pos[out_length-1] == '\n')
- if (--out_length && pos[out_length-1] == '\r') /* Remove '\n' */
+ if (--out_length && opt_remove_eol_carret && pos[out_length-1] == '\r') /* Remove '\n' */
       out_length--; /* Remove '\r' */
   line_buff->read_length=out_length;
   pos[out_length]=0;

--> if the default setting, the "if()" condition are changed to "always false".

Is it correct as intention of the patch?
Does it not affect to current users any?

[Stewart Smith (stewart)]

has fix been released? should this be "fix released"?

[Shahriyar Rzayev (rzayev-sehriyar)]

Percona now uses JIRA for bug reports so this bug report is migrated to: https://jira.percona.com/browse/PS-2554