On Thu, Dec 06, 2012 at 10:30:29PM -0600, Domsch, Matt wrote: > On Thu, Dec 06, 2012 at 09:47:01PM -0600, Craig Ringer wrote: > > On 6/12/2012 10:32 AM, Matt Domsch wrote: > > > > > The signurl patch needs a little work to keep it from breaking other > > > commands. > > It's fixed in an updated pull. Sorry about that. > > Thanks! I've pulled this into my merge branch now.
Craig, can you add a test to the run-tests.py test suite for the sign and signurl commands please? I copy the list, as: a) we need more tests in the test suite, especially for any new commands added b) the current test suite is broken as-is, first at the (no-)check-MD5 tests, and if those are skipped, at test 31 (Sync more from S3) which may be related. Can anyone step up to root cause and fix the test suite failures so it runs correctly to completion again? Until that's done, taking additional features will be a lower priority. Thanks, Matt -- Matt Domsch Technology Strategist Dell | Office of the CTO ------------------------------------------------------------------------------ LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial Remotely access PCs and mobile devices and provide instant support Improve your efficiency, and focus on delivering more value-add services Discover what IT Professionals Know. Rescue delivers http://p.sf.net/sfu/logmein_12329d2d _______________________________________________ S3tools-general mailing list S3tools-general@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/s3tools-general
