Re: New: games/serioussam-vk

[Alexander Pavlov]

Hello. Today I installed OpenBSD on a computer with a KAVERI APU (A10-7800)
with an integrated  R7 graphics core supporting Vulkan. I had to take a
bite out of Gentoo and install OpenBSD in the free space. Unfortunately,
only the Radeon driver worked for me on this APU. It works well, with all
the accelerations, but without Vulkan support, unlike Gentoo, where the
AMDGPU driver with Vulkan support works. Therefore, I can’t check the
functionality of the game on Vulkan. I think one available "serioussam"
port on OpenBSD is enough.

Happy Christmas and New Year everyone!
Best Regards. Alexander.

пт, 15 дек. 2023 г. в 09:49, Alexander Pavlov :

> Hello.
> I closed the pull request. When I have the opportunity, I'll test it on
> real hardware.
>
> Best Regards.
> Alexander.
>
> чт, 14 дек. 2023 г. в 10:07, Alexander Pavlov :
>
>> Yes, I noticed this now. My mistake. The sam_iGfxAPI variable is
>> responsible for this. By default it is equal to one and the vulkan starts.
>> If the vulkan is not detected when the game starts, then it switches to
>> OpenGL. This variable can be set to zero in the configuration files, then
>> the OpenGL render will be the default. This variable can be set to zero in
>> the configuration file
>> (~/.local/share/Serious-Engine/serioussam/Scripts/PersistentSymbols.ini)
>> then the OpenGL render will be the default. This file is generated on first
>> launch. You need to change the sam_iGfxAPI value to zero or write the line:
>> persistent extern INDEX sam_iGfxAPI=(INDEX)0;
>>
>> чт, 14 дек. 2023 г. в 02:56, Thomas Frohwein :
>>
>>> On Wed, Dec 13, 2023 at 10:34:01AM +0300, Alexander Pavlov wrote:
>>> > Hello
>>> > You can attach the game log.
>>> > ~.local/share/Serious-Engine/serioussam/SeriousSam.log
>>> >
>>> > It looks like I was in a hurry to add this port to OpenBSD without
>>> testing
>>> > it on real hardware with Vulkan support. Or Serious Sam has no luck
>>> with
>>> > integrated Intel graphics. This is already the second case. The first
>>> one
>>> > had graphic artifacts on the OpenGL renderer.
>>>
>>> How do you run with opengl? I can't find anything about this in the man
>>> page or the README.
>>>
>>> > I have a video card with Vulkan support on one computer. When I have
>>> the
>>> > opportunity, I’ll try to test it on it, but for now it’s probably
>>> better
>>> > for me to close the pull request, because without testing on real
>>> hardware
>>> > it’s difficult to understand the problem.
>>> >
>>> > ср, 13 дек. 2023 г. в 00:35, Kirill Bychkov :
>>> >
>>> > > On Tue, December 12, 2023 22:36, Thomas Frohwein wrote:
>>> > > > On Mon, Dec 11, 2023 at 10:52:58PM +0300, Alexander Pavlov wrote:
>>> > > >> Hello!
>>> > > >> Thanks @Kirill for approved the pull request.
>>> > > >>
>>> > > >> This is my other ever port for OpenBSD. It's the open source
>>> Serious Sam
>>> > > >> Classic engine with vulkan support. The port has two renders
>>> OpenGL and
>>> > > >> Vulkan. Render switches to the game menu.
>>> > > >>
>>> > > >> Tested using the OpenGL renderer, since I do not have the
>>> opportunity to
>>> > > >> test the Vulkan renderer on OpenBSD. On the computer where
>>> OpenBSD is
>>> > > >> installed, the Radeon HD3000 does not support Vulkan.
>>> > > >>
>>> > > >> I wish to get this added to the official ports tree, so please
>>> leave
>>> > > >> any feedback or suggestions on what can be improved. Thank you!
>>> > > >>
>>> > > >> Source on Github:
>>> > > >> https://github.com/tx00100xt/SeriousSamClassic-VK
>>> > > >
>>> > > > Thanks for submitting this:
>>> > > >
>>> > > >> @conflict serioussam-*
>>> > > >
>>> > > > Is this needed? We don't have any other serioussam ports as far as
>>> I
>>> > > > know.
>>> > >
>>> > > Yep, if we import both ports - serioussam and serioussam-vk
>>> > >
>>> > > >
>>> > > > DESCR should be formatted, typically with fmt -w 80...
>>> > > > README also should be formatted similarly. This means that the line
>>> > > > with "directories:" should be pulled up, even if you're going with
>>> > > > fmt -w 72 only.
>>> > > >
>>> > > > Looks like SamTFE/Sources/CMakeLists.txt and
>>> > > > SamTSE/Sources/CMakeLists.txt add their own -O2 and -pipe which is
>>> > > > better left to our ports build system.
>>> > > >
>>> > > > I'm getting a segfault in vulkan when trying to launch:
>>> > > >
>>> > > > $ serioussam
>>> > > > STUBBED: load window icon in
>>> > > >
>>> > >
>>> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
>>> > > > line 176.
>>> > > > STUBBED: Need SDL invisible window or something in
>>> > > >
>>> > >
>>> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
>>> > > > line 357.
>>> > > > STUBBED: !!! FIXME: get the code back in from Ryan's original
>>> port. in
>>> > > >
>>> > >
>>> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Base/Stream.cpp,
>>> > > > line 995.
>>> > > > 

