KolibriOS/kolibrios
15
24
Fork 61
Code Issues 132 Pull Requests 32 Actions Projects 1 Releases Wiki Activity

programs/fs: Post-SVN tidy (part 1) #423

Merged
Burer merged 4 commits from ace-dent/kolibrios-gitea:PR-TidySVN-fs-1 into main 2026-04-16 16:33:59 +00:00
Conversation 4 Commits 4 Files Changed 52
+417 -408
ace-dent commented 2026-04-14 19:28:38 +00:00
Member
Copy Link
Copy Source
  • Move source code from trunk into root directory for each program.
  • Update build files and ASM include paths. Add SPDX license header where appropriate.
  • Note: Line endings standardised from CRLF > LF, so best to view diffs with whitespace changes hidden.

(Work towards #75, point 3)

- Move source code from `trunk` into root directory for each program. - Update build files and ASM include paths. Add SPDX license header where appropriate. - Note: Line endings standardised from `CRLF` > `LF`, so best to view diffs with whitespace changes hidden. (Work towards #75, point 3)
ace-dent added 3 commits 2026-04-14 19:28:41 +00:00
fs/copyr: Post-SVN tidy 04b28808a4 
- Move source code from `trunk` into program root directory.
- Update build files and ASM include paths.
fs/kfm: Post-SVN tidy 24db4c3ad3 
- Move source code from `trunk` into program root directory.
- Update build files and ASM include paths.
- Note: Line endings standardised from `CRLF` > `LF`, so best to view diffs with whitespace changes hidden.
programs/fs: Post-SVN tidy
Build system / Check kernel codestyle (pull_request) Successful in 1m3s
Details
Build system / Build (pull_request) Successful in 10m59s
Details
a85c94b815 
- Tidy `build_all.bat` build file; comment out absent programs.
ace-dent commented 2026-04-14 19:35:20 +00:00
Author
Member
Copy Link
Copy Source

This PR is split (/cherry picked) from the monolithic PR #250 - for easier merging! 🤞
This is part 1.

Updates these programs (for testing):

  • fs/copyr (not in standard builds?)
  • fs/kfm (the older original, not KFM2!)
This PR is split (/cherry picked) from the monolithic PR #250 - for easier merging! 🤞 This is part 1. Updates these programs (for testing): - `fs/copyr` (not in standard builds?) - `fs/kfm` (the older original, not KFM2!)
ace-dent added the Category/Applications
Kind
Build
Priority
Low
PR
Review required
 labels 2026-04-14 19:35:46 +00:00
ace-dent requested review from Burer 2026-04-14 19:35:53 +00:00
ace-dent requested review from Doczom 2026-04-14 19:35:53 +00:00
ace-dent requested review from dunkaist 2026-04-14 19:35:53 +00:00
ace-dent requested review from Sweetbread 2026-04-14 19:35:54 +00:00
Doczom approved these changes 2026-04-15 05:18:50 +00:00
Burer requested changes 2026-04-16 12:49:10 +00:00
Dismissed
programs/build_all.bat
@@ -113,3 +113,2 @@
echo *
fasm fs\copy2\trunk\copy2.asm %BIN%\copy2
fasm fs\copyr\trunk\copyr.asm %BIN%\copyr
rem fasm fs\copy2\trunk\copy2.asm %BIN%\copy2
Burer commented 2026-04-16 12:49:05 +00:00
Owner
Copy Link
Copy Source

build_all.bat now comments out copy2 compilation, but still runs kpack %BIN%\copy2 later.
On a clean build this can fail because %BIN%\copy2 is never produced.
Please either remove/guard the kpack line, or keep the compile step enabled.

`build_all.bat` now comments out `copy2` compilation, but still runs `kpack %BIN%\copy2` later. On a clean build this can fail because `%BIN%\copy2` is never produced. Please either remove/guard the `kpack` line, or keep the compile step enabled.
ace-dent commented 2026-04-16 16:12:12 +00:00
Author
Member
Copy Link
Copy Source

Fixed by commenting out kpack line.

Fixed by commenting out kpack line.
ace-dent marked this conversation as resolved
ace-dent added 1 commit 2026-04-16 16:13:10 +00:00
programs/fs: Post-SVN tidy
Build system / Check kernel codestyle (pull_request) Successful in 2m37s
Details
Build system / Build (pull_request) Successful in 13m29s
Details
89eec4887a 
- Tidy `build_all.bat` build file; comment out absent programs.
ace-dent requested review from Burer 2026-04-16 16:13:32 +00:00
Burer approved these changes 2026-04-16 16:33:08 +00:00
Burer merged commit bd548e0373 into main 2026-04-16 16:33:59 +00:00
ace-dent removed the
PR
Review required
 label 2026-04-16 16:43:17 +00:00
ace-dent removed review request for dunkaist 2026-04-16 16:43:21 +00:00
ace-dent removed review request for Sweetbread 2026-04-16 16:43:24 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Doczom
Burer
Labels
Clear labels
C
Category/Applications
Category/Drivers
Category/General
Category/Kernel
Category/Libraries
Eolite
FASM
FS
GSoC
This issue in GSoC program
 HardwareTested
HLL
Influence/Settings
Influence/Text/TYPO
IRCC
Kernel
Kind
Breaking
Breaking change that won't be backward compatible
Kind
Bug
Something is not working
Kind
Build
Kind
Documentation
Documentation changes
Kind
Enhancement
Improve existing functionality
Kind
Feature
New functionality
Kind
Security
This is security issue
Kind
Testing
Issue or pull request related to testing
Pay for the code
Paid task
PR
Conflicts
PR conflicts with main
PR
Dependent
This PR is dependent on another PR
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
PR
Ready to merge
Pull request is ready for merge
PR
Request changes
Changes requested in pull request
PR
Review required
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No Label Category/Applications
Kind
Build
Priority
Low
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ace-dent (Andrew) Burer Doczom (Mikhail Frolov) dunkaist (Ivan B) Egor00f (Егор) hidnplayr IgorA Leency (Kiril Lipatov) mxlgv (Max Logaev) ramenu rgimad sdongles (Serhii Sakhno) Sweetbread (Gleb Zaharov)
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: KolibriOS/kolibrios#423
Delete Branch "ace-dent/kolibrios-gitea:PR-TidySVN-fs-1"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?