#8379 py3: forgehg

unreleased
closed
py3 (9)
General
nobody
2020-11-12
2020-08-27
No

These aren't part of core Allura for licensing reasons, but would be good to keep maintaining them.

Discussion

  • Dave Brondsema

    Dave Brondsema - 2020-08-27
    • labels: --> py3
     
  • Dave Brondsema

    Dave Brondsema - 2020-11-05
    • summary: forgehg, forgepastebin, tracwikiimporter --> py3: forgehg
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,3 +1 @@
    -These aren't part of core Allura for licensing and demonstration reasons, but would be good to keep maintaining them.
    -
    -TracWikiImporter doesn't have much test coverage, and doesn't get used much at all.  Main Allura codebase has a test that uses it: forgeimporters.trac.tests.functional.test_trac:TestTracImportController
    +These aren't part of core Allura for licensing reasons, but would be good to keep maintaining them.
    
    • status: open --> review
    • assigned_to: Dave Brondsema
     
  • Dave Brondsema

    Dave Brondsema - 2020-11-05

    allura,forgehg:db/8379

    nosetests must be run with --nocapture on py3, due to nosetests replacing sys.stdout with a simple object that doesn't have .buffer attribute that Mercurial expects

    allura branch has some misc other py3 fixes included

     

    Last edit: Dave Brondsema 2020-11-05
  • Dave Brondsema

    Dave Brondsema - 2020-11-05

    Also I ended up using pip install -e . instead of python setup.py develop in the forgehg dir. Its the newer pip-friendly equivalent, not sure if its always needed but I had some issues with entry points not being found with the setup.py version

     
    • Ingo

      Ingo - 2020-11-06

      Thanks Dave!
      This was an important upgrade info.

      As I am planning a smoke test with python 3 for my site, I will need this
      for sure.

       
      • Dave Brondsema

        Dave Brondsema - 2020-11-06

        Cool 😄 I think with this now Allura is pretty much python3 ready, but hasn't gotten a lot of real-world testing yet. So let us know if you have any issues.

         
  • Dillon Walls

    Dillon Walls - 2020-11-12

    Looks good to me!

     
  • Dave Brondsema

    Dave Brondsema - 2020-11-12
    • status: review --> closed
     

Log in to post a comment.