Re: New: games/serioussam

[Kirill Bychkov]

On Mon, December 4, 2023 15:03, Kirill Bychkov wrote:
> On Sun, December 3, 2023 22:18, Stefan Hagen wrote:
>> Stefan Hagen wrote (2023-12-03 18:38 CET):
>>> Morgan Aldridge wrote (2023-12-03 17:04 CET):
>>> > On Sun, Dec 3, 2023 at 09:57 Александр Павлов 
>>> wrote:
>>> >
>>> > > Hello!
>>> > > This is my first ever port for OpenBSD. It's the open source Serious
>>> Sam
>>> > > Classic engine.
>>> > >
>>> > > Tested on amd64 and i386 architectures. Works just as well as on Linux
>>> or
>>> > > FreeBSD.
>>> > >
>>> > > I wish to get this added to the official ports tree, so please leave
>>> > > any feedback or suggestions on what can be improved. Thank you!
>>> > >
>>> > > Source on Github:
>>> > > https://github.com/tx00100xt/SeriousSamClassic
>>> > >
>>> >
>>> > Excellent and thanks! We were just discussing your work in #openbsd-gamin
>>> > on irc.libera.chat yesterday.
>>> >
>>> > There was no attachment for your submission and a .tgz containing the new
>>> > port directory is preferred for new ports. I saw you submitted a PR to an
>>> > individual's WIP ports repo on GitHub:
>>> >
>>> > 
>>> >
>>> > I haven't tested, but I'd suggest tarballing that (less the 'FINISHED'
>>> > file) and attaching it here for easier testing by porters.
>>>
>>> Ahh so many memories. I played quite a bit of ssam on lan parties.
>>> Back then, when monitors were small and heavy. :-)
>>>
>>> Tarball attached with the following changes:
>>>
>>> - changed to DIST_TUPLE
>>> - Added missing RUN_DEPENDS
>>> - Uncommented the aarch64 + armv7 bits (let's try to build it)
>>> - Removed REVISION (not needed on a new port)
>>> - simplified the README
>>> - some indentation changes
>>> - update-plist (for @tag update-desktop-database after adding RDEPS)
>>
>> I forgot to remove ONLY_FOR_ARCH, thanks mbuhl@ for spotting.
>>
>> There was also a trailing \ at LIB_DEPENDS.
>> While there, I also sorted *DEPENDS and WANTLIB.
>> And I changed the COMMENT wording to read a bit more fluent in English.
>>
>> @Alexander: This is a pretty good first contribution. All the stuff I'm
>> changeing is pretty much nit picking. The port worked on the first try!
>>
>> Updated tarball attached.
>>
>> Stefan
>>
>>
> Hi!
> Some tweaks:
>  - no need for N variable which is used only once
>  - no need for HOMEPAGE - its dealt with TUPLE mechanics
>  - No need fir DISTNAME
>  - no need for @conflict marker in PLIST
>  - fix typo in README
>  - add a comment for COMPILER
>  - set SHARED_LIBS for lib versioning
>
> Works fine for me on amd64. ok kirby@ to import.
> Updated tarball attached.

Looking for another OK :)

serioussam.tar.gz
Description: application/gzip

Re: New: games/serioussam-vk

[Alexander Pavlov]

Hello.
I closed the pull request. When I have the opportunity, I'll test it on
real hardware.

Best Regards.
Alexander.

чт, 14 дек. 2023 г. в 10:07, Alexander Pavlov :

