Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added Qt Installer Framework #314

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added Qt Installer Framework #314

wants to merge 1 commit into from

Conversation

KOLANICH
Copy link
Contributor

@KOLANICH KOLANICH commented Aug 17, 2020 •
edited
Loading

Since my PR into QIF repo was ignored for years, it seems they are both disinterested in maintaining the spec and disrespectful enough to just ignore PRs. So I post the spec here.

Fixes: #225

generalmimon
generalmimon reviewed Aug 17, 2020
View reviewed changes
archive/qt_installer_framework.ksy
Qt installer framework is a set of libs to make SFX installers. Installers usually contain 7zip-compressed archives. Obviously, Qt installer itself is built using this framework.

Warning 1: KSC has a bug. It makes the computed values be int32_t. Of course their type should be either explicitly specified by a programmer or derived automatically. The workaroind is to just replace all int32_t to int64_t in sources.
Warning 2: don't use this spec on Linux against Qt distribution with overcommit enabled unless you have lot of RAM (> 12 GiB). There is a severe memory leak somewhere (currently I have no idea where exactly). The leak is present in both C++ and python-compiled code. In python even if I have patched the generated source to explicitly free all the `bytes`, `BytesIO`s and `KaitaiStream` objects the leak is still present. At least it is neither in `bytes` nor in `BytesIO`. In C++ I have not patched anything but used move semantics to free the stuff since std::unique_ptr is used. The leak is still present. IDK where it is and how to fix it.
Copy link
Member

@generalmimon generalmimon Aug 17, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please wrap the long lines to be at most 80 characters long, as the KSY Style Guide recommends?

Lines should be wrapped to be 80 columns long. If it doesn’t fit into single line after wrapping, then it’s a multi-line docstring, so use proper multi-line syntax.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am against manual wrapping. Instead wrapping must be done by a text editor depending on window size.

@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 34ab09f to b2ac983 Compare August 20, 2020 13:42
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from b2ac983 to bd0ef27 Compare September 21, 2020 10:42
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from bd0ef27 to 6ce2b88 Compare October 8, 2020 19:00
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 6ce2b88 to 218dc72 Compare October 27, 2020 12:54
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 218dc72 to e0cc3ec Compare November 9, 2020 17:18
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from e0cc3ec to 30fa220 Compare December 15, 2020 17:30
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 30fa220 to 7750b88 Compare December 28, 2020 18:36
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 2 times, most recently from b6d0770 to 2e7ea60 Compare January 18, 2021 17:22
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 2e7ea60 to 5a48c95 Compare February 24, 2021 17:13
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 3 times, most recently from 66046ab to be4f31a Compare March 10, 2021 17:20
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from be4f31a to a05ebc6 Compare March 15, 2021 17:30
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 2 times, most recently from dedbe78 to 89f2ab9 Compare April 19, 2021 18:09
@KOLANICH KOLANICH mentioned this pull request Apr 23, 2021
Closed
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 2 times, most recently from f15c126 to 43f4931 Compare April 30, 2021 14:18
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 43f4931 to 675dc3d Compare June 21, 2021 10:30
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 3 times, most recently from 0185a7f to ec15491 Compare July 5, 2021 17:25
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from ec15491 to fbb0664 Compare July 15, 2021 10:39
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from fbb0664 to 95ff28f Compare August 23, 2021 11:00
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 2 times, most recently from 37ba021 to 9cee143 Compare September 3, 2021 16:56
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 2 times, most recently from d9aa1dc to 3a0553a Compare September 14, 2021 18:51
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 2 times, most recently from 5f1d675 to 4b1c257 Compare October 28, 2021 17:28
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 4b1c257 to 052336d Compare November 8, 2021 14:33
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 052336d to fc8dac2 Compare December 2, 2021 15:51
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from fc8dac2 to 750657d Compare December 21, 2021 10:10
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 750657d to 9d40555 Compare January 1, 2022 21:46
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 9d40555 to 2b1ea4d Compare February 7, 2022 18:40
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 2b1ea4d to 8b3bf55 Compare February 17, 2022 17:57
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 8b3bf55 to bc77a91 Compare March 17, 2022 18:54
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from bc77a91 to 55526ce Compare April 7, 2022 14:10
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 55526ce to 3dc2279 Compare April 15, 2022 18:00
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 2 times, most recently from 8fb22dc to 04b2b3e Compare May 31, 2022 10:47
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 04b2b3e to cde8a7d Compare June 27, 2022 13:11
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from cde8a7d to e99f6cc Compare July 25, 2022 08:01
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from e99f6cc to a8d25c6 Compare October 4, 2022 15:48
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from a8d25c6 to afda369 Compare October 20, 2022 16:34
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from afda369 to 874dfa0 Compare November 7, 2022 09:57
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 874dfa0 to 2e14e10 Compare February 6, 2023 18:17
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 2e14e10 to e7b98d8 Compare March 2, 2023 11:45
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 2 times, most recently from 7188255 to 34ba365 Compare March 20, 2023 13:20
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch 2 times, most recently from 19aba8a to 9b28feb Compare March 28, 2023 15:29
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 9b28feb to 9d990a9 Compare April 7, 2023 16:55
@KOLANICH KOLANICH force-pushed the qt_installer_framework branch from 9d990a9 to 39992b7 Compare June 3, 2023 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@generalmimon generalmimon generalmimon left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Qt Installer Framework
2 participants
@KOLANICH @generalmimon