max-solitare1 #362

Merged
Burer merged 34 commits from Max/kolibrios:max-solitare1 into main 2026-04-11 12:04:34 +00:00
Contributor

Sorry for the confusion, I'm learning this git.
This is final version. Bitmaps free license.

Sorry for the confusion, I'm learning this git. This is final version. Bitmaps free license.
Max added 4 commits 2026-03-11 16:51:10 +00:00
Max requested review from Burer 2026-03-11 16:51:10 +00:00
Max requested review from mxlgv 2026-03-11 16:51:12 +00:00
Max added 1 commit 2026-03-12 07:12:30 +00:00
Update data/Tupfile.lua
Build system / Check kernel codestyle (pull_request) Successful in 1m17s
Build system / Build (pull_request) Successful in 12m12s
b203b39d18
Max added 1 commit 2026-03-12 07:22:53 +00:00
Upload files to "data"
Build system / Build (pull_request) Failing after 1m22s
Build system / Check kernel codestyle (pull_request) Successful in 1m41s
7ec5e099bc
Add my game
Max added 1 commit 2026-03-12 09:52:32 +00:00
Upload files to "data"
Build system / Check kernel codestyle (pull_request) Failing after 3s
Build system / Build (pull_request) Failing after 51s
aabd819b23
Max added 1 commit 2026-03-12 09:54:17 +00:00
Merge branch 'main' into max-solitare1
Build system / Build (pull_request) Failing after 11s
Build system / Check kernel codestyle (pull_request) Successful in 26s
d15a2d4f22
Max added 1 commit 2026-03-12 10:00:03 +00:00
Upload files to "programs/games/solitare"
Build system / Build (pull_request) Failing after 2s
Build system / Check kernel codestyle (pull_request) Successful in 1m36s
01a976b171
Max added 1 commit 2026-03-12 10:09:14 +00:00
Update programs/games/solitare/solitare.asm
Build system / Build (pull_request) Failing after 3s
Build system / Check kernel codestyle (pull_request) Successful in 54s
795ba3dfc1
Author
Contributor

Is it a problem on the server or in my files?

Is it a problem on the server or in my files?
Max added 1 commit 2026-03-12 13:51:26 +00:00
Merge branch 'main' into max-solitare1
Build system / Check kernel codestyle (pull_request) Successful in 1m41s
Build system / Build (pull_request) Failing after 2m15s
c232ed3de1
Author
Contributor

the program is looking for the file 'icons16.png' and you changed it to 'icons18.png'

the program is looking for the file 'icons16.png' and you changed it to 'icons18.png'
Max added 1 commit 2026-03-12 15:34:19 +00:00
Update data/Tupfile.lua
Build system / Check kernel codestyle (pull_request) Successful in 31s
Build system / Build (pull_request) Failing after 8m12s
65e08fbe38
Author
Contributor