> Yes, I noticed this now. My mistake. The sam_iGfxAPI variable is
> responsible for this. By default it is equal to one and the vulkan starts.
> If the vulkan is not detected when the game starts, then it switches to
> OpenGL. This variable can be set to zero in the configuration files, then
> the OpenGL render will be the default. This variable can be set to zero in
> the configuration file
> (~/.local/share/Serious-Engine/serioussam/Scripts/PersistentSymbols.ini)
> then the OpenGL render will be the default. This file is generated on first
> launch. You need to change the sam_iGfxAPI value to zero or write the line:
> persistent extern INDEX sam_iGfxAPI=(INDEX)0;
>
> чт, 14 дек. 2023 г. в 02:56, Thomas Frohwein :
>
>> On Wed, Dec 13, 2023 at 10:34:01AM +0300, Alexander Pavlov wrote:
>> > Hello
>> > You can attach the game log.
>> > ~.local/share/Serious-Engine/serioussam/SeriousSam.log
>> >
>> > It looks like I was in a hurry to add this port to OpenBSD without
>> testing
>> > it on real hardware with Vulkan support. Or Serious Sam has no luck with
>> > integrated Intel graphics. This is already the second case. The first
>> one
>> > had graphic artifacts on the OpenGL renderer.
>>
>> How do you run with opengl? I can't find anything about this in the man
>> page or the README.
>>
>> > I have a video card with Vulkan support on one computer. When I have the
>> > opportunity, I’ll try to test it on it, but for now it’s probably better
>> > for me to close the pull request, because without testing on real
>> hardware
>> > it’s difficult to understand the problem.
>> >
>> > ср, 13 дек. 2023 г. в 00:35, Kirill Bychkov :
>> >
>> > > On Tue, December 12, 2023 22:36, Thomas Frohwein wrote:
>> > > > On Mon, Dec 11, 2023 at 10:52:58PM +0300, Alexander Pavlov wrote:
>> > > >> Hello!
>> > > >> Thanks @Kirill for approved the pull request.
>> > > >>
>> > > >> This is my other ever port for OpenBSD. It's the open source
>> Serious Sam
>> > > >> Classic engine with vulkan support. The port has two renders
>> OpenGL and
>> > > >> Vulkan. Render switches to the game menu.
>> > > >>
>> > > >> Tested using the OpenGL renderer, since I do not have the
>> opportunity to
>> > > >> test the Vulkan renderer on OpenBSD. On the computer where OpenBSD
>> is
>> > > >> installed, the Radeon HD3000 does not support Vulkan.
>> > > >>
>> > > >> I wish to get this added to the official ports tree, so please
>> leave
>> > > >> any feedback or suggestions on what can be improved. Thank you!
>> > > >>
>> > > >> Source on Github:
>> > > >> https://github.com/tx00100xt/SeriousSamClassic-VK
>> > > >
>> > > > Thanks for submitting this:
>> > > >
>> > > >> @conflict serioussam-*
>> > > >
>> > > > Is this needed? We don't have any other serioussam ports as far as I
>> > > > know.
>> > >
>> > > Yep, if we import both ports - serioussam and serioussam-vk
>> > >
>> > > >
>> > > > DESCR should be formatted, typically with fmt -w 80...
>> > > > README also should be formatted similarly. This means that the line
>> > > > with "directories:" should be pulled up, even if you're going with
>> > > > fmt -w 72 only.
>> > > >
>> > > > Looks like SamTFE/Sources/CMakeLists.txt and
>> > > > SamTSE/Sources/CMakeLists.txt add their own -O2 and -pipe which is
>> > > > better left to our ports build system.
>> > > >
>> > > > I'm getting a segfault in vulkan when trying to launch:
>> > > >
>> > > > $ serioussam
>> > > > STUBBED: load window icon in
>> > > >
>> > >
>> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
>> > > > line 176.
>> > > > STUBBED: Need SDL invisible window or something in
>> > > >
>> > >
>> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
>> > > > line 357.
>> > > > STUBBED: !!! FIXME: get the code back in from Ryan's original port.
>> in
>> > > >
>> > >
>> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Base/Stream.cpp,
>> > > > line 995.
>> > > > STUBBED: Report actual SDL device name? in
>> > > >
>> > >
>> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Sound/SoundLibrary.cpp,
>> > > > line 270.
>> > > > Segmentation fault (core dumped)
>> > > >
>> > > > With this here as part of the backtrace:
>> > > >
>> > > > #0  0x04de18ccda2f in ?? () from
>> /usr/X11R6/lib/libvulkan_intel.so
>> > > > No symbol table info available.
>> > > > #1  0x04de18e3dc49 in ?? () from
>> /usr/X11R6/lib/libvulkan_intel.so
>> > > > No symbol table info available.
>> > > > #2  0x04de3b846d14 in SvkMain::InitTexture32Bit(unsigned int&,
>> > > VkFormat,
>> > > > void*, VkExtent2D*, unsigned int, bool) () from
>> > > > /usr/local/lib/libEngine.so.0.0
>> > > >
>> > > > That's on an Intel 

Re: New: games/serioussam-vk

[Alexander Pavlov]

Yes, I noticed this now. My mistake. The sam_iGfxAPI variable is
responsible for this. By default it is equal to one and the vulkan starts.
If the vulkan is not detected when the game starts, then it switches to
OpenGL. This variable can be set to zero in the configuration files, then
the OpenGL render will be the default. This variable can be set to zero in
the configuration file
(~/.local/share/Serious-Engine/serioussam/Scripts/PersistentSymbols.ini)
then the OpenGL render will be the default. This file is generated on first
launch. You need to change the sam_iGfxAPI value to zero or write the line:
persistent extern INDEX sam_iGfxAPI=(INDEX)0;

чт, 14 дек. 2023 г. в 02:56, Thomas Frohwein :

