Thanks for your input. I'll see if I have time next week to brush something up via Joda!
--Flo On Tue, Aug 13, 2013 at 12:12 PM, Adrian Cole <[email protected]>wrote: > So, to summarize, there's basically 2 paths that involve a code change: > > - If you really want to do a global change (encouraged, but more > effort than a 3 line patch) > - Squash the duplicate commits here into one after adding unit > tests, and deprecating the iso8601Seconds* methods that are now served > by > the same thing > - Make another commit that updates the apis and providers that were > using iso8601Seconds* methods to use the now converged iso8601 method > - Scope the change to route53 > - here's what I think is the only affected file: > > https://github.com/jclouds/jclouds/blob/master/apis/route53/src/main/java/org/jclouds/route53/xml/ChangeHandler.java > - probably need to add another w/o milliseconds like this: > > https://github.com/jclouds/jclouds/blob/master/apis/route53/src/test/java/org/jclouds/route53/parse/GetChangeResponseTest.java > > — > Reply to this email directly or view it on > GitHub<https://github.com/jclouds/jclouds/pull/105#issuecomment-22589621> > . > -- --------------------------------------- t: http://twitter.com/flo s: florianleibert l: http://www.linkedin.com/in/florianleibert/ --------------------------------------- --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/105#issuecomment-22613529
