On Wed, Jun 19, 2019 at 11:27:41PM +0530, Shekhar Sharma wrote: > This patch changes the file to run on both python2 and python3. > > The tempfile module has been imported and used. > Although the previous replacement of cmp() by eric works, > I have replaced cmp(a,b) by ((a>b)-(a<b)) which works correctly. Any reason not to use Eric's approach? This ((a>b)-(a<b)) is confusing. > Thanks! BTW, strictly place your patch description here. Things like "Thanks!" to someone specifically and the cmp() explanation should go below the --- marker, like versioning. BTW, Cc Eric Garver in your patches, he's helping us with reviewing :-)