> On Wed, Dec 13, 2023 at 10:34:01AM +0300, Alexander Pavlov wrote:
> > Hello
> > You can attach the game log.
> > ~.local/share/Serious-Engine/serioussam/SeriousSam.log
> >
> > It looks like I was in a hurry to add this port to OpenBSD without
> testing
> > it on real hardware with Vulkan support. Or Serious Sam has no luck with
> > integrated Intel graphics. This is already the second case. The first one
> > had graphic artifacts on the OpenGL renderer.
>
> How do you run with opengl? I can't find anything about this in the man
> page or the README.
>
> > I have a video card with Vulkan support on one computer. When I have the
> > opportunity, I’ll try to test it on it, but for now it’s probably better
> > for me to close the pull request, because without testing on real
> hardware
> > it’s difficult to understand the problem.
> >
> > ср, 13 дек. 2023 г. в 00:35, Kirill Bychkov :
> >
> > > On Tue, December 12, 2023 22:36, Thomas Frohwein wrote:
> > > > On Mon, Dec 11, 2023 at 10:52:58PM +0300, Alexander Pavlov wrote:
> > > >> Hello!
> > > >> Thanks @Kirill for approved the pull request.
> > > >>
> > > >> This is my other ever port for OpenBSD. It's the open source
> Serious Sam
> > > >> Classic engine with vulkan support. The port has two renders OpenGL
> and
> > > >> Vulkan. Render switches to the game menu.
> > > >>
> > > >> Tested using the OpenGL renderer, since I do not have the
> opportunity to
> > > >> test the Vulkan renderer on OpenBSD. On the computer where OpenBSD
> is
> > > >> installed, the Radeon HD3000 does not support Vulkan.
> > > >>
> > > >> I wish to get this added to the official ports tree, so please leave
> > > >> any feedback or suggestions on what can be improved. Thank you!
> > > >>
> > > >> Source on Github:
> > > >> https://github.com/tx00100xt/SeriousSamClassic-VK
> > > >
> > > > Thanks for submitting this:
> > > >
> > > >> @conflict serioussam-*
> > > >
> > > > Is this needed? We don't have any other serioussam ports as far as I
> > > > know.
> > >
> > > Yep, if we import both ports - serioussam and serioussam-vk
> > >
> > > >
> > > > DESCR should be formatted, typically with fmt -w 80...
> > > > README also should be formatted similarly. This means that the line
> > > > with "directories:" should be pulled up, even if you're going with
> > > > fmt -w 72 only.
> > > >
> > > > Looks like SamTFE/Sources/CMakeLists.txt and
> > > > SamTSE/Sources/CMakeLists.txt add their own -O2 and -pipe which is
> > > > better left to our ports build system.
> > > >
> > > > I'm getting a segfault in vulkan when trying to launch:
> > > >
> > > > $ serioussam
> > > > STUBBED: load window icon in
> > > >
> > >
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
> > > > line 176.
> > > > STUBBED: Need SDL invisible window or something in
> > > >
> > >
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
> > > > line 357.
> > > > STUBBED: !!! FIXME: get the code back in from Ryan's original port.
> in
> > > >
> > >
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Base/Stream.cpp,
> > > > line 995.
> > > > STUBBED: Report actual SDL device name? in
> > > >
> > >
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Sound/SoundLibrary.cpp,
> > > > line 270.
> > > > Segmentation fault (core dumped)
> > > >
> > > > With this here as part of the backtrace:
> > > >
> > > > #0  0x04de18ccda2f in ?? () from
> /usr/X11R6/lib/libvulkan_intel.so
> > > > No symbol table info available.
> > > > #1  0x04de18e3dc49 in ?? () from
> /usr/X11R6/lib/libvulkan_intel.so
> > > > No symbol table info available.
> > > > #2  0x04de3b846d14 in SvkMain::InitTexture32Bit(unsigned int&,
> > > VkFormat,
> > > > void*, VkExtent2D*, unsigned int, bool) () from
> > > > /usr/local/lib/libEngine.so.0.0
> > > >
> > > > That's on an Intel Tigerlake integrated GPU.
> > > >
> > > >
> > >
> > >
> > >
>

Re: New: games/serioussam-vk

[Thomas Frohwein]

On Wed, Dec 13, 2023 at 10:34:01AM +0300, Alexander Pavlov wrote:
> Hello
> You can attach the game log.
> ~.local/share/Serious-Engine/serioussam/SeriousSam.log
> 
> It looks like I was in a hurry to add this port to OpenBSD without testing
> it on real hardware with Vulkan support. Or Serious Sam has no luck with
> integrated Intel graphics. This is already the second case. The first one
> had graphic artifacts on the OpenGL renderer.

How do you run with opengl? I can't find anything about this in the man
page or the README.

