Bug 251584 - games/rlvm: remove python2 requirement
Summary: games/rlvm: remove python2 requirement
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Jan Beich
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-12-04 18:40 UTC by Dmitry Marakasov
Modified: 2020-12-05 22:04 UTC (History)
0 users

See Also:
jbeich: maintainer-feedback+
jbeich: merge-quarterly+

Attachments
Patch (5.18 KB, patch)
2020-12-04 18:40 UTC, Dmitry Marakasov 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dmitry Marakasov freebsd_committer freebsd_triage 2020-12-04 18:40:44 UTC 
Created attachment 220254 [details]
Patch

Was just passing by and made a patch to fix build with python3 scons. Builds fine, but still probably needs testing.
Comment 1 commit-hook freebsd_committer freebsd_triage 2020-12-05 22:00:35 UTC 
A commit references this bug:

Author: jbeich
Date: Sat Dec  5 22:00:15 UTC 2020
New revision: 557088
URL: https://svnweb.freebsd.org/changeset/ports/557088

Log:
  games/rlvm: drop python2 dependency

  PR:		251584
  Submitted by:	amdmi3 (based on)

Changes:
  head/games/rlvm/Makefile
  head/games/rlvm/distinfo
  head/games/rlvm/files/patch-SConstruct
Comment 2 commit-hook freebsd_committer freebsd_triage 2020-12-05 22:03:36 UTC 
A commit references this bug:

Author: jbeich
Date: Sat Dec  5 22:02:58 UTC 2020
New revision: 557089
URL: https://svnweb.freebsd.org/changeset/ports/557089

Log:
  MFH: r557088

  games/rlvm: drop python2 dependency

  PR:		251584
  Submitted by:	amdmi3 (based on)

Changes:
_U  branches/2020Q4/
  branches/2020Q4/games/rlvm/Makefile
  branches/2020Q4/games/rlvm/distinfo
  branches/2020Q4/games/rlvm/files/patch-SConstruct
Comment 3 Jan Beich freebsd_committer freebsd_triage 2020-12-05 22:04:15 UTC 
Thanks for the help. With your patch:
- runtime still works fine, tested with CLANNAD
- TEST=on build needed one more "print" fix but otherwise all tests pass