FYI I've created HARMONY-5494 about the issue in the port_user_timezone() implementation. I would be nice if someone from our DRLVM experts could find a time to take a look at it. Thanks, Alexei 2008/2/11, Tim Ellison : > Alexei Zakharov wrote: > > FYI I've marked the fix as a "workaround" in JIRA. I'll create new > > DRLVM issue with detailed description of the problem. I promise to > > roll back this workaround as soon as we have this issue fixed in > > DRLVM. Does this make sense? > > Sure > > Thanks, > Tim > > > 2008/2/11, Tim Ellison : > >> Alexey Petrenko wrote: > >>> 2008/2/11, Tim Ellison : > >>>> Is this the right place to fix the problem? Why not teach DRLVM to send > >>>> the timezone ID without the trailing newline? > >>> +1 > >>> > >>>> ayza@apache.org wrote: > >>>>> + if (zone.contains("\n")) { > >>>>> + zone = zone.substring(0, zone.indexOf("\n")); > >>>>> + } > >>> As far as I understood the string ends with "\n". > >>> If so I believe it's better use the following construction here: > >>> if (zone.endsWith("\n")) > >>> zone = zone.sunstring(0, zone.length() -1); > >> If we can fix DRLVM by M5 then I suggest we don't need the workaround at > >> all. Otherwise leave it in but leave the JIRA open and redirect it to > >> DRLVM with a reminder to remove the workaround when fixed properly. > >> > >> Regards, > >> Tim > >> > >> > >> > > >