> I have a video card with Vulkan support on one computer. When I have the
> opportunity, I’ll try to test it on it, but for now it’s probably better
> for me to close the pull request, because without testing on real hardware
> it’s difficult to understand the problem.
> 
> ср, 13 дек. 2023 г. в 00:35, Kirill Bychkov :
> 
> > On Tue, December 12, 2023 22:36, Thomas Frohwein wrote:
> > > On Mon, Dec 11, 2023 at 10:52:58PM +0300, Alexander Pavlov wrote:
> > >> Hello!
> > >> Thanks @Kirill for approved the pull request.
> > >>
> > >> This is my other ever port for OpenBSD. It's the open source Serious Sam
> > >> Classic engine with vulkan support. The port has two renders OpenGL and
> > >> Vulkan. Render switches to the game menu.
> > >>
> > >> Tested using the OpenGL renderer, since I do not have the opportunity to
> > >> test the Vulkan renderer on OpenBSD. On the computer where OpenBSD is
> > >> installed, the Radeon HD3000 does not support Vulkan.
> > >>
> > >> I wish to get this added to the official ports tree, so please leave
> > >> any feedback or suggestions on what can be improved. Thank you!
> > >>
> > >> Source on Github:
> > >> https://github.com/tx00100xt/SeriousSamClassic-VK
> > >
> > > Thanks for submitting this:
> > >
> > >> @conflict serioussam-*
> > >
> > > Is this needed? We don't have any other serioussam ports as far as I
> > > know.
> >
> > Yep, if we import both ports - serioussam and serioussam-vk
> >
> > >
> > > DESCR should be formatted, typically with fmt -w 80...
> > > README also should be formatted similarly. This means that the line
> > > with "directories:" should be pulled up, even if you're going with
> > > fmt -w 72 only.
> > >
> > > Looks like SamTFE/Sources/CMakeLists.txt and
> > > SamTSE/Sources/CMakeLists.txt add their own -O2 and -pipe which is
> > > better left to our ports build system.
> > >
> > > I'm getting a segfault in vulkan when trying to launch:
> > >
> > > $ serioussam
> > > STUBBED: load window icon in
> > >
> > /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
> > > line 176.
> > > STUBBED: Need SDL invisible window or something in
> > >
> > /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
> > > line 357.
> > > STUBBED: !!! FIXME: get the code back in from Ryan's original port. in
> > >
> > /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Base/Stream.cpp,
> > > line 995.
> > > STUBBED: Report actual SDL device name? in
> > >
> > /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Sound/SoundLibrary.cpp,
> > > line 270.
> > > Segmentation fault (core dumped)
> > >
> > > With this here as part of the backtrace:
> > >
> > > #0  0x04de18ccda2f in ?? () from /usr/X11R6/lib/libvulkan_intel.so
> > > No symbol table info available.
> > > #1  0x04de18e3dc49 in ?? () from /usr/X11R6/lib/libvulkan_intel.so
> > > No symbol table info available.
> > > #2  0x04de3b846d14 in SvkMain::InitTexture32Bit(unsigned int&,
> > VkFormat,
> > > void*, VkExtent2D*, unsigned int, bool) () from
> > > /usr/local/lib/libEngine.so.0.0
> > >
> > > That's on an Intel Tigerlake integrated GPU.
> > >
> > >
> >
> >
> >

Re: New: games/serioussam-vk

[Alexander Pavlov]

Hello
You can attach the game log.
~.local/share/Serious-Engine/serioussam/SeriousSam.log

It looks like I was in a hurry to add this port to OpenBSD without testing
it on real hardware with Vulkan support. Or Serious Sam has no luck with
integrated Intel graphics. This is already the second case. The first one
had graphic artifacts on the OpenGL renderer.

I have a video card with Vulkan support on one computer. When I have the
opportunity, I’ll try to test it on it, but for now it’s probably better
for me to close the pull request, because without testing on real hardware
it’s difficult to understand the problem.

ср, 13 дек. 2023 г. в 00:35, Kirill Bychkov :

> On Tue, December 12, 2023 22:36, Thomas Frohwein wrote:
> > On Mon, Dec 11, 2023 at 10:52:58PM +0300, Alexander Pavlov wrote:
> >> Hello!
> >> Thanks @Kirill for approved the pull request.
> >>
> >> This is my other ever port for OpenBSD. It's the open source Serious Sam
> >> Classic engine with vulkan support. The port has two renders OpenGL and
> >> Vulkan. Render switches to the game menu.
> >>
> >> Tested using the OpenGL renderer, since I do not have the opportunity to
> >> test the Vulkan renderer on OpenBSD. On the computer where OpenBSD is
> >> installed, the Radeon HD3000 does not support Vulkan.
> >>
> >> I wish to get this added to the official ports tree, so please leave
> >> any feedback or suggestions on what can be improved. Thank you!
> >>
> >> Source on Github:
> >> https://github.com/tx00100xt/SeriousSamClassic-VK
> >
> > Thanks for submitting this:
> >
> >> @conflict serioussam-*
> >
> > Is this needed? We don't have any other serioussam ports as far as I
> > know.
>
> Yep, if we import both ports - serioussam and serioussam-vk
>
> >
> > DESCR should be formatted, typically with fmt -w 80...
> > README also should be formatted similarly. This means that the line
> > with "directories:" should be pulled up, even if you're going with
> > fmt -w 72 only.
> >
> > Looks like SamTFE/Sources/CMakeLists.txt and
> > SamTSE/Sources/CMakeLists.txt add their own -O2 and -pipe which is
> > better left to our ports build system.
> >
> > I'm getting a segfault in vulkan when trying to launch:
> >
> > $ serioussam
> > STUBBED: load window icon in
> >
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
> > line 176.
> > STUBBED: Need SDL invisible window or something in
> >
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
> > line 357.
> > STUBBED: !!! FIXME: get the code back in from Ryan's original port. in
> >
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Base/Stream.cpp,
> > line 995.
> > STUBBED: Report actual SDL device name? in
> >
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Sound/SoundLibrary.cpp,
> > line 270.
> > Segmentation fault (core dumped)
> >
> > With this here as part of the backtrace:
> >
> > #0  0x04de18ccda2f in ?? () from /usr/X11R6/lib/libvulkan_intel.so
> > No symbol table info available.
> > #1  0x04de18e3dc49 in ?? () from /usr/X11R6/lib/libvulkan_intel.so
> > No symbol table info available.
> > #2  0x04de3b846d14 in SvkMain::InitTexture32Bit(unsigned int&,
> VkFormat,
> > void*, VkExtent2D*, unsigned int, bool) () from
> > /usr/local/lib/libEngine.so.0.0
> >
> > That's on an Intel Tigerlake integrated GPU.
> >
> >
>
>
>

Re: New: games/serioussam-vk

[Kirill Bychkov]

On Tue, December 12, 2023 22:36, Thomas Frohwein wrote:
> On Mon, Dec 11, 2023 at 10:52:58PM +0300, Alexander Pavlov wrote:
>> Hello!
>> Thanks @Kirill for approved the pull request.
>>
>> This is my other ever port for OpenBSD. It's the open source Serious Sam
>> Classic engine with vulkan support. The port has two renders OpenGL and
>> Vulkan. Render switches to the game menu.
>>
>> Tested using the OpenGL renderer, since I do not have the opportunity to
>> test the Vulkan renderer on OpenBSD. On the computer where OpenBSD is
>> installed, the Radeon HD3000 does not support Vulkan.
>>
>> I wish to get this added to the official ports tree, so please leave
>> any feedback or suggestions on what can be improved. Thank you!
>>
>> Source on Github:
>> https://github.com/tx00100xt/SeriousSamClassic-VK
>
> Thanks for submitting this:
>
>> @conflict serioussam-*
>
> Is this needed? We don't have any other serioussam ports as far as I
> know.

Yep, if we import both ports - serioussam and serioussam-vk

>
> DESCR should be formatted, typically with fmt -w 80...
> README also should be formatted similarly. This means that the line
> with "directories:" should be pulled up, even if you're going with
> fmt -w 72 only.
>
> Looks like SamTFE/Sources/CMakeLists.txt and
> SamTSE/Sources/CMakeLists.txt add their own -O2 and -pipe which is
> better left to our ports build system.
>
> I'm getting a segfault in vulkan when trying to launch:
>
> $ serioussam
> STUBBED: load window icon in
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
> line 176.
> STUBBED: Need SDL invisible window or something in
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
> line 357.
> STUBBED: !!! FIXME: get the code back in from Ryan's original port. in
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Base/Stream.cpp,
> line 995.
> STUBBED: Report actual SDL device name? in
> /usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Sound/SoundLibrary.cpp,
> line 270.
> Segmentation fault (core dumped)
>
> With this here as part of the backtrace:
>
> #0  0x04de18ccda2f in ?? () from /usr/X11R6/lib/libvulkan_intel.so
> No symbol table info available.
> #1  0x04de18e3dc49 in ?? () from /usr/X11R6/lib/libvulkan_intel.so
> No symbol table info available.
> #2  0x04de3b846d14 in SvkMain::InitTexture32Bit(unsigned int&, VkFormat,
> void*, VkExtent2D*, unsigned int, bool) () from
> /usr/local/lib/libEngine.so.0.0
>
> That's on an Intel Tigerlake integrated GPU.
>
>

Re: New: games/serioussam-vk

[Thomas Frohwein]

On Mon, Dec 11, 2023 at 10:52:58PM +0300, Alexander Pavlov wrote:
> Hello!
> Thanks @Kirill for approved the pull request.
> 
> This is my other ever port for OpenBSD. It's the open source Serious Sam
> Classic engine with vulkan support. The port has two renders OpenGL and
> Vulkan. Render switches to the game menu.
> 
> Tested using the OpenGL renderer, since I do not have the opportunity to
> test the Vulkan renderer on OpenBSD. On the computer where OpenBSD is
> installed, the Radeon HD3000 does not support Vulkan.
> 
> I wish to get this added to the official ports tree, so please leave
> any feedback or suggestions on what can be improved. Thank you!
> 
> Source on Github:
> https://github.com/tx00100xt/SeriousSamClassic-VK

Thanks for submitting this:

> @conflict serioussam-*

Is this needed? We don't have any other serioussam ports as far as I
know.

DESCR should be formatted, typically with fmt -w 80...
README also should be formatted similarly. This means that the line
with "directories:" should be pulled up, even if you're going with
fmt -w 72 only.

Looks like SamTFE/Sources/CMakeLists.txt and
SamTSE/Sources/CMakeLists.txt add their own -O2 and -pipe which is
better left to our ports build system.

I'm getting a segfault in vulkan when trying to launch:

$ serioussam
STUBBED: load window icon in 
/usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
 line 176.
STUBBED: Need SDL invisible window or something in 
/usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/SeriousSam/MainWindow.cpp,
 line 357.
STUBBED: !!! FIXME: get the code back in from Ryan's original port. in 
/usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Base/Stream.cpp,
 line 995.
STUBBED: Report actual SDL device name? in 
/usr/ports/pobj/serioussam-vk-1.10.6c/SeriousSamClassic-VK-1.10.6c/SamTFE/Sources/Engine/Sound/SoundLibrary.cpp,
 line 270.
Segmentation fault (core dumped)

With this here as part of the backtrace:

#0  0x04de18ccda2f in ?? () from /usr/X11R6/lib/libvulkan_intel.so
No symbol table info available.
#1  0x04de18e3dc49 in ?? () from /usr/X11R6/lib/libvulkan_intel.so
No symbol table info available.
#2  0x04de3b846d14 in SvkMain::InitTexture32Bit(unsigned int&, VkFormat, 
void*, VkExtent2D*, unsigned int, bool) () from /usr/local/lib/libEngine.so.0.0

