Skip to content

[RAPPS-DB] Some file sizes corrections#387

Open
xlucn wants to merge 4 commits into
reactos:masterfrom
xlucn:fix_size
Open

[RAPPS-DB] Some file sizes corrections#387
xlucn wants to merge 4 commits into
reactos:masterfrom
xlucn:fix_size

Conversation

@xlucn

@xlucn xlucn commented May 24, 2026

Copy link
Copy Markdown
Contributor

Corrections to the wrong file sizes I could find. All of them are confirmed by both

  • content-length reported by curl -IL $url command, and
  • locally downloaded file size

Feel free to check them again :)

@katahiromz

Copy link
Copy Markdown
Contributor

@xlucn Hi! Thank you for your contribution.

I have a question: Can you give us a Python3 script that automates this size check?

@xlucn

xlucn commented May 24, 2026

Copy link
Copy Markdown
Contributor Author

@xlucn Hi! Thank you for your contribution.

I have a question: Can you give us a Python3 script that automates this size check?

Sure! check.py

This is the python script I am using to check size and hash of the local files. It also output a file which I can feed to aria2 to download the missing or mismatched files. Hope you find something useful in it.
However, currently only the [Section] is considered, not other architectures.

The curl command is done manually. This script only checks local files.

@xlucn

xlucn commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

Beside rebasing to master when needed, do I just need to wait for someone to finally merge this? Sorry it's my first time to contribute here.

@julenuri

julenuri commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

@xlucn Yep, it needs to resolve the iqpuzzle.txt and vc2015 conflict and I have to check the files, I suppose they are okkay, but I have like zero time for me, so I will check this between today and Sunday.

@julenuri julenuri changed the title some file sizes corrections [RAPPS-DB] Some file sizes corrections Jun 10, 2026
@xlucn

xlucn commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Thanks, @whindsaks for merging the master branch!

Following that, I found

  1. iqpuzzle's size does not agree again for the new version 1.5.1. However in the merged branch now it seems fixed. Maybe @whindsaks changed it?
  2. Newly added UT's size does not agree. I have changed it. I just found that size is the total download size including what's downloaded during the run. If that's OK, then exclude/revert my last commit above.

@whindsaks

Copy link
Copy Markdown
Contributor

2. I just found that size is the total download size including what's downloaded during the run. If that's OK, then exclude/revert my last commit above.

Yes, that is on purpose.

Comment thread unrealtournament.txt Outdated
This reverts commit 2bdbe12.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants