Skip to content

Fix deterministic build#153

Merged
RolandUI merged 2 commits intoGingerPrivacy:masterfrom
luzius1089:fix_det
Aug 29, 2025
Merged

Fix deterministic build#153
RolandUI merged 2 commits intoGingerPrivacy:masterfrom
luzius1089:fix_det

Conversation

@luzius1089
Copy link
Contributor

Fixes: #152

The GingerCommon project missed some configurations to ensure deterministic builds. For example:

For example, one typical diff after disassembly:

Logger.Log("Exception from " + GetType().Name, ex, LogLevel.Warning, "C:\\Users\\Neo\\Documents\\work\\GingerPrivacy\\GingerWallet\\GingerCommon\\Providers\\ExchangeRateProviders\\ExchangeRateProvider.cs", "GetSupportedCurrenciesAsync", 55);

Logger.Log("Exception from " + GetType().Name, ex, LogLevel.Warning, "C:\\Users\\Neo\\Documents\\work\\GingerPrivacy\\GingerWalletClone\\GingerCommon\\Providers\\ExchangeRateProviders\\ExchangeRateProvider.cs", "GetSupportedCurrenciesAsync", 55);

[CallerFilePath] inserts the filepath compile time into the binary, <PathMap> ensures to insert the same path.

After the fix, all the files were identical.

@RolandUI RolandUI merged commit 281fd7c into GingerPrivacy:master Aug 29, 2025
4 of 6 checks passed
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.

Reproducibility Verification Report – Ginger Wallet Desktop v2.0.21

2 participants