That's on an Intel Tigerlake integrated GPU.

Re: New: games/serioussam

[Kirill Bychkov]

On Sun, December 3, 2023 22:18, Stefan Hagen wrote:
> Stefan Hagen wrote (2023-12-03 18:38 CET):
>> Morgan Aldridge wrote (2023-12-03 17:04 CET):
>> > On Sun, Dec 3, 2023 at 09:57 Александр Павлов 
>> wrote:
>> >
>> > > Hello!
>> > > This is my first ever port for OpenBSD. It's the open source Serious Sam
>> > > Classic engine.
>> > >
>> > > Tested on amd64 and i386 architectures. Works just as well as on Linux
>> or
>> > > FreeBSD.
>> > >
>> > > I wish to get this added to the official ports tree, so please leave
>> > > any feedback or suggestions on what can be improved. Thank you!
>> > >
>> > > Source on Github:
>> > > https://github.com/tx00100xt/SeriousSamClassic
>> > >
>> >
>> > Excellent and thanks! We were just discussing your work in #openbsd-gamin
>> > on irc.libera.chat yesterday.
>> >
>> > There was no attachment for your submission and a .tgz containing the new
>> > port directory is preferred for new ports. I saw you submitted a PR to an
>> > individual's WIP ports repo on GitHub:
>> >
>> > 
>> >
>> > I haven't tested, but I'd suggest tarballing that (less the 'FINISHED'
>> > file) and attaching it here for easier testing by porters.
>>
>> Ahh so many memories. I played quite a bit of ssam on lan parties.
>> Back then, when monitors were small and heavy. :-)
>>
>> Tarball attached with the following changes:
>>
>> - changed to DIST_TUPLE
>> - Added missing RUN_DEPENDS
>> - Uncommented the aarch64 + armv7 bits (let's try to build it)
>> - Removed REVISION (not needed on a new port)
>> - simplified the README
>> - some indentation changes
>> - update-plist (for @tag update-desktop-database after adding RDEPS)
>
> I forgot to remove ONLY_FOR_ARCH, thanks mbuhl@ for spotting.
>
> There was also a trailing \ at LIB_DEPENDS.
> While there, I also sorted *DEPENDS and WANTLIB.
> And I changed the COMMENT wording to read a bit more fluent in English.
>
> @Alexander: This is a pretty good first contribution. All the stuff I'm
> changeing is pretty much nit picking. The port worked on the first try!
>
> Updated tarball attached.
>
> Stefan
>
>
Hi!
Some tweaks:
 - no need for N variable which is used only once
 - no need for HOMEPAGE - its dealt with TUPLE mechanics
 - No need fir DISTNAME
 - no need for @conflict marker in PLIST
 - fix typo in README
 - add a comment for COMPILER
 - set SHARED_LIBS for lib versioning

Works fine for me on amd64. ok kirby@ to import.
Updated tarball attached.

serioussam.tar.gz
Description: application/gzip

Re: New: games/serioussam

[Александр Павлов]

Thanks for the quick reply.

Thanks @Stefan for the archive with fixed files.
I will add all the fixes from Stefan to the pull request at
https://github.com/jasperla/openbsd-wip/pull/170

There is a known problem on the aarch64 architecture in Serious Sam The
Second Encounter. Game freezes on the splash screen when you select New
Game in the menu. The solution is to edit or create a new
SamTSE/Scripts/Game_startup.ini file with the line sam_strFirstLevel =
"Levels\\LevelsMP\\1_1_Palenque.wld;]

Serious Sam's code is portable, so adding supported architectures is not
difficult. Need to add the architecture definition here:
https://github.com/tx00100xt/SeriousSamClassic/blob/70b3d5eb58be32501fd7c1c3fd2a11e911f69db4/SamTFE/Sources/Engine/Base/Types.h#L79
And add compiler flags here:
https://github.com/tx00100xt/SeriousSamClassic/blob/70b3d5eb58be32501fd7c1c3fd2a11e911f69db4/SamTFE/Sources/CMakeLists.txt#L355
For such an extraordinary architecture as e2k, it was enough to specify the
definition in Engine/Base/Types.h and the game started.

If you need any more fixes please write.
Best Regards,
Alexander.

пн, 4 дек. 2023 г. в 00:59, Kirill Bychkov :

> On Sun, December 3, 2023 17:53, Александр Павлов wrote:
> > Hello!
> > This is my first ever port for OpenBSD. It's the open source Serious Sam
> > Classic engine.
> >
> > Tested on amd64 and i386 architectures. Works just as well as on Linux or
> > FreeBSD.
> >
> > I wish to get this added to the official ports tree, so please leave
> > any feedback or suggestions on what can be improved. Thank you!
> >
> > Source on Github:
> > https://github.com/tx00100xt/SeriousSamClassic
> >
> Damn... My school-days... Will take a look in a few days.
>
>

Re: New: games/serioussam

[Kirill Bychkov]

On Sun, December 3, 2023 17:53, Александр Павлов wrote:
> Hello!
> This is my first ever port for OpenBSD. It's the open source Serious Sam
> Classic engine.
>
> Tested on amd64 and i386 architectures. Works just as well as on Linux or
> FreeBSD.
>
> I wish to get this added to the official ports tree, so please leave
> any feedback or suggestions on what can be improved. Thank you!
>
> Source on Github:
> https://github.com/tx00100xt/SeriousSamClassic
>
Damn... My school-days... Will take a look in a few days.

Re: New: games/serioussam

[Stefan Hagen]

Stefan Hagen wrote (2023-12-03 18:38 CET):
> Morgan Aldridge wrote (2023-12-03 17:04 CET):
> > On Sun, Dec 3, 2023 at 09:57 Александр Павлов  wrote:
> > 
> > > Hello!
> > > This is my first ever port for OpenBSD. It's the open source Serious Sam
> > > Classic engine.
> > >
> > > Tested on amd64 and i386 architectures. Works just as well as on Linux or
> > > FreeBSD.
> > >
> > > I wish to get this added to the official ports tree, so please leave
> > > any feedback or suggestions on what can be improved. Thank you!
> > >
> > > Source on Github:
> > > https://github.com/tx00100xt/SeriousSamClassic
> > >
> > 
> > Excellent and thanks! We were just discussing your work in #openbsd-gamin
> > on irc.libera.chat yesterday.
> > 
> > There was no attachment for your submission and a .tgz containing the new
> > port directory is preferred for new ports. I saw you submitted a PR to an
> > individual's WIP ports repo on GitHub:
> > 
> > 
> > 
> > I haven't tested, but I'd suggest tarballing that (less the 'FINISHED'
> > file) and attaching it here for easier testing by porters.
> 
> Ahh so many memories. I played quite a bit of ssam on lan parties.
> Back then, when monitors were small and heavy. :-)
> 
> Tarball attached with the following changes:
> 
> - changed to DIST_TUPLE
> - Added missing RUN_DEPENDS
> - Uncommented the aarch64 + armv7 bits (let's try to build it)
> - Removed REVISION (not needed on a new port)
> - simplified the README
> - some indentation changes
> - update-plist (for @tag update-desktop-database after adding RDEPS)

I forgot to remove ONLY_FOR_ARCH, thanks mbuhl@ for spotting.

There was also a trailing \ at LIB_DEPENDS.
While there, I also sorted *DEPENDS and WANTLIB.
And I changed the COMMENT wording to read a bit more fluent in English.

@Alexander: This is a pretty good first contribution. All the stuff I'm
changeing is pretty much nit picking. The port worked on the first try!

Updated tarball attached.

Stefan



serioussam.tgz
Description: application/tar-gz

Re: New: games/serioussam

[Stefan Hagen]

Morgan Aldridge wrote (2023-12-03 17:04 CET):
> On Sun, Dec 3, 2023 at 09:57 Александр Павлов  wrote:
> 
> > Hello!
> > This is my first ever port for OpenBSD. It's the open source Serious Sam
> > Classic engine.
> >
> > Tested on amd64 and i386 architectures. Works just as well as on Linux or
> > FreeBSD.
> >
> > I wish to get this added to the official ports tree, so please leave
> > any feedback or suggestions on what can be improved. Thank you!
> >
> > Source on Github:
> > https://github.com/tx00100xt/SeriousSamClassic
> >
> 
> Excellent and thanks! We were just discussing your work in #openbsd-gamin
> on irc.libera.chat yesterday.
> 
> There was no attachment for your submission and a .tgz containing the new
> port directory is preferred for new ports. I saw you submitted a PR to an
> individual's WIP ports repo on GitHub:
> 
> 
> 
> I haven't tested, but I'd suggest tarballing that (less the 'FINISHED'
> file) and attaching it here for easier testing by porters.

Ahh so many memories. I played quite a bit of ssam on lan parties.
Back then, when monitors were small and heavy. :-)

