-
-
Notifications
You must be signed in to change notification settings - Fork 173
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.
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
Added codesign and resources #1069
base: master
Are you sure you want to change the base?
Conversation
- for Windows: rehlds/.github#30 and rehlds/.github#29 - for Linux: rehlds/.github#28
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for this functionality!
I need some time (I will gradually review this PR ) to look at this PR in detail.
You can immediately see the violation of the declarative component of the .yml
file.
Tip We collectively came to the conclusion that signing each file inside the archive with .asc files is a bit redundant. For now, we'll just leave the signatures of the final archive and the native Windows ones. |
2b653aa
to
a0c03e9
Compare
Added signing of windows binaries, resolves Add signing of windows binaries .github#30
Added resource metainfo to windows binaries, resolves Add resource metainfo to windows binaries .github#29
Added signing for linux binaries, resolves Add signing for linux binaries .github#28
Caution
This is
infrastructure
update! NO code and NO bugfixes was implemented atReHLDS
at this PR.Also, I know that we have .rc files that are used to create binaries.
The problem is that we have not yet fully decided how their future will develop from a development point of view. Also, now there are only 2 .rc files that are built into hlds.exe and hltv.exe. All other dlls are built naked, which excites antiviruses.
For now, I have chosen to use rcedit, written by the Electron project, which can successfully read and edit resources in compiled files. Which is what I am actually doing.
For now, I suggest leaving it as is. To launch the mechanism. And optimize it in subsequent iterations of development, to move the progress forward a little.