Project

General

Profile

Bug #11522

illumos-gate has some runtime dependencies on Python 2.7

Added by Alexander Pyhalov 6 months ago. Updated 14 days ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:

Description

illumos-gate packages still has some runtime dependencies on Python 2.7, in particular test-runner and pyzfs depends on it.


Related issues

Related to illumos gate - Bug #12192: python3 modules treat strings incorrectlyClosed

Actions

History

#1

Updated by Andy Fiddaman 6 months ago

I can see the dependency for test-runner, but pyzfs should be ok..

bloody% pkgtree -U dependants python-27
| pkg://omnios/system/test/testrunner@0.5.11-151031.0:20190723T232757Z
|------(    require)--pkg://omnios/runtime/python-27@2.7.16-151031.0

bloody% head -1 /usr/lib/zfs/pyzfs.py
#!/usr/bin/python3.5 -S
#2

Updated by Alexander Pyhalov 6 months ago

This piece is fixed in illumos-omnios.

#3

Updated by Dan McDonald 14 days ago

Testing done (as submitted in the RTI mail):

1) Built gate with BUILDPY2='#'
- pyzfs.py and /opt/test-runner/bin/run have python3.5 shebang
- /opt/util-tests/bin/utiltest passes (when /opt/util-tests/runfiles/openindiana.run symlink to default.run is created)
- the following commands
/usr/lib/zfs/pyzfs.py allow user mount rpool/export/home/userhome
/usr/lib/zfs/pyzfs.py allow rpool/export/home/userhome
/usr/lib/zfs/pyzfs.py unallow user mount rpool/export/home/userhome
work
- Run the following with Python 3.5
import libbe_py as be
be.beCopy('test', 'illumos-py3',None, None, {'ru.rsu.cc:test':'value'})

New BE is created, zfs dataset has ru.rsu.cc:test property set to value.

2) Built gate with unset BUILDPY2, BUILDPY3
- pyzfs.py and /opt/test-runner/bin/run have python2.7 shebang
- /opt/util-tests/bin/utiltest passes (when /opt/util-tests/runfiles/openindiana.run symlink to default.run is created)
- the following commands
/usr/lib/zfs/pyzfs.py allow user mount rpool/export/home/userhome
/usr/lib/zfs/pyzfs.py allow rpool/export/home/userhome
/usr/lib/zfs/pyzfs.py unallow user mount rpool/export/home/userhome
work
- Run the following with Python 2.7
import libbe_py as be
be.beCopy('test2', 'illumos-py3',None, None, {'ru.rsu.cc:test2':'value2'})

New BE is created, zfs dataset has ru.rsu.cc:test2 property set to value2.

#4

Updated by Dan McDonald 14 days ago

  • Related to Bug #12192: python3 modules treat strings incorrectly added
#5

Updated by Electric Monk 14 days ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

git commit b0858fdc3790ef1d1f955bd584621c6fd539050d

commit  b0858fdc3790ef1d1f955bd584621c6fd539050d
Author: Alexander Pyhalov <apyhalov@gmail.com>
Date:   2020-01-15T20:23:30.000Z

    11522 illumos-gate has some runtime dependencies on Python 2.7
    12192 python3 modules treat strings incorrectly
    Reviewed by: Andrew Stormont <andyjstormont@gmail.com>
    Reviewed by: Andy Fiddaman <omnios@citrus-it.co.uk>
    Reviewed by: Hans Rosenfeld <rosenfeld@grumpf.hope-2000.org>
    Reviewed by: Toomas Soome <tsoome@me.com>
    Approved by: Dan McDonald <danmcd@joyent.com>

Also available in: Atom PDF