Tarball attached with the following changes:

- changed to DIST_TUPLE
- Added missing RUN_DEPENDS
- Uncommented the aarch64 + armv7 bits (let's try to build it)
- Removed REVISION (not needed on a new port)
- simplified the README
- some indentation changes
- update-plist (for @tag update-desktop-database after adding RDEPS)

Best Regards,
Stefan


serioussam.tgz
Description: application/tar-gz

Re: New: games/serioussam

[Morgan Aldridge]

On Sun, Dec 3, 2023 at 09:57 Александр Павлов  wrote:

> Hello!
> This is my first ever port for OpenBSD. It's the open source Serious Sam
> Classic engine.
>
> Tested on amd64 and i386 architectures. Works just as well as on Linux or
> FreeBSD.
>
> I wish to get this added to the official ports tree, so please leave
> any feedback or suggestions on what can be improved. Thank you!
>
> Source on Github:
> https://github.com/tx00100xt/SeriousSamClassic
>

Excellent and thanks! We were just discussing your work in #openbsd-gamin
on irc.libera.chat yesterday.

There was no attachment for your submission and a .tgz containing the new
port directory is preferred for new ports. I saw you submitted a PR to an
individual's WIP ports repo on GitHub:



I haven't tested, but I'd suggest tarballing that (less the 'FINISHED'
file) and attaching it here for easier testing by porters.

Morgan