On 12/22/2017 4:22 PM, Thomas Frohwein wrote:
Great job! Since I also worked in this one for a while (what a coincidence ;-) 
here my comments:

- I played through the first 5-10 minutes on my build on amd64 without issues 
(besides fullscreen/resolution)

Fullscreen worked ok for me in multiplayer mode but not single player mode. Upstream says that single player mode is less polished, so I'm willing to believe that is part of being less polished.

- crashes (X window closes) when trying to switch resolution from the default 
800x600. Not sure if there is a working commandline option to change the 
resolution, especially on displays with higher resolution

It does not crash for me. I was able to change the resolution to 640x480 in single player. I would not try anything higher, seeing as my monitor is only 1024x600.

- tons of strcat/strcpy/sprintf, however my attempts at replacing with safer 
memory functions led to too many bugs, including subtle ones so that weeding 
this out would be too time-consuming for me at this point

I'm not really sure why you tried this in the first place. I just took a look at what you did on openbsd-wip and it was far too much. Maybe if you were doing it step-by-step with upstream, then fine. But not as one gigantic patch.

- upstream is interested in "unobtrusive" bsd support 
(https://github.com/JACoders/OpenJK/issues/948). It might be worth to submit a pull 
request with your patches

Why? Only 2.5 of the patches are meaningful to upstream. The rest are dealing with their stupid decision to put the architecture name in their binaries and have nothing to do with *BSD support. I will submit the meaningful patches only.

- Upstream instructions suggest renaming the executables to jasp (for 
singleplayer) and jamp (for multiplayer) - this may be a consideration for 
renaming the wrapper scripts

Yeah that's fine.

Unified diff for typos in DESCR:

Go complain to GOG.com, that's their description of the game :)

~Brian

$ diff -u pkg/DESCR{.orig,}
--- pkg/DESCR.orig      Fri Dec 22 13:19:44 2017
+++ pkg/DESCR   Fri Dec 22 13:21:01 2017
@@ -1,9 +1,9 @@
  As a new student of Luke Skywalker's Jedi Academy, follow an ancient
  tradition and learn the powers - and dangers - of the Force.
-As a Dark Jedi Cult knows as the Disciples of Ragnos threaten the
+As a Dark Jedi Cult known as the Disciples of Ragnos threaten the
  galaxy, Jaden Korr, a student of Luke Skywalker's Jedi Academy must
-learn the ways of the jedi in order to defend it. On the path to
+learn the ways of the Jedi in order to defend it. On the path to
  becoming a Jedi Master Jaden must fight through various missions taking
  place accross the galaxy, from Hoth to Tattoine. Upon each victory,
  Jaden's powers grow until eventually becoming an apprentice to Kyle


--------------------------------------------
On Fri, 12/22/17, Brian Callahan <[email protected]> wrote:

  Subject: NEW: games/openjk
  To: "[email protected]" <[email protected]>
  Date: Friday, December 22, 2017, 7:27 AM
Hi ports -- Attached is a new port, games/openjk.
  OpenJK is an open source
  reimplementation of the Jedi Academy
  game from LucasArts. I've had this
  sitting in my tree for a while, finally
  got it to play for the first
  time today.
---
  pkg/DESCR:
  As a new student of Luke Skywalker's
  Jedi Academy, follow an ancient
  tradition and learn the powers - and
  dangers - of the Force.
As a Dark Jedi Cult knows as the
  Disciples of Ragnos threaten the
  galaxy, Jaden Korr, a student of Luke
  Skywalker's Jedi Academy must
  learn the ways of the jedi in order to
  defend it. On the path to
  becoming a Jedi Master Jaden must fight
  through various missions taking
  place accross the galaxy, from Hoth to
  Tattoine. Upon each victory,
  Jaden's powers grow until eventually
  becoming an apprentice to Kyle
  Katarn, and must fight the temptation
  of turning to the dark side.
Interact with famous classic Star Wars
  locations and characters while
  facing the ultimate choice: fight for
  good and freedom in the light side
  or follow the path of power and evil to
  the dark side. Create your own
  characters by defining various
  characteristics, such as species, gender,
  clothing and physical attributes
  allowing for a more personal experience
  before entering the academy to learn
  the powers - and dangers - of the
  Force. Construct your own lightsaber
  from handle to blade color. Unique
  tiered level selection allows you to
  choose you adventure and the
  missions you take based on the skills
  you need to become the ultimate
  Jedi Knight.
  ---
It's another game that's available on
  GOG.com -- and on pretty good sale
  too: https://www.gog.com/game/star_wars_jedi_knight_jedi_academy
Upstream says that the multiplayer is
  in a better state than the single
  player. I was able to start up both,
  play both, and watch the opening
  cinematics of the single player game.
  The binaries have different names
  depending on if you want to play
  multiplayer or single player. As
  always, follow the pkg/README for setup
  instructions.
I had the tiniest of audio glitches
  with the Lucas Arts logo when
  starting the single player binary, but
  the in-game audio was fine.
Fullscreen mode does not appear to work
  in the single player binary, but
  works fine in the multiplayer binary.
  Go figure.
OK? ~Brian

Reply via email to