# Update esutil to version that works on Python 3

XMLWordPrintable

#### Details

• Type: Story
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
• Story Points:
1

#### Activity

Hide
Tim Jenness added a comment -

It looks like v0.6.0 is now out and compatible with python 3: https://github.com/esheldon/esutil/releases/tag/v0.6.0

Show
Tim Jenness added a comment - See https://github.com/esheldon/esutil/issues/17 It looks like v0.6.0 is now out and compatible with python 3: https://github.com/esheldon/esutil/releases/tag/v0.6.0
Hide
Tim Jenness added a comment -

Builds fine for me on python 3. Please remember to tag it when you merge.

Show
Tim Jenness added a comment - Builds fine for me on python 3. Please remember to tag it when you merge.
Hide
Nate Lust added a comment -

What tag are you referring to? the eups version tag? I am not sure how to do that.

Show
Nate Lust added a comment - What tag are you referring to? the eups version tag? I am not sure how to do that.
Hide
Tim Jenness added a comment - - edited

 $git checkout master $ git merge --no-ff tickets/DM-7294 $git push $ git tag -a 0.6.0 $git push --tags  Standard process for working with third party packages: https://developer.lsst.io/build-ci/third_party.html#distributing-the-package Show Tim Jenness added a comment - - edited$ git checkout master $git merge --no-ff tickets/DM-7294$ git push $git tag -a 0.6.0$ git push --tags Standard process for working with third party packages: https://developer.lsst.io/build-ci/third_party.html#distributing-the-package
Hide
Nate Lust added a comment -

ah ok, its just a git tag, good to know. I will read over the link before merging just to be sure I cover everything

Show
Nate Lust added a comment - ah ok, its just a git tag, good to know. I will read over the link before merging just to be sure I cover everything
Hide
John Swinbank added a comment -

Changes look ok. But...

I noticed all this guff in eupspkg.cfg.sh. This seems like it's unnecessary, since esutil 0.6 doesn't ship with its own table file as far as I can see. Can we drop it?

Show
John Swinbank added a comment - Changes look ok. But... I noticed all this guff in eupspkg.cfg.sh . This seems like it's unnecessary, since esutil 0.6 doesn't ship with its own table file as far as I can see. Can we drop it?
Hide
Nate Lust added a comment -

I checked into this as well, and ran it through ci with the prep function removed. If you agree with the latest change then I will merge.

Show
Nate Lust added a comment - I checked into this as well, and ran it through ci with the prep function removed. If you agree with the latest change then I will merge.
Hide
Nate Lust added a comment -

merged to master

Show
Nate Lust added a comment - merged to master

#### People

Assignee:
Nate Lust
Reporter:
Tim Jenness
Reviewers:
John Swinbank
Watchers:
John Swinbank, Nate Lust, Simon Krughoff, Tim Jenness