Menu

#11 The test case SimplePluginAdvancedManipulationTestsCase fails sometimes in python3.3

Next Minor Release
closed
nobody
python3.3 (1)
1
2013-05-22
2013-05-09
Anonymous
No

In python3.3, the test case SimplePluginAdvancedManipulationTestsCase fails sometimes. I think it is because you pass the categories as a dict and the order in a dictionary is not insured. I attach a patch for fix it.

Discussion

  • Henze  Agustin

    Henze Agustin - 2013-05-09

    Here the patch

     
    • Thibauld Nion

      Thibauld Nion - 2013-05-13

      Thanks for the report, this looks like a silly bug, so much so that I wanted to take some time and check wether the failure and apparent sillyness was due to a regression in the lib's code.

      It seems it's just pure sillyness...

      I've looked at your patch and it's ok, thanks for that !

      I've also seen this is related to a Debian bug: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=707162

      Do you need something else quickly (like a new release) or a "formal" acceptance is enough ? (I'll see if I can make a release in the next days, but it often happens that I can't find time to do so as quickly as I'd like...)

       
      • Henze  Agustin

        Henze Agustin - 2013-05-13

        Hi Thibauld, I'm the maintainer of .deb package python-yapsy :).
        About the fix, would be great a new release but if you can't make it on a week or so I'll apply the patch.

        Thanks

         
  • Thibauld Nion

    Thibauld Nion - 2013-05-21
    • status: open --> pending
     
  • Thibauld Nion

    Thibauld Nion - 2013-05-21

    The patch has been applied to the sources but I took this occasion to configure travis-ci for yapsy. If everything goes well, there'll be a release tomorrow.

     
  • Thibauld Nion

    Thibauld Nion - 2013-05-22
    • status: pending --> closed
     
  • Thibauld Nion

    Thibauld Nion - 2013-05-22

    fixed in release 1.10.2

     

Anonymous
Anonymous

Add attachments
Cancel





Auth0 Logo