What should I change to make the game (if it's too big) be compiled in .iso but not in .img?

What should I change to make the game (if it's too big) be compiled in .iso but not in .img?
Max added 1 commit 2026-03-12 16:18:37 +00:00
Upload files to "data"
Build system / Check kernel codestyle (pull_request) Successful in 25s
Build system / Build (pull_request) Failing after 5m1s
fc9fd1776b
Try to build my game in iso but not on img
Max added 1 commit 2026-03-12 16:34:29 +00:00
Upload files to "data"
Build system / Build (pull_request) Failing after 3m26s
Build system / Check kernel codestyle (pull_request) Successful in 1m17s
2b3b66594d
Max added 1 commit 2026-03-12 16:48:59 +00:00
Upload files to "data"
Build system / Check kernel codestyle (pull_request) Successful in 53s
Build system / Build (pull_request) Failing after 7m42s
267373c7c5
Max added 1 commit 2026-03-12 17:22:09 +00:00
Upload files to "data"
Build system / Check kernel codestyle (pull_request) Successful in 1m40s
Build system / Build (pull_request) Failing after 3m32s
4b7174b96d
Max added 1 commit 2026-03-12 17:50:00 +00:00
Upload files to "data"
Build system / Check kernel codestyle (pull_request) Successful in 1m9s
Build system / Build (pull_request) Failing after 8m23s
fc09e860a8
Max added 1 commit 2026-03-13 09:24:14 +00:00
Upload files to "programs/games/solitare"
Build system / Check kernel codestyle (pull_request) Successful in 1m5s
Build system / Build (pull_request) Failing after 7m20s
7cee8d29f7
Max added 1 commit 2026-03-13 19:20:13 +00:00
Update programs/games/solitare/solitare.asm
Build system / Check kernel codestyle (pull_request) Successful in 39s
Build system / Build (pull_request) Successful in 9m52s
48bbc800b9
Burer approved these changes 2026-03-14 16:56:09 +00:00
Owner

Hi!
Looks fine enough to me now.
Let's wait for the second approve.
And thank you for nice game!

Hi! Looks fine enough to me now. Let's wait for the second approve. And thank you for nice game!
Max added 1 commit 2026-03-17 09:30:52 +00:00
Merge branch 'main' into max-solitare1
Build system / Check kernel codestyle (pull_request) Successful in 2m5s
Build system / Build (pull_request) Successful in 11m39s
cda7cb7c81
Max requested review from Doczom 2026-03-17 09:32:43 +00:00
Max added 1 commit 2026-03-23 20:02:19 +00:00
Upload files to "data"
Build system / Check kernel codestyle (pull_request) Successful in 1m33s
Build system / Build (pull_request) Failing after 2m4s
5f135c5c6b
Max added 1 commit 2026-03-23 20:03:17 +00:00
merge upstream
Build system / Check kernel codestyle (pull_request) Successful in 1m14s
Build system / Build (pull_request) Successful in 10m38s
1ee9e02e3a
Max added 1 commit 2026-03-23 21:21:50 +00:00
Update data/Tupfile.lua
Build system / Check kernel codestyle (pull_request) Successful in 1m10s
Build system / Build (pull_request) Successful in 10m6s
3c6b508209
Max added 1 commit 2026-03-23 21:34:15 +00:00
Update data/Tupfile.lua
Build system / Check kernel codestyle (pull_request) Successful in 1m9s
Build system / Build (pull_request) Failing after 1m11s
3423ad2ed3
Max added 1 commit 2026-03-23 21:37:46 +00:00
Update data/Tupfile.lua
Build system / Check kernel codestyle (pull_request) Successful in 49s
Build system / Build (pull_request) Successful in 10m56s
4729cfa32c
Max requested review from Burer 2026-03-26 14:17:54 +00:00
Max requested review from dunkaist 2026-03-26 14:19:10 +00:00
mxlgv requested changes 2026-03-27 01:37:08 +00:00
@@ -0,0 +1,1575 @@
; SPDX-License-Identifier: GPL-2.0-only
; Program - Solitaire card game
; Copyright (C) 2026 KolibriOS team
; Author: Max, <snapemax@wp.pl>
Owner

Please remove ,

Please remove `,`
@@ -0,0 +2,4 @@
; Program - Solitaire card game
; Copyright (C) 2026 KolibriOS team
; Author: Max, <snapemax@wp.pl>
; Graphics: Designed by rawpixel.com / Freepik
Owner

From this it is not at all clear who owns these images and under what license they are distributed

From this it is not at all clear who owns these images and under what license they are distributed
Author
Contributor

is this their license, should I include it somewhere?

is this their license, should I include it somewhere?
Owner

I think you should include this custom license file to your commit and explicitly mention it here like this:
; Graphics: Designed by rawpixel.com / Freepik (see 'License free.txt')

Looks good otherwise, thank you

I think you should include this custom license file to your commit and explicitly mention it here like this: ; Graphics: Designed by rawpixel.com / Freepik (see 'License free.txt') Looks good otherwise, thank you
Author
Contributor

I made the changes you wrote about.

I made the changes you wrote about.
@@ -0,0 +56,4 @@
dd 0, 0
start:
mov esp, stacktop
Owner

At startup, the stack address is taken from the header. This is a meaningless action.

At startup, the stack address is taken from the header. This is a meaningless action.
@@ -0,0 +1464,4 @@
txt_replay db 'REPLAY', 0
txt_exit db 'EXIT', 0
score_str rb 16
score dd 0
Owner

It seems many of these things can come after i_end:

It seems many of these things can come after i_end:
Max added 1 commit 2026-03-27 09:15:19 +00:00
Upload files to "programs/games/solitare"
Build system / Check kernel codestyle (pull_request) Successful in 1m59s
Build system / Build (pull_request) Successful in 11m44s
e64a8bcf68
the requested changes were made
Max requested review from mxlgv 2026-03-27 14:37:24 +00:00
Max removed review request for Doczom 2026-03-27 14:38:18 +00:00
Max removed review request for Burer 2026-03-27 14:38:22 +00:00
Max requested review from Burer 2026-03-27 14:38:27 +00:00
Max removed review request for dunkaist 2026-03-27 14:39:00 +00:00
Max requested review from dunkaist 2026-03-27 14:39:33 +00:00
Max added 1 commit 2026-03-29 19:08:54 +00:00
Merge branch 'main' into max-solitare1
Build system / Check kernel codestyle (pull_request) Successful in 1m35s
Build system / Build (pull_request) Successful in 20m25s
ec65012535
Max added 1 commit 2026-04-01 09:12:13 +00:00
Merge branch 'main' into max-solitare1
Build system / Check kernel codestyle (pull_request) Successful in 42s
Build system / Build (pull_request) Successful in 27m39s
d95cca5b2a
Max added 1 commit 2026-04-03 09:30:26 +00:00
Upload files to "data"
Build system / Check kernel codestyle (pull_request) Successful in 51s
Build system / Build (pull_request) Failing after 2m30s
5b64b0a7cf
Max added 1 commit 2026-04-03 09:30:54 +00:00
merge upstream
Build system / Check kernel codestyle (pull_request) Successful in 1m21s
Build system / Build (pull_request) Successful in 37m25s
f20dc97c7e
Max added 1 commit 2026-04-10 08:40:44 +00:00
merge upstream
Build system / Check kernel codestyle (pull_request) Successful in 2m14s
Build system / Build (pull_request) Successful in 40m4s
aa12adf656
Max added 1 commit 2026-04-10 08:47:42 +00:00
Upload files to "programs/games/solitare"
Build system / Check kernel codestyle (pull_request) Successful in 44s
Build system / Build (pull_request) Failing after 3h6m31s
44b2c27487
Max added 1 commit 2026-04-10 08:50:32 +00:00
Update programs/games/solitare/solitare.asm
Build system / Check kernel codestyle (pull_request) Successful in 59s
Build system / Build (pull_request) Successful in 37m9s
cd1df67ab5
Author
Contributor

Burer can you review again? I made some changes requested by other reviewers.

Burer can you review again? I made some changes requested by other reviewers.
Burer added 1 commit 2026-04-11 10:52:32 +00:00
games/solitare: fix RNG, add to autobuild
Build system / Check kernel codestyle (pull_request) Successful in 26s
Build system / Build (pull_request) Successful in 10m5s
5d66143ca1
Owner

Hi, @Max.
I am going to finally add all three of your games to autobuild and merge into system image.
Sorry for the delay and thank you for your work!
Also, some code modifications will be done, kind of better practicies.
You can check them for future projects, if you want.

Hi, @Max. I am going to finally add all three of your games to autobuild and merge into system image. Sorry for the delay and thank you for your work! Also, some code modifications will be done, kind of better practicies. You can check them for future projects, if you want.
Owner

Totally playable, from what I can see.
Will be merged.
P.S. Personal thanks for Solitaire. Spend hours playing this game my e-book, nice to see proper implementation in KOS.
image.png

Totally playable, from what I can see. Will be merged. P.S. Personal thanks for Solitaire. Spend hours playing this game my e-book, nice to see proper implementation in KOS. <img width="584" alt="image.png" src="attachments/86db508c-c232-450e-b683-53472edef92b">
Burer merged commit 87adc0a1df into main 2026-04-11 12:04:34 +00:00
Burer deleted branch max-solitare1 2026-04-11 12:04:35 +00:00
Burer referenced this issue from a commit 2026-04-11 12:04:35 +00:00
Sign in to join this conversation.
No Reviewers
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: KolibriOS/kolibrios#362