https://bugs.winehq.org/show_bug.cgi?id=45208
Zebediah Figura z.figura12@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |WONTFIX
--- Comment #12 from Zebediah Figura z.figura12@gmail.com --- Thank you for your comments, Dmitry. Yes, you are correct in that I should have arranged my patches better, and I should have been more rigorous in testing what APIs do and don't work from a custom action. I thought I had found that MsiSummaryInfoGetProperty() didn't actually work, and thus neglected to do anything with it, but clearly I did so incorrectly; in any case I will try to add some more explicit tests for APIs that shouldn't work.
Nevertheless I'd ask you to please try not to take such an accusatory tone; we are all working on the same project after all :-)
Anyway, with regards to this bug, Hans Leidekker has found that on Windows the ProcessBranding action in fact takes over 5 seconds to execute. This is apparently because it triggers native MSI to rebuild the string pool. Obviously this is not a behaviour we would want to try to match, so I'm sort of inclined to resolve this WONTFIX, unless anyone has